WordImpress / Give

includes/class-give-db-customers.php

Doc Comments +2 added lines, -2 removed lines

@@ -282,7 +282,7 @@ 
 	 * @param int $user_id
 	 * @param $old_user_data
 	 *
-	 * @return bool
+	 * @return false|null
 	 */
 	public function update_customer_email_on_user_update( $user_id = 0, $old_user_data ) {
 
@@ -331,7 +331,7 @@ 
 	 * @since  1.0
 	 *
 	 * @param  string $field id or email
-	 * @param  mixed  $value The Customer ID or email to search
+	 * @param  integer  $value The Customer ID or email to search
 	 *
 	 * @return mixed          Upon success, an object of the customer. Upon failure, NULL
 	 */

Spacing +129 added lines, -129 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -33,11 +33,11 @@ 
 
 		global $wpdb;
 
-		$this->table_name  = $wpdb->prefix . 'give_customers';
+		$this->table_name  = $wpdb->prefix.'give_customers';
 		$this->primary_key = 'id';
 		$this->version     = '1.0';
 		
-		add_action( 'profile_update', array( $this, 'update_customer_email_on_user_update' ), 10, 2 );
+		add_action('profile_update', array($this, 'update_customer_email_on_user_update'), 10, 2);
 
 	}
 
@@ -76,7 +76,7 @@ 
 			'purchase_value' => 0.00,
 			'purchase_count' => 0,
 			'notes'          => '',
-			'date_created'   => date( 'Y-m-d H:i:s' ),
+			'date_created'   => date('Y-m-d H:i:s'),
 		);
 	}
 
@@ -86,40 +86,40 @@ 
 	 * @access  public
 	 * @since   1.0
 	 */
-	public function add( $data = array() ) {
+	public function add($data = array()) {
 
 		$defaults = array(
 			'payment_ids' => ''
 		);
 
-		$args = wp_parse_args( $data, $defaults );
+		$args = wp_parse_args($data, $defaults);
 
-		if ( empty( $args['email'] ) ) {
+		if (empty($args['email'])) {
 			return false;
 		}
 
-		if ( ! empty( $args['payment_ids'] ) && is_array( $args['payment_ids'] ) ) {
-			$args['payment_ids'] = implode( ',', array_unique( array_values( $args['payment_ids'] ) ) );
+		if ( ! empty($args['payment_ids']) && is_array($args['payment_ids'])) {
+			$args['payment_ids'] = implode(',', array_unique(array_values($args['payment_ids'])));
 		}
 
-		$customer = $this->get_customer_by( 'email', $args['email'] );
+		$customer = $this->get_customer_by('email', $args['email']);
 
-		if ( $customer ) {
+		if ($customer) {
 			// update an existing customer
 
 			// Update the payment IDs attached to the customer
-			if ( ! empty( $args['payment_ids'] ) ) {
+			if ( ! empty($args['payment_ids'])) {
 
-				if ( empty( $customer->payment_ids ) ) {
+				if (empty($customer->payment_ids)) {
 
 					$customer->payment_ids = $args['payment_ids'];
 
 				} else {
 
-					$existing_ids          = array_map( 'absint', explode( ',', $customer->payment_ids ) );
-					$payment_ids           = array_map( 'absint', explode( ',', $args['payment_ids'] ) );
-					$payment_ids           = array_merge( $payment_ids, $existing_ids );
-					$customer->payment_ids = implode( ',', array_unique( array_values( $payment_ids ) ) );
+					$existing_ids          = array_map('absint', explode(',', $customer->payment_ids));
+					$payment_ids           = array_map('absint', explode(',', $args['payment_ids']));
+					$payment_ids           = array_merge($payment_ids, $existing_ids);
+					$customer->payment_ids = implode(',', array_unique(array_values($payment_ids)));
 
 				}
 
@@ -127,13 +127,13 @@ 
 
 			}
 
-			$this->update( $customer->id, $args );
+			$this->update($customer->id, $args);
 
 			return $customer->id;
 
 		} else {
 
-			return $this->insert( $args, 'customer' );
+			return $this->insert($args, 'customer');
 
 		}
 
@@ -152,20 +152,20 @@ 
 	 *
 	 * @return bool|false|int
 	 */
-	public function delete( $_id_or_email = false ) {
+	public function delete($_id_or_email = false) {
 
-		if ( empty( $_id_or_email ) ) {
+		if (empty($_id_or_email)) {
 			return false;
 		}
 
-		$column   = is_email( $_id_or_email ) ? 'email' : 'id';
-		$customer = $this->get_customer_by( $column, $_id_or_email );
+		$column   = is_email($_id_or_email) ? 'email' : 'id';
+		$customer = $this->get_customer_by($column, $_id_or_email);
 
-		if ( $customer->id > 0 ) {
+		if ($customer->id > 0) {
 
 			global $wpdb;
 
-			return $wpdb->delete( $this->table_name, array( 'id' => $customer->id ), array( '%d' ) );
+			return $wpdb->delete($this->table_name, array('id' => $customer->id), array('%d'));
 
 		} else {
 			return false;
@@ -179,14 +179,14 @@ 
 	 * @access  public
 	 * @since   1.0
 	 */
-	public function exists( $value = '', $field = 'email' ) {
+	public function exists($value = '', $field = 'email') {
 		
 		$columns = $this->get_columns();
-		if ( ! array_key_exists( $field, $columns ) ) {
+		if ( ! array_key_exists($field, $columns)) {
 			return false;
 		}
 
-		return (bool) $this->get_column_by( 'id', $field, $value );
+		return (bool) $this->get_column_by('id', $field, $value);
 
 	}
 
@@ -196,16 +196,16 @@ 
 	 * @access  public
 	 * @since   1.0
 	 */
-	public function attach_payment( $customer_id = 0, $payment_id = 0 ) {
+	public function attach_payment($customer_id = 0, $payment_id = 0) {
 
-		$customer = new Give_Customer( $customer_id );
+		$customer = new Give_Customer($customer_id);
 
-		if ( empty( $customer->id ) ) {
+		if (empty($customer->id)) {
 			return false;
 		}
 
 		// Attach the payment, but don't increment stats, as this function previously did not
-		return $customer->attach_payment( $payment_id, false );
+		return $customer->attach_payment($payment_id, false);
 
 	}
 
@@ -215,16 +215,16 @@ 
 	 * @access  public
 	 * @since   1.0
 	 */
-	public function remove_payment( $customer_id = 0, $payment_id = 0 ) {
+	public function remove_payment($customer_id = 0, $payment_id = 0) {
 
-		$customer = new Give_Customer( $customer_id );
+		$customer = new Give_Customer($customer_id);
 
-		if ( ! $customer ) {
+		if ( ! $customer) {
 			return false;
 		}
 
 		// Remove the payment, but don't decrease stats, as this function previously did not
-		return $customer->remove_payment( $payment_id, false );
+		return $customer->remove_payment($payment_id, false);
 
 	}
 
@@ -236,18 +236,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function increment_stats( $customer_id = 0, $amount = 0.00 ) {
+	public function increment_stats($customer_id = 0, $amount = 0.00) {
 
-		$customer = new Give_Customer( $customer_id );
+		$customer = new Give_Customer($customer_id);
 
-		if ( empty( $customer->id ) ) {
+		if (empty($customer->id)) {
 			return false;
 		}
 
 		$increased_count = $customer->increase_purchase_count();
-		$increased_value = $customer->increase_value( $amount );
+		$increased_value = $customer->increase_value($amount);
 
-		return ( $increased_count && $increased_value ) ? true : false;
+		return ($increased_count && $increased_value) ? true : false;
 
 	}
 
@@ -257,18 +257,18 @@ 
 	 * @access  public
 	 * @since   1.0
 	 */
-	public function decrement_stats( $customer_id = 0, $amount = 0.00 ) {
+	public function decrement_stats($customer_id = 0, $amount = 0.00) {
 
-		$customer = new Give_Customer( $customer_id );
+		$customer = new Give_Customer($customer_id);
 
-		if ( ! $customer ) {
+		if ( ! $customer) {
 			return false;
 		}
 
 		$decreased_count = $customer->decrease_purchase_count();
-		$decreased_value = $customer->decrease_value( $amount );
+		$decreased_value = $customer->decrease_value($amount);
 
-		return ( $decreased_count && $decreased_value ) ? true : false;
+		return ($decreased_count && $decreased_value) ? true : false;
 
 	}
 
@@ -284,37 +284,37 @@ 
 	 *
 	 * @return bool
 	 */
-	public function update_customer_email_on_user_update( $user_id = 0, $old_user_data ) {
+	public function update_customer_email_on_user_update($user_id = 0, $old_user_data) {
 
-		$customer = new Give_Customer( $user_id, true );
+		$customer = new Give_Customer($user_id, true);
 
-		if( ! $customer ) {
+		if ( ! $customer) {
 			return false;
 		}
 
-		$user = get_userdata( $user_id );
+		$user = get_userdata($user_id);
 
-		if( ! empty( $user ) && $user->user_email !== $customer->email ) {
+		if ( ! empty($user) && $user->user_email !== $customer->email) {
 
-			if( ! $this->get_customer_by( 'email', $user->user_email ) ) {
+			if ( ! $this->get_customer_by('email', $user->user_email)) {
 
-				$success = $this->update( $customer->id, array( 'email' => $user->user_email ) );
+				$success = $this->update($customer->id, array('email' => $user->user_email));
 
-				if( $success ) {
+				if ($success) {
 					// Update some payment meta if we need to
-					$payments_array = explode( ',', $customer->payment_ids );
+					$payments_array = explode(',', $customer->payment_ids);
 
-					if( ! empty( $payments_array ) ) {
+					if ( ! empty($payments_array)) {
 
-						foreach ( $payments_array as $payment_id ) {
+						foreach ($payments_array as $payment_id) {
 
-							give_update_payment_meta( $payment_id, 'email', $user->user_email );
+							give_update_payment_meta($payment_id, 'email', $user->user_email);
 
 						}
 
 					}
 
-					do_action( 'give_update_customer_email_on_user_update', $user, $customer );
+					do_action('give_update_customer_email_on_user_update', $user, $customer);
 
 				}
 
@@ -335,45 +335,45 @@ 
 	 *
 	 * @return mixed          Upon success, an object of the customer. Upon failure, NULL
 	 */
-	public function get_customer_by( $field = 'id', $value = 0 ) {
+	public function get_customer_by($field = 'id', $value = 0) {
 		global $wpdb;
 
-		if ( empty( $field ) || empty( $value ) ) {
+		if (empty($field) || empty($value)) {
 			return null;
 		}
 
-		if ( 'id' == $field || 'user_id' == $field ) {
+		if ('id' == $field || 'user_id' == $field) {
 			// Make sure the value is numeric to avoid casting objects, for example,
 			// to int 1.
-			if ( ! is_numeric( $value ) ) {
+			if ( ! is_numeric($value)) {
 				return false;
 			}
 
-			$value = intval( $value );
+			$value = intval($value);
 
-			if ( $value < 1 ) {
+			if ($value < 1) {
 				return false;
 			}
 
-		} elseif ( 'email' === $field ) {
+		} elseif ('email' === $field) {
 
-			if ( ! is_email( $value ) ) {
+			if ( ! is_email($value)) {
 				return false;
 			}
 
-			$value = trim( $value );
+			$value = trim($value);
 		}
 
-		if ( ! $value ) {
+		if ( ! $value) {
 			return false;
 		}
 
-		switch ( $field ) {
+		switch ($field) {
 			case 'id':
 				$db_field = 'id';
 				break;
 			case 'email':
-				$value    = sanitize_text_field( $value );
+				$value    = sanitize_text_field($value);
 				$db_field = 'email';
 				break;
 			case 'user_id':
@@ -383,7 +383,7 @@ 
 				return false;
 		}
 
-		if ( ! $customer = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $this->table_name WHERE $db_field = %s LIMIT 1", $value ) ) ) {
+		if ( ! $customer = $wpdb->get_row($wpdb->prepare("SELECT * FROM $this->table_name WHERE $db_field = %s LIMIT 1", $value))) {
 			return false;
 		}
 
@@ -396,7 +396,7 @@ 
 	 * @access  public
 	 * @since   1.0
 	 */
-	public function get_customers( $args = array() ) {
+	public function get_customers($args = array()) {
 
 		global $wpdb;
 
@@ -408,21 +408,21 @@ 
 			'order'   => 'DESC'
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		if ( $args['number'] < 1 ) {
+		if ($args['number'] < 1) {
 			$args['number'] = 999999999999;
 		}
 
 		$where = ' WHERE 1=1 ';
 
 		// specific customers
-		if ( ! empty( $args['id'] ) ) {
+		if ( ! empty($args['id'])) {
 
-			if ( is_array( $args['id'] ) ) {
-				$ids = implode( ',', array_map( 'intval', $args['id'] ) );
+			if (is_array($args['id'])) {
+				$ids = implode(',', array_map('intval', $args['id']));
 			} else {
-				$ids = intval( $args['id'] );
+				$ids = intval($args['id']);
 			}
 
 			$where .= " AND `id` IN( {$ids} ) ";
@@ -430,12 +430,12 @@ 
 		}
 
 		// customers for specific user accounts
-		if ( ! empty( $args['user_id'] ) ) {
+		if ( ! empty($args['user_id'])) {
 
-			if ( is_array( $args['user_id'] ) ) {
-				$user_ids = implode( ',', array_map( 'intval', $args['user_id'] ) );
+			if (is_array($args['user_id'])) {
+				$user_ids = implode(',', array_map('intval', $args['user_id']));
 			} else {
-				$user_ids = intval( $args['user_id'] );
+				$user_ids = intval($args['user_id']);
 			}
 
 			$where .= " AND `user_id` IN( {$user_ids} ) ";
@@ -443,41 +443,41 @@ 
 		}
 
 		//specific customers by email
-		if( ! empty( $args['email'] ) ) {
+		if ( ! empty($args['email'])) {
 
-			if( is_array( $args['email'] ) ) {
+			if (is_array($args['email'])) {
 
-				$emails_count       = count( $args['email'] );
-				$emails_placeholder = array_fill( 0, $emails_count, '%s' );
-				$emails             = implode( ', ', $emails_placeholder );
+				$emails_count       = count($args['email']);
+				$emails_placeholder = array_fill(0, $emails_count, '%s');
+				$emails             = implode(', ', $emails_placeholder);
 
-				$where .= $wpdb->prepare( " AND `email` IN( $emails ) ", $args['email'] );
+				$where .= $wpdb->prepare(" AND `email` IN( $emails ) ", $args['email']);
 			} else {
-				$where .= $wpdb->prepare( " AND `email` = %s ", $args['email'] );
+				$where .= $wpdb->prepare(" AND `email` = %s ", $args['email']);
 			}
 		}
 
 		// specific customers by name
-		if( ! empty( $args['name'] ) ) {
-			$where .= $wpdb->prepare( " AND `name` LIKE '%%%%" . '%s' . "%%%%' ", $args['name'] );
+		if ( ! empty($args['name'])) {
+			$where .= $wpdb->prepare(" AND `name` LIKE '%%%%".'%s'."%%%%' ", $args['name']);
 		}
 
 		// Customers created for a specific date or in a date range
-		if ( ! empty( $args['date'] ) ) {
+		if ( ! empty($args['date'])) {
 
-			if ( is_array( $args['date'] ) ) {
+			if (is_array($args['date'])) {
 
-				if ( ! empty( $args['date']['start'] ) ) {
+				if ( ! empty($args['date']['start'])) {
 
-					$start = date( 'Y-m-d H:i:s', strtotime( $args['date']['start'] ) );
+					$start = date('Y-m-d H:i:s', strtotime($args['date']['start']));
 
 					$where .= " AND `date_created` >= '{$start}'";
 
 				}
 
-				if ( ! empty( $args['date']['end'] ) ) {
+				if ( ! empty($args['date']['end'])) {
 
-					$end = date( 'Y-m-d H:i:s', strtotime( $args['date']['end'] ) );
+					$end = date('Y-m-d H:i:s', strtotime($args['date']['end']));
 
 					$where .= " AND `date_created` <= '{$end}'";
 
@@ -485,31 +485,31 @@ 
 
 			} else {
 
-				$year  = date( 'Y', strtotime( $args['date'] ) );
-				$month = date( 'm', strtotime( $args['date'] ) );
-				$day   = date( 'd', strtotime( $args['date'] ) );
+				$year  = date('Y', strtotime($args['date']));
+				$month = date('m', strtotime($args['date']));
+				$day   = date('d', strtotime($args['date']));
 
 				$where .= " AND $year = YEAR ( date_created ) AND $month = MONTH ( date_created ) AND $day = DAY ( date_created )";
 			}
 
 		}
 
-		$args['orderby'] = ! array_key_exists( $args['orderby'], $this->get_columns() ) ? 'id' : $args['orderby'];
+		$args['orderby'] = ! array_key_exists($args['orderby'], $this->get_columns()) ? 'id' : $args['orderby'];
 
-		if ( 'purchase_value' == $args['orderby'] ) {
+		if ('purchase_value' == $args['orderby']) {
 			$args['orderby'] = 'purchase_value+0';
 		}
 
-		$cache_key = md5( 'give_customers_' . serialize( $args ) );
+		$cache_key = md5('give_customers_'.serialize($args));
 
-		$customers = wp_cache_get( $cache_key, 'customers' );
+		$customers = wp_cache_get($cache_key, 'customers');
 
-		$args['orderby'] = esc_sql( $args['orderby'] );
-		$args['order']   = esc_sql( $args['order'] );
+		$args['orderby'] = esc_sql($args['orderby']);
+		$args['order']   = esc_sql($args['order']);
 
-		if ( $customers === false ) {
-			$customers = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM  $this->table_name $where ORDER BY {$args['orderby']} {$args['order']} LIMIT %d,%d;", absint( $args['offset'] ), absint( $args['number'] ) ) );
-			wp_cache_set( $cache_key, $customers, 'customers', 3600 );
+		if ($customers === false) {
+			$customers = $wpdb->get_results($wpdb->prepare("SELECT * FROM  $this->table_name $where ORDER BY {$args['orderby']} {$args['order']} LIMIT %d,%d;", absint($args['offset']), absint($args['number'])));
+			wp_cache_set($cache_key, $customers, 'customers', 3600);
 		}
 
 		return $customers;
@@ -523,26 +523,26 @@ 
 	 * @access  public
 	 * @since   1.0
 	 */
-	public function count( $args = array() ) {
+	public function count($args = array()) {
 
 		global $wpdb;
 
 		$where = ' WHERE 1=1 ';
 
-		if ( ! empty( $args['date'] ) ) {
+		if ( ! empty($args['date'])) {
 
-			if ( is_array( $args['date'] ) ) {
+			if (is_array($args['date'])) {
 
-				$start = date( 'Y-m-d H:i:s', strtotime( $args['date']['start'] ) );
-				$end   = date( 'Y-m-d H:i:s', strtotime( $args['date']['end'] ) );
+				$start = date('Y-m-d H:i:s', strtotime($args['date']['start']));
+				$end   = date('Y-m-d H:i:s', strtotime($args['date']['end']));
 
 				$where .= " AND `date_created` >= '{$start}' AND `date_created` <= '{$end}'";
 
 			} else {
 
-				$year  = date( 'Y', strtotime( $args['date'] ) );
-				$month = date( 'm', strtotime( $args['date'] ) );
-				$day   = date( 'd', strtotime( $args['date'] ) );
+				$year  = date('Y', strtotime($args['date']));
+				$month = date('m', strtotime($args['date']));
+				$day   = date('d', strtotime($args['date']));
 
 				$where .= " AND $year = YEAR ( date_created ) AND $month = MONTH ( date_created ) AND $day = DAY ( date_created )";
 			}
@@ -550,16 +550,16 @@ 
 		}
 
 
-		$cache_key = md5( 'give_customers_count' . serialize( $args ) );
+		$cache_key = md5('give_customers_count'.serialize($args));
 
-		$count = wp_cache_get( $cache_key, 'customers' );
+		$count = wp_cache_get($cache_key, 'customers');
 
-		if ( $count === false ) {
-			$count = $wpdb->get_var( "SELECT COUNT($this->primary_key) FROM " . $this->table_name . "{$where};" );
-			wp_cache_set( $cache_key, $count, 'customers', 3600 );
+		if ($count === false) {
+			$count = $wpdb->get_var("SELECT COUNT($this->primary_key) FROM ".$this->table_name."{$where};");
+			wp_cache_set($cache_key, $count, 'customers', 3600);
 		}
 
-		return absint( $count );
+		return absint($count);
 
 	}
 
@@ -571,9 +571,9 @@ 
 	 */
 	public function create_table() {
 		
-		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
+		require_once(ABSPATH.'wp-admin/includes/upgrade.php');
 
-		$sql = "CREATE TABLE " . $this->table_name . " (
+		$sql = "CREATE TABLE ".$this->table_name." (
 		id bigint(20) NOT NULL AUTO_INCREMENT,
 		user_id bigint(20) NOT NULL,
 		email varchar(50) NOT NULL,
@@ -588,9 +588,9 @@ 
 		KEY user (user_id)
 		) CHARACTER SET utf8 COLLATE utf8_general_ci;";
 
-		dbDelta( $sql );
+		dbDelta($sql);
 
-		update_option( $this->table_name . '_db_version', $this->version );
+		update_option($this->table_name.'_db_version', $this->version);
 	}
 	
 	/**
@@ -600,6 +600,6 @@ 
 	 * @return bool Returns if the customers table was installed and upgrade routine run
 	 */
 	public function installed() {
-		return $this->table_exists( $this->table_name );
+		return $this->table_exists($this->table_name);
 	}
 }

includes/install.php

Spacing +53 added lines, -53 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,15 +24,15 @@ 
  * @global $wp_version
  * @return void
  */
-function give_install( $network_wide = false ) {
+function give_install($network_wide = false) {
 
 	global $wpdb;
 
-	if ( is_multisite() && $network_wide ) {
+	if (is_multisite() && $network_wide) {
 
-		foreach ( $wpdb->get_col( "SELECT blog_id FROM $wpdb->blogs LIMIT 100" ) as $blog_id ) {
+		foreach ($wpdb->get_col("SELECT blog_id FROM $wpdb->blogs LIMIT 100") as $blog_id) {
 
-			switch_to_blog( $blog_id );
+			switch_to_blog($blog_id);
 			give_run_install();
 			restore_current_blog();
 
@@ -46,7 +46,7 @@ 
 
 }
 
-register_activation_hook( GIVE_PLUGIN_FILE, 'give_install' );
+register_activation_hook(GIVE_PLUGIN_FILE, 'give_install');
 
 /**
  * Run the Give Install process
@@ -62,25 +62,25 @@ 
 	give_setup_post_types();
 
 	// Clear the permalinks
-	flush_rewrite_rules( false );
+	flush_rewrite_rules(false);
 
 	// Add Upgraded From Option
-	$current_version = get_option( 'give_version' );
-	if ( $current_version ) {
-		update_option( 'give_version_upgraded_from', $current_version );
+	$current_version = get_option('give_version');
+	if ($current_version) {
+		update_option('give_version_upgraded_from', $current_version);
 	}
 
 	// Setup some default options
 	$options = array();
 
 	// Checks if the Success Page option exists AND that the page exists
-	if ( ! get_post( give_get_option( 'success_page' ) ) ) {
+	if ( ! get_post(give_get_option('success_page'))) {
 
 		// Purchase Confirmation (Success) Page
 		$success = wp_insert_post(
 			array(
-				'post_title'     => __( 'Donation Confirmation', 'give' ),
-				'post_content'   => __( '[give_receipt]', 'give' ),
+				'post_title'     => __('Donation Confirmation', 'give'),
+				'post_content'   => __('[give_receipt]', 'give'),
 				'post_status'    => 'publish',
 				'post_author'    => 1,
 				'post_type'      => 'page',
@@ -93,13 +93,13 @@ 
 	}
 
 	// Checks if the Failure Page option exists AND that the page exists
-	if ( ! get_post( give_get_option( 'failure_page' ) ) ) {
+	if ( ! get_post(give_get_option('failure_page'))) {
 
 		// Failed Purchase Page
 		$failed = wp_insert_post(
 			array(
-				'post_title'     => __( 'Transaction Failed', 'give' ),
-				'post_content'   => __( 'We\'re sorry, your transaction failed to process. Please try again or contact site support.', 'give' ),
+				'post_title'     => __('Transaction Failed', 'give'),
+				'post_content'   => __('We\'re sorry, your transaction failed to process. Please try again or contact site support.', 'give'),
 				'post_status'    => 'publish',
 				'post_author'    => 1,
 				'post_type'      => 'page',
@@ -111,11 +111,11 @@ 
 	}
 
 	// Checks if the History Page option exists AND that the page exists
-	if ( ! get_post( give_get_option( 'history_page' ) ) ) {
+	if ( ! get_post(give_get_option('history_page'))) {
 		// Purchase History (History) Page
 		$history = wp_insert_post(
 			array(
-				'post_title'     => __( 'Donation History', 'give' ),
+				'post_title'     => __('Donation History', 'give'),
 				'post_content'   => '[donation_history]',
 				'post_status'    => 'publish',
 				'post_author'    => 1,
@@ -128,7 +128,7 @@ 
 	}
 
 	//Fresh Install? Setup Test Mode, Base Country (US), Test Gateway, Currency
-	if ( empty( $current_version ) ) {
+	if (empty($current_version)) {
 		$options['base_country']       = 'US';
 		$options['test_mode']          = 1;
 		$options['currency']           = 'USD';
@@ -145,12 +145,12 @@ 
 	}
 
 	// Populate some default values
-	update_option( 'give_settings', array_merge( $give_options, $options ) );
-	update_option( 'give_version', GIVE_VERSION );
+	update_option('give_settings', array_merge($give_options, $options));
+	update_option('give_version', GIVE_VERSION);
 
 	//Update Version Number
-	if ( $current_version ) {
-		update_option( 'give_version_upgraded_from', $current_version );
+	if ($current_version) {
+		update_option('give_version_upgraded_from', $current_version);
 	}
 
 	// Create Give roles
@@ -159,7 +159,7 @@ 
 	$roles->add_caps();
 
 	$api = new Give_API();
-	update_option( 'give_default_api_version', 'v' . $api->get_version() );
+	update_option('give_default_api_version', 'v'.$api->get_version());
 
 	// Create the customers database
 	@Give()->customers->create_table();
@@ -168,11 +168,11 @@ 
 	Give()->session->use_php_sessions();
 
 	// Add a temporary option to note that Give pages have been created
-	set_transient( '_give_installed', $options, 30 );
+	set_transient('_give_installed', $options, 30);
 	
-	if ( ! $current_version ) {
+	if ( ! $current_version) {
 
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/upgrade-functions.php';
+		require_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/upgrade-functions.php';
 
 		// When new upgrade routines are added, mark them as complete on fresh install
 		$upgrade_routines = array(
@@ -180,22 +180,22 @@ 
 			'upgrade_give_offline_status'
 		);
 
-		foreach ( $upgrade_routines as $upgrade ) {
-			give_set_upgrade_complete( $upgrade );
+		foreach ($upgrade_routines as $upgrade) {
+			give_set_upgrade_complete($upgrade);
 		}
 	}
 
 	// Bail if activating from network, or bulk
-	if ( is_network_admin() || isset( $_GET['activate-multi'] ) ) {
+	if (is_network_admin() || isset($_GET['activate-multi'])) {
 		return;
 	}
 
 	// Add the transient to redirect
-	set_transient( '_give_activation_redirect', true, 30 );
+	set_transient('_give_activation_redirect', true, 30);
 
 }
 
-register_activation_hook( GIVE_PLUGIN_FILE, 'give_install' );
+register_activation_hook(GIVE_PLUGIN_FILE, 'give_install');
 
 /**
  * Network Activated New Site Setup
@@ -211,11 +211,11 @@ 
  * @param  int    $site_id The Site ID
  * @param  array  $meta    Blog Meta
  */
-function on_create_blog( $blog_id, $user_id, $domain, $path, $site_id, $meta ) {
+function on_create_blog($blog_id, $user_id, $domain, $path, $site_id, $meta) {
 
-	if ( is_plugin_active_for_network( GIVE_PLUGIN_BASENAME ) ) {
+	if (is_plugin_active_for_network(GIVE_PLUGIN_BASENAME)) {
 
-		switch_to_blog( $blog_id );
+		switch_to_blog($blog_id);
 		give_install();
 		restore_current_blog();
 
@@ -223,7 +223,7 @@ 
 
 }
 
-add_action( 'wpmu_new_blog', 'on_create_blog', 10, 6 );
+add_action('wpmu_new_blog', 'on_create_blog', 10, 6);
 
 
 /**
@@ -236,11 +236,11 @@ 
  *
  * @return array          The tables to drop
  */
-function give_wpmu_drop_tables( $tables, $blog_id ) {
+function give_wpmu_drop_tables($tables, $blog_id) {
 
-	switch_to_blog( $blog_id );
+	switch_to_blog($blog_id);
 	$customers_db = new Give_DB_Customers();
-	if ( $customers_db->installed() ) {
+	if ($customers_db->installed()) {
 		$tables[] = $customers_db->table_name;
 	}
 	restore_current_blog();
@@ -249,7 +249,7 @@ 
 
 }
 
-add_filter( 'wpmu_drop_tables', 'give_wpmu_drop_tables', 10, 2 );
+add_filter('wpmu_drop_tables', 'give_wpmu_drop_tables', 10, 2);
 
 /**
  * Post-installation
@@ -261,35 +261,35 @@ 
  */
 function give_after_install() {
 
-	if ( ! is_admin() ) {
+	if ( ! is_admin()) {
 		return;
 	}
 
-	$give_options     = get_transient( '_give_installed' );
-	$give_table_check = get_option( '_give_table_check', false );
+	$give_options     = get_transient('_give_installed');
+	$give_table_check = get_option('_give_table_check', false);
 
-	if ( false === $give_table_check || current_time( 'timestamp' ) > $give_table_check ) {
+	if (false === $give_table_check || current_time('timestamp') > $give_table_check) {
 
-		if ( ! @Give()->customers->installed() ) {
+		if ( ! @Give()->customers->installed()) {
 			// Create the customers database (this ensures it creates it on multisite instances where it is network activated)
 			@Give()->customers->create_table();
 
-			do_action( 'give_after_install', $give_options );
+			do_action('give_after_install', $give_options);
 		}
 
-		update_option( '_give_table_check', ( current_time( 'timestamp' ) + WEEK_IN_SECONDS ) );
+		update_option('_give_table_check', (current_time('timestamp') + WEEK_IN_SECONDS));
 
 	}
 
 	// Delete the transient
-	if ( false !== $give_options ) {
-		delete_transient( '_give_installed' );
+	if (false !== $give_options) {
+		delete_transient('_give_installed');
 	}
 
 
 }
 
-add_action( 'admin_init', 'give_after_install' );
+add_action('admin_init', 'give_after_install');
 
 
 /**
@@ -304,11 +304,11 @@ 
 
 	global $wp_roles;
 
-	if ( ! is_object( $wp_roles ) ) {
+	if ( ! is_object($wp_roles)) {
 		return;
 	}
 
-	if ( ! array_key_exists( 'give_manager', $wp_roles->roles ) ) {
+	if ( ! array_key_exists('give_manager', $wp_roles->roles)) {
 
 		// Create Give plugin roles
 		$roles = new Give_Roles;
@@ -319,4 +319,4 @@ 
 
 }
 
-add_action( 'admin_init', 'give_install_roles_on_network' );
\ No newline at end of file
+add_action('admin_init', 'give_install_roles_on_network');
\ No newline at end of file

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 +182 added lines, -182 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,17 +257,17 @@ 
 	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' ),
-			'donation' => __( 'Donation', 'give' ),
-			'status'   => __( 'Status', 'give' ),
-			'date'     => __( 'Date', 'give' ),
-			'donor'    => __( 'Donor', 'give' ),
-			'ID'       => __( 'ID', 'give' ),
+			'email'    => __('Email', 'give'),
+			'details'  => __('Details', 'give'),
+			'amount'   => __('Amount', 'give'),
+			'donation' => __('Donation', '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);
 	}
 
 	/**
@@ -279,12 +279,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);
 	}
 
 	/**
@@ -310,41 +310,41 @@ 
 	 *
 	 * @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 'donation' :
-				$value = '<a href="' . get_permalink( $payment->form_id ) . '">' . $payment->form_title . '</a>';
-				$level = give_get_payment_form_title( $payment->meta, true );
+				$value = '<a href="'.get_permalink($payment->form_id).'">'.$payment->form_title.'</a>';
+				$level = give_get_payment_form_title($payment->meta, true);
 
-				if ( ! empty( $level ) ) {
+				if ( ! empty($level)) {
 					$value .= $level;
 				}
 
 				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);
 	}
 
 	/**
@@ -357,40 +357,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');
 	}
 
 	/**
@@ -403,7 +403,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',
@@ -421,8 +421,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>';
 	}
 
 	/**
@@ -435,19 +435,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-donors&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-donors&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');
 	}
 
 	/**
@@ -459,18 +459,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);
 	}
 
 	/**
@@ -481,61 +481,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());
 		}
 
 	}
@@ -553,27 +553,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;
@@ -582,7 +582,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;
 		}
 	}
@@ -597,26 +597,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,
@@ -630,14 +630,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();
 
@@ -657,17 +657,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;
@@ -694,18 +694,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-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 +97 added lines, -97 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);
 
 	}
 
@@ -636,9 +636,9 @@ 
 	 */
 	public function is_close_donation_form() {
 		return (
-				'yes' === get_post_meta( $this->ID, '_give_goal_option', true ) )
-				&& ( 'yes' === get_post_meta( $this->ID, '_give_close_form_when_goal_achieved', true ) )
-				&& ( $this->get_goal() <= $this->get_earnings()
+				'yes' === get_post_meta($this->ID, '_give_goal_option', true) )
+				&& ('yes' === get_post_meta($this->ID, '_give_close_form_when_goal_achieved', true))
+				&& ($this->get_goal() <= $this->get_earnings()
 		);
 	}
 
@@ -654,28 +654,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;
 

includes/class-give-logging.php

Doc Comments +2 added lines, -2 removed lines

@@ -214,7 +214,7 @@ 
 	 * @param array $log_data Log entry data
 	 * @param array $log_meta Log entry meta
 	 *
-	 * @return bool True if successful, false otherwise
+	 * @return boolean|null True if successful, false otherwise
 	 */
 	public function update_log( $log_data = array(), $log_meta = array() ) {
 
@@ -392,7 +392,7 @@ 
  * @param string $title
  * @param string $message
  * @param int $parent
- * @param null $type
+ * @param string $type
  *
  * @global $give_logs GIVE Logs Object
  *

Spacing +56 added lines, -56 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -30,10 +30,10 @@ 
 	 */
 	public function __construct() {
 		// Create the log post type
-		add_action( 'init', array( $this, 'register_post_type' ), 1 );
+		add_action('init', array($this, 'register_post_type'), 1);
 
 		// Create types taxonomy and default types
-		add_action( 'init', array( $this, 'register_taxonomy' ), 1 );
+		add_action('init', array($this, 'register_taxonomy'), 1);
 
 	}
 
@@ -47,7 +47,7 @@ 
 	public function register_post_type() {
 		/* Logs post type */
 		$log_args = array(
-			'labels'              => array( 'name' => __( 'Logs', 'give' ) ),
+			'labels'              => array('name' => __('Logs', 'give')),
 			'public'              => false,
 			'exclude_from_search' => true,
 			'publicly_queryable'  => false,
@@ -55,11 +55,11 @@ 
 			'query_var'           => false,
 			'rewrite'             => false,
 			'capability_type'     => 'post',
-			'supports'            => array( 'title', 'editor' ),
+			'supports'            => array('title', 'editor'),
 			'can_export'          => true
 		);
 
-		register_post_type( 'give_log', $log_args );
+		register_post_type('give_log', $log_args);
 	}
 
 	/**
@@ -72,7 +72,7 @@ 
 	 * @return void
 	 */
 	public function register_taxonomy() {
-		register_taxonomy( 'give_log_type', 'give_log', array( 'public' => false ) );
+		register_taxonomy('give_log_type', 'give_log', array('public' => false));
 	}
 
 	/**
@@ -91,7 +91,7 @@ 
 			'api_request'
 		);
 
-		return apply_filters( 'give_log_types', $terms );
+		return apply_filters('give_log_types', $terms);
 	}
 
 	/**
@@ -107,8 +107,8 @@ 
 	 *
 	 * @return bool Whether log type is valid
 	 */
-	function valid_type( $type ) {
-		return in_array( $type, $this->log_types() );
+	function valid_type($type) {
+		return in_array($type, $this->log_types());
 	}
 
 	/**
@@ -128,7 +128,7 @@ 
 	 *
 	 * @return int Log ID
 	 */
-	public function add( $title = '', $message = '', $parent = 0, $type = null ) {
+	public function add($title = '', $message = '', $parent = 0, $type = null) {
 		$log_data = array(
 			'post_title'   => $title,
 			'post_content' => $message,
@@ -136,7 +136,7 @@ 
 			'log_type'     => $type
 		);
 
-		return $this->insert_log( $log_data );
+		return $this->insert_log($log_data);
 	}
 
 	/**
@@ -152,12 +152,12 @@ 
 	 *
 	 * @return array Array of the connected logs
 	 */
-	public function get_logs( $object_id = 0, $type = null, $paged = null ) {
-		return $this->get_connected_logs( array(
+	public function get_logs($object_id = 0, $type = null, $paged = null) {
+		return $this->get_connected_logs(array(
 			'post_parent' => $object_id,
 			'paged'       => $paged,
 			'log_type'    => $type
-		) );
+		));
 	}
 
 	/**
@@ -172,7 +172,7 @@ 
 	 *
 	 * @return int The ID of the newly created log item
 	 */
-	function insert_log( $log_data = array(), $log_meta = array() ) {
+	function insert_log($log_data = array(), $log_meta = array()) {
 		$defaults = array(
 			'post_type'    => 'give_log',
 			'post_status'  => 'publish',
@@ -181,26 +181,26 @@ 
 			'log_type'     => false
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
+		$args = wp_parse_args($log_data, $defaults);
 
-		do_action( 'give_pre_insert_log', $log_data, $log_meta );
+		do_action('give_pre_insert_log', $log_data, $log_meta);
 
 		// Store the log entry
-		$log_id = wp_insert_post( $args );
+		$log_id = wp_insert_post($args);
 
 		// Set the log type, if any
-		if ( $log_data['log_type'] && $this->valid_type( $log_data['log_type'] ) ) {
-			wp_set_object_terms( $log_id, $log_data['log_type'], 'give_log_type', false );
+		if ($log_data['log_type'] && $this->valid_type($log_data['log_type'])) {
+			wp_set_object_terms($log_id, $log_data['log_type'], 'give_log_type', false);
 		}
 
 		// Set log meta, if any
-		if ( $log_id && ! empty( $log_meta ) ) {
-			foreach ( (array) $log_meta as $key => $meta ) {
-				update_post_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				update_post_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 			}
 		}
 
-		do_action( 'give_post_insert_log', $log_id, $log_data, $log_meta );
+		do_action('give_post_insert_log', $log_id, $log_data, $log_meta);
 
 		return $log_id;
 	}
@@ -216,9 +216,9 @@ 
 	 *
 	 * @return bool True if successful, false otherwise
 	 */
-	public function update_log( $log_data = array(), $log_meta = array() ) {
+	public function update_log($log_data = array(), $log_meta = array()) {
 
-		do_action( 'give_pre_update_log', $log_data, $log_meta );
+		do_action('give_pre_update_log', $log_data, $log_meta);
 
 		$defaults = array(
 			'post_type'   => 'give_log',
@@ -226,20 +226,20 @@ 
 			'post_parent' => 0
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
+		$args = wp_parse_args($log_data, $defaults);
 
 		// Store the log entry
-		$log_id = wp_update_post( $args );
+		$log_id = wp_update_post($args);
 
-		if ( $log_id && ! empty( $log_meta ) ) {
-			foreach ( (array) $log_meta as $key => $meta ) {
-				if ( ! empty( $meta ) ) {
-					update_post_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				if ( ! empty($meta)) {
+					update_post_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 				}
 			}
 		}
 
-		do_action( 'give_post_update_log', $log_id, $log_data, $log_meta );
+		do_action('give_post_update_log', $log_id, $log_data, $log_meta);
 	}
 
 	/**
@@ -254,19 +254,19 @@ 
 	 *
 	 * @return mixed array if logs were found, false otherwise
 	 */
-	public function get_connected_logs( $args = array() ) {
+	public function get_connected_logs($args = array()) {
 
 		$defaults = array(
 			'post_type'      => 'give_log',
 			'posts_per_page' => 20,
 			'post_status'    => 'publish',
-			'paged'          => get_query_var( 'paged' ),
+			'paged'          => get_query_var('paged'),
 			'log_type'       => false
 		);
 
-		$query_args = wp_parse_args( $args, $defaults );
+		$query_args = wp_parse_args($args, $defaults);
 
-		if ( $query_args['log_type'] && $this->valid_type( $query_args['log_type'] ) ) {
+		if ($query_args['log_type'] && $this->valid_type($query_args['log_type'])) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -276,9 +276,9 @@ 
 			);
 		}
 		
-		$logs = get_posts( $query_args );
+		$logs = get_posts($query_args);
 	
-		if ( $logs ) {
+		if ($logs) {
 			return $logs;
 		}
 
@@ -299,17 +299,17 @@ 
 	 *
 	 * @return int Log count
 	 */
-	public function get_log_count( $object_id = 0, $type = null, $meta_query = null, $date_query = null ) {
+	public function get_log_count($object_id = 0, $type = null, $meta_query = null, $date_query = null) {
 
 		$query_args = array(
 			'post_parent'    => $object_id,
 			'post_type'      => 'give_log',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'publish',
 			'fields'         => 'ids',
 		);
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -319,15 +319,15 @@ 
 			);
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		if ( ! empty( $date_query ) ) {
+		if ( ! empty($date_query)) {
 			$query_args['date_query'] = $date_query;
 		}
 
-		$logs = new WP_Query( $query_args );
+		$logs = new WP_Query($query_args);
 
 		return (int) $logs->post_count;
 	}
@@ -345,16 +345,16 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_logs( $object_id = 0, $type = null, $meta_query = null ) {
+	public function delete_logs($object_id = 0, $type = null, $meta_query = null) {
 		$query_args = array(
 			'post_parent'    => $object_id,
 			'post_type'      => 'give_log',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'publish',
 			'fields'         => 'ids'
 		);
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -364,15 +364,15 @@ 
 			);
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		$logs = get_posts( $query_args );
+		$logs = get_posts($query_args);
 
-		if ( $logs ) {
-			foreach ( $logs as $log ) {
-				wp_delete_post( $log, true );
+		if ($logs) {
+			foreach ($logs as $log) {
+				wp_delete_post($log, true);
 			}
 		}
 	}
@@ -400,9 +400,9 @@ 
  *
  * @return mixed ID of the new log entry
  */
-function give_record_log( $title = '', $message = '', $parent = 0, $type = null ) {
+function give_record_log($title = '', $message = '', $parent = 0, $type = null) {
 	global $give_logs;
-	$log = $give_logs->add( $title, $message, $parent, $type );
+	$log = $give_logs->add($title, $message, $parent, $type);
 
 	return $log;
 }
\ No newline at end of file

includes/payments/class-give-payment.php

Doc Comments +3 added lines, -3 removed lines

@@ -293,7 +293,7 @@ 
 	 *
 	 * @param int $payment_id A given payment
 	 *
-	 * @return mixed void|false
+	 * @return false|null void|false
 	 */
 	public function __construct( $payment_id = false ) {
 
@@ -361,7 +361,7 @@ 
 	 *
 	 * @param  string $name The attribute to get
 	 *
-	 * @return boolean       If the item is set or not
+	 * @return boolean|null       If the item is set or not
 	 */
 	public function __isset( $name ) {
 		if ( property_exists( $this, $name ) ) {
@@ -1149,7 +1149,7 @@ 
 	 *
 	 * @param string $note The note to add
 	 *
-	 * @return void
+	 * @return false|null
 	 */
 	public function add_note( $note = false ) {
 		// Bail if no note specified

Indentation +1 added lines, -1 removed lines

@@ -899,7 +899,7 @@
 					//Find a match between price_id and level_id
 					//First verify array keys exists THEN make the match
 					if ( ( isset( $args['price_id'] ) && isset( $price['_give_id']['level_id'] ) )
-					     && $args['price_id'] == $price['_give_id']['level_id']
+						 && $args['price_id'] == $price['_give_id']['level_id']
 					) {
 						$item_price = $price['_give_amount'];
 					}

Spacing +302 added lines, -302 removed lines

@@ -13,7 +13,7 @@ 
 
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -295,13 +295,13 @@ 
 	 *
 	 * @return mixed void|false
 	 */
-	public function __construct( $payment_id = false ) {
+	public function __construct($payment_id = false) {
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		$this->setup_payment( $payment_id );
+		$this->setup_payment($payment_id);
 	}
 
 	/**
@@ -313,11 +313,11 @@ 
 	 *
 	 * @return mixed        The value
 	 */
-	public function __get( $key ) {
+	public function __get($key) {
 
-		if ( method_exists( $this, 'get_' . $key ) ) {
+		if (method_exists($this, 'get_'.$key)) {
 
-			$value = call_user_func( array( $this, 'get_' . $key ) );
+			$value = call_user_func(array($this, 'get_'.$key));
 
 		} else {
 
@@ -338,18 +338,18 @@ 
 	 * @param string $key The property name
 	 * @param mixed $value The value of the property
 	 */
-	public function __set( $key, $value ) {
-		$ignore = array( '_ID' );
+	public function __set($key, $value) {
+		$ignore = array('_ID');
 
-		if ( $key === 'status' ) {
+		if ($key === 'status') {
 			$this->old_status = $this->status;
 		}
 
-		if ( ! in_array( $key, $ignore ) ) {
-			$this->pending[ $key ] = $value;
+		if ( ! in_array($key, $ignore)) {
+			$this->pending[$key] = $value;
 		}
 
-		if ( '_ID' !== $key ) {
+		if ('_ID' !== $key) {
 			$this->$key = $value;
 		}
 	}
@@ -363,9 +363,9 @@ 
 	 *
 	 * @return boolean       If the item is set or not
 	 */
-	public function __isset( $name ) {
-		if ( property_exists( $this, $name ) ) {
-			return false === empty( $this->$name );
+	public function __isset($name) {
+		if (property_exists($this, $name)) {
+			return false === empty($this->$name);
 		} else {
 			return null;
 		}
@@ -380,31 +380,31 @@ 
 	 *
 	 * @return bool            If the setup was successful or not
 	 */
-	private function setup_payment( $payment_id ) {
+	private function setup_payment($payment_id) {
 		$this->pending = array();
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		$payment = get_post( $payment_id );
+		$payment = get_post($payment_id);
 
-		if ( ! $payment || is_wp_error( $payment ) ) {
+		if ( ! $payment || is_wp_error($payment)) {
 			return false;
 		}
 
-		if ( 'give_payment' !== $payment->post_type ) {
+		if ('give_payment' !== $payment->post_type) {
 			return false;
 		}
 
 		// Allow extensions to perform actions before the payment is loaded
-		do_action( 'give_pre_setup_payment', $this, $payment_id );
+		do_action('give_pre_setup_payment', $this, $payment_id);
 
 		// Primary Identifier
-		$this->ID = absint( $payment_id );
+		$this->ID = absint($payment_id);
 
 		// Protected ID that can never be changed
-		$this->_ID = absint( $payment_id );
+		$this->_ID = absint($payment_id);
 
 		// We have a payment, get the generic payment_meta item to reduce calls to it
 		$this->payment_meta = $this->get_meta();
@@ -419,7 +419,7 @@ 
 		$this->parent_payment = $payment->post_parent;
 
 		$all_payment_statuses  = give_get_payment_statuses();
-		$this->status_nicename = array_key_exists( $this->status, $all_payment_statuses ) ? $all_payment_statuses[ $this->status ] : ucfirst( $this->status );
+		$this->status_nicename = array_key_exists($this->status, $all_payment_statuses) ? $all_payment_statuses[$this->status] : ucfirst($this->status);
 
 		// Items
 		$this->fees = $this->setup_fees();
@@ -452,7 +452,7 @@ 
 		$this->number     = $this->setup_payment_number();
 
 		// Allow extensions to add items to this object via hook
-		do_action( 'give_setup_payment', $this, $payment_id );
+		do_action('give_setup_payment', $this, $payment_id);
 
 		return true;
 	}
@@ -467,24 +467,24 @@ 
 
 		// Construct the payment title
 		$payment_title = '';
-		if ( ! empty( $this->first_name ) && ! empty( $this->last_name ) ) {
-			$payment_title = $this->first_name . ' ' . $this->last_name;
-		} else if ( ! empty( $this->first_name ) && empty( $this->last_name ) ) {
+		if ( ! empty($this->first_name) && ! empty($this->last_name)) {
+			$payment_title = $this->first_name.' '.$this->last_name;
+		} else if ( ! empty($this->first_name) && empty($this->last_name)) {
 			$payment_title = $this->first_name;
-		} else if ( ! empty( $this->email ) && is_email( $this->email ) ) {
+		} else if ( ! empty($this->email) && is_email($this->email)) {
 			$payment_title = $this->email;
 		}
 
 		//Set Key
-		if ( empty( $this->key ) ) {
+		if (empty($this->key)) {
 
-			$auth_key             = defined( 'AUTH_KEY' ) ? AUTH_KEY : '';
-			$this->key            = strtolower( md5( $this->email . date( 'Y-m-d H:i:s' ) . $auth_key . uniqid( 'give', true ) ) );  // Unique key
+			$auth_key             = defined('AUTH_KEY') ? AUTH_KEY : '';
+			$this->key            = strtolower(md5($this->email.date('Y-m-d H:i:s').$auth_key.uniqid('give', true))); // Unique key
 			$this->pending['key'] = $this->key;
 		}
 
 		//Set IP
-		if ( empty( $this->ip ) ) {
+		if (empty($this->ip)) {
 
 			$this->ip            = give_get_ip();
 			$this->pending['ip'] = $this->ip;
@@ -511,58 +511,58 @@ 
 			'fees'         => $this->fees,
 		);
 
-		$args = apply_filters( 'give_insert_payment_args', array(
+		$args = apply_filters('give_insert_payment_args', array(
 			'post_title'    => $payment_title,
 			'post_status'   => $this->status,
 			'post_type'     => 'give_payment',
-			'post_date'     => ! empty( $this->date ) ? $this->date : null,
-			'post_date_gmt' => ! empty( $this->date ) ? get_gmt_from_date( $this->date ) : null,
+			'post_date'     => ! empty($this->date) ? $this->date : null,
+			'post_date_gmt' => ! empty($this->date) ? get_gmt_from_date($this->date) : null,
 			'post_parent'   => $this->parent_payment,
-		), $payment_data );
+		), $payment_data);
 
 		// Create a blank payment
-		$payment_id = wp_insert_post( $args );
+		$payment_id = wp_insert_post($args);
 
-		if ( ! empty( $payment_id ) ) {
+		if ( ! empty($payment_id)) {
 
 			$this->ID  = $payment_id;
 			$this->_ID = $payment_id;
 
 			$customer = new stdClass;
 
-			if ( did_action( 'give_pre_process_purchase' ) && is_user_logged_in() ) {
-				$customer = new Give_Customer( get_current_user_id(), true );
+			if (did_action('give_pre_process_purchase') && is_user_logged_in()) {
+				$customer = new Give_Customer(get_current_user_id(), true);
 			}
 
-			if ( empty( $customer->id ) ) {
-				$customer = new Give_Customer( $this->email );
+			if (empty($customer->id)) {
+				$customer = new Give_Customer($this->email);
 			}
 
-			if ( empty( $customer->id ) ) {
+			if (empty($customer->id)) {
 
 				$customer_data = array(
-					'name'    => ! is_email( $payment_title ) ? $this->first_name . ' ' . $this->last_name : '',
+					'name'    => ! is_email($payment_title) ? $this->first_name.' '.$this->last_name : '',
 					'email'   => $this->email,
 					'user_id' => $this->user_id,
 				);
 
-				$customer->create( $customer_data );
+				$customer->create($customer_data);
 
 			}
 
 			$this->customer_id            = $customer->id;
 			$this->pending['customer_id'] = $this->customer_id;
-			$customer->attach_payment( $this->ID, false );
+			$customer->attach_payment($this->ID, false);
 
-			$this->payment_meta = apply_filters( 'give_payment_meta', $this->payment_meta, $payment_data );
-			if ( ! empty( $this->payment_meta['fees'] ) ) {
-				$this->fees = array_merge( $this->fees, $this->payment_meta['fees'] );
-				foreach ( $this->fees as $fee ) {
-					$this->increase_fees( $fee['amount'] );
+			$this->payment_meta = apply_filters('give_payment_meta', $this->payment_meta, $payment_data);
+			if ( ! empty($this->payment_meta['fees'])) {
+				$this->fees = array_merge($this->fees, $this->payment_meta['fees']);
+				foreach ($this->fees as $fee) {
+					$this->increase_fees($fee['amount']);
 				}
 			}
 
-			$this->update_meta( '_give_payment_meta', $this->payment_meta );
+			$this->update_meta('_give_payment_meta', $this->payment_meta);
 			$this->new = true;
 		}
 
@@ -582,11 +582,11 @@ 
 		$saved = false;
 
 		//Must have an ID 
-		if ( empty( $this->ID ) ) {
+		if (empty($this->ID)) {
 
 			$payment_id = $this->insert_payment();
 
-			if ( false === $payment_id ) {
+			if (false === $payment_id) {
 				$saved = false;
 			} else {
 				$this->ID = $payment_id;
@@ -595,48 +595,48 @@ 
 		}
 
 		//Set ID if not matching
-		if ( $this->ID !== $this->_ID ) {
+		if ($this->ID !== $this->_ID) {
 			$this->ID = $this->_ID;
 		}
 
 		// If we have something pending, let's save it
-		if ( ! empty( $this->pending ) ) {
+		if ( ! empty($this->pending)) {
 
 			$total_increase = 0;
 			$total_decrease = 0;
 
-			foreach ( $this->pending as $key => $value ) {
+			foreach ($this->pending as $key => $value) {
 
-				switch ( $key ) {
+				switch ($key) {
 
 					case 'donations':
 						// Update totals for pending donations
-						foreach ( $this->pending[ $key ] as $item ) {
+						foreach ($this->pending[$key] as $item) {
 
-							$quantity = isset( $item['quantity'] ) ? $item['quantity'] : 1;
-							$price_id = isset( $item['price_id'] ) ? $item['price_id'] : 0;
+							$quantity = isset($item['quantity']) ? $item['quantity'] : 1;
+							$price_id = isset($item['price_id']) ? $item['price_id'] : 0;
 
-							switch ( $item['action'] ) {
+							switch ($item['action']) {
 
 								case 'add':
 
 									$price = $item['price'];
 
-									if ( 'publish' === $this->status || 'complete' === $this->status || 'revoked' === $this->status ) {
+									if ('publish' === $this->status || 'complete' === $this->status || 'revoked' === $this->status) {
 
 										// Add sales logs
-										$log_date = date_i18n( 'Y-m-d G:i:s', current_time( 'timestamp' ) );
+										$log_date = date_i18n('Y-m-d G:i:s', current_time('timestamp'));
 
 										$y = 0;
-										while ( $y < $quantity ) {
+										while ($y < $quantity) {
 
-											give_record_sale_in_log( $item['id'], $this->ID, $price_id, $log_date );
-											$y ++;
+											give_record_sale_in_log($item['id'], $this->ID, $price_id, $log_date);
+											$y++;
 										}
 
-										$form = new Give_Donate_Form( $item['id'] );
-										$form->increase_sales( $quantity );
-										$form->increase_earnings( $price );
+										$form = new Give_Donate_Form($item['id']);
+										$form->increase_sales($quantity);
+										$form->increase_earnings($price);
 
 										$total_increase += $price;
 									}
@@ -661,15 +661,15 @@ 
 										)
 									);
 
-									$found_logs = get_posts( $log_args );
-									foreach ( $found_logs as $log ) {
-										wp_delete_post( $log->ID, true );
+									$found_logs = get_posts($log_args);
+									foreach ($found_logs as $log) {
+										wp_delete_post($log->ID, true);
 									}
 
-									if ( 'publish' === $this->status || 'complete' === $this->status || 'revoked' === $this->status ) {
-										$form = new Give_Donate_Form( $item['id'] );
-										$form->decrease_sales( $quantity );
-										$form->decrease_earnings( $item['amount'] );
+									if ('publish' === $this->status || 'complete' === $this->status || 'revoked' === $this->status) {
+										$form = new Give_Donate_Form($item['id']);
+										$form->decrease_sales($quantity);
+										$form->decrease_earnings($item['amount']);
 
 										$total_decrease += $item['amount'];
 									}
@@ -682,17 +682,17 @@ 
 
 					case 'fees':
 
-						if ( 'publish' !== $this->status && 'complete' !== $this->status && 'revoked' !== $this->status ) {
+						if ('publish' !== $this->status && 'complete' !== $this->status && 'revoked' !== $this->status) {
 							break;
 						}
 
-						if ( empty( $this->pending[ $key ] ) ) {
+						if (empty($this->pending[$key])) {
 							break;
 						}
 
-						foreach ( $this->pending[ $key ] as $fee ) {
+						foreach ($this->pending[$key] as $fee) {
 
-							switch ( $fee['action'] ) {
+							switch ($fee['action']) {
 
 								case 'add':
 									$total_increase += $fee['amount'];
@@ -709,43 +709,43 @@ 
 						break;
 
 					case 'status':
-						$this->update_status( $this->status );
+						$this->update_status($this->status);
 						break;
 
 					case 'gateway':
-						$this->update_meta( '_give_payment_gateway', $this->gateway );
+						$this->update_meta('_give_payment_gateway', $this->gateway);
 						break;
 
 					case 'mode':
-						$this->update_meta( '_give_payment_mode', $this->mode );
+						$this->update_meta('_give_payment_mode', $this->mode);
 						break;
 
 					case 'transaction_id':
-						$this->update_meta( '_give_payment_transaction_id', $this->transaction_id );
+						$this->update_meta('_give_payment_transaction_id', $this->transaction_id);
 						break;
 
 					case 'ip':
-						$this->update_meta( '_give_payment_user_ip', $this->ip );
+						$this->update_meta('_give_payment_user_ip', $this->ip);
 						break;
 
 					case 'customer_id':
-						$this->update_meta( '_give_payment_customer_id', $this->customer_id );
+						$this->update_meta('_give_payment_customer_id', $this->customer_id);
 						break;
 
 					case 'user_id':
-						$this->update_meta( '_give_payment_user_id', $this->user_id );
+						$this->update_meta('_give_payment_user_id', $this->user_id);
 						break;
 
 					case 'form_title':
-						$this->update_meta( '_give_payment_form_title', $this->form_title );
+						$this->update_meta('_give_payment_form_title', $this->form_title);
 						break;
 
 					case 'form_id':
-						$this->update_meta( '_give_payment_form_id', $this->form_id );
+						$this->update_meta('_give_payment_form_id', $this->form_id);
 						break;
 
 					case 'price_id':
-						$this->update_meta( '_give_payment_price_id', $this->price_id );
+						$this->update_meta('_give_payment_price_id', $this->price_id);
 						break;
 
 					case 'first_name':
@@ -761,15 +761,15 @@ 
 						break;
 
 					case 'email':
-						$this->update_meta( '_give_payment_user_email', $this->email );
+						$this->update_meta('_give_payment_user_email', $this->email);
 						break;
 
 					case 'key':
-						$this->update_meta( '_give_payment_purchase_key', $this->key );
+						$this->update_meta('_give_payment_purchase_key', $this->key);
 						break;
 
 					case 'number':
-						$this->update_meta( '_give_payment_number', $this->number );
+						$this->update_meta('_give_payment_number', $this->number);
 						break;
 
 					case 'date':
@@ -779,11 +779,11 @@ 
 							'edit_date' => true,
 						);
 
-						wp_update_post( $args );
+						wp_update_post($args);
 						break;
 
 					case 'completed_date':
-						$this->update_meta( '_give_completed_date', $this->completed_date );
+						$this->update_meta('_give_completed_date', $this->completed_date);
 						break;
 
 					case 'parent_payment':
@@ -792,38 +792,38 @@ 
 							'post_parent' => $this->parent_payment,
 						);
 
-						wp_update_post( $args );
+						wp_update_post($args);
 						break;
 
 					default:
-						do_action( 'give_payment_save', $this, $key );
+						do_action('give_payment_save', $this, $key);
 						break;
 				}
 			}
 
-			if ( 'pending' !== $this->status ) {
+			if ('pending' !== $this->status) {
 
-				$customer = new Give_Customer( $this->customer_id );
+				$customer = new Give_Customer($this->customer_id);
 
 				$total_change = $total_increase - $total_decrease;
-				if ( $total_change < 0 ) {
+				if ($total_change < 0) {
 
-					$total_change = - ( $total_change );
+					$total_change = - ($total_change);
 					// Decrease the customer's purchase stats
-					$customer->decrease_value( $total_change );
-					give_decrease_total_earnings( $total_change );
+					$customer->decrease_value($total_change);
+					give_decrease_total_earnings($total_change);
 
-				} else if ( $total_change > 0 ) {
+				} else if ($total_change > 0) {
 
 					// Increase the customer's purchase stats
-					$customer->increase_value( $total_change );
-					give_increase_total_earnings( $total_change );
+					$customer->increase_value($total_change);
+					give_increase_total_earnings($total_change);
 
 				}
 
 			}
 
-			$this->update_meta( '_give_payment_total', $this->total );
+			$this->update_meta('_give_payment_total', $this->total);
 
 			$new_meta = array(
 				'form_title' => $this->form_title,
@@ -835,12 +835,12 @@ 
 			);
 
 			$meta        = $this->get_meta();
-			$merged_meta = array_merge( $meta, $new_meta );
+			$merged_meta = array_merge($meta, $new_meta);
 
 			// Only save the payment meta if it's changed
-			if ( md5( serialize( $meta ) ) !== md5( serialize( $merged_meta ) ) ) {
-				$updated = $this->update_meta( '_give_payment_meta', $merged_meta );
-				if ( false !== $updated ) {
+			if (md5(serialize($meta)) !== md5(serialize($merged_meta))) {
+				$updated = $this->update_meta('_give_payment_meta', $merged_meta);
+				if (false !== $updated) {
 					$saved = true;
 				}
 			}
@@ -849,8 +849,8 @@ 
 			$saved         = true;
 		}
 
-		if ( true === $saved ) {
-			$this->setup_payment( $this->ID );
+		if (true === $saved) {
+			$this->setup_payment($this->ID);
 		}
 
 		return $saved;
@@ -867,12 +867,12 @@ 
 	 *
 	 * @return bool True when successful, false otherwise
 	 */
-	public function add_donation( $form_id = 0, $args = array(), $options = array() ) {
+	public function add_donation($form_id = 0, $args = array(), $options = array()) {
 
-		$donation = new Give_Donate_Form( $form_id );
+		$donation = new Give_Donate_Form($form_id);
 
 		// Bail if this post isn't a give donation form
-		if ( ! $donation || $donation->post_type !== 'give_forms' ) {
+		if ( ! $donation || $donation->post_type !== 'give_forms') {
 			return false;
 		}
 
@@ -883,60 +883,60 @@ 
 			'fees'     => array(),
 		);
 
-		$args = wp_parse_args( apply_filters( 'give_payment_add_donation_args', $args, $donation->ID ), $defaults );
+		$args = wp_parse_args(apply_filters('give_payment_add_donation_args', $args, $donation->ID), $defaults);
 
 		// Allow overriding the price
-		if ( false !== $args['price'] ) {
+		if (false !== $args['price']) {
 			$item_price = $args['price'];
 		} else {
 
 			// Deal with variable pricing
-			if ( give_has_variable_prices( $donation->ID ) ) {
-				$prices     = maybe_unserialize( get_post_meta( $form_id, '_give_donation_levels', true ) );
+			if (give_has_variable_prices($donation->ID)) {
+				$prices     = maybe_unserialize(get_post_meta($form_id, '_give_donation_levels', true));
 				$item_price = '';
 				//Loop through prices
-				foreach ( $prices as $price ) {
+				foreach ($prices as $price) {
 					//Find a match between price_id and level_id
 					//First verify array keys exists THEN make the match
-					if ( ( isset( $args['price_id'] ) && isset( $price['_give_id']['level_id'] ) )
+					if ((isset($args['price_id']) && isset($price['_give_id']['level_id']))
 					     && $args['price_id'] == $price['_give_id']['level_id']
 					) {
 						$item_price = $price['_give_amount'];
 					}
 				}
 				//Fallback to the lowest price point
-				if ( $item_price == '' ) {
-					$item_price       = give_get_lowest_price_option( $donation->ID );
-					$args['price_id'] = give_get_lowest_price_id( $donation->ID );
+				if ($item_price == '') {
+					$item_price       = give_get_lowest_price_option($donation->ID);
+					$args['price_id'] = give_get_lowest_price_id($donation->ID);
 				}
 			} else {
 				//Simple form price
-				$item_price = give_get_form_price( $donation->ID );
+				$item_price = give_get_form_price($donation->ID);
 			}
 
 		}
 
 		// Sanitizing the price here so we don't have a dozen calls later
-		$item_price = give_sanitize_amount( $item_price );
-		$total      = round( $item_price, give_currency_decimal_filter() );
+		$item_price = give_sanitize_amount($item_price);
+		$total      = round($item_price, give_currency_decimal_filter());
 
 		//Add Options
 		$default_options = array();
-		if ( false !== $args['price_id'] ) {
+		if (false !== $args['price_id']) {
 			$default_options['price_id'] = (int) $args['price_id'];
 		}
-		$options = wp_parse_args( $options, $default_options );
+		$options = wp_parse_args($options, $default_options);
 
 		// Do not allow totals to go negative
-		if ( $total < 0 ) {
+		if ($total < 0) {
 			$total = 0;
 		}
 
 		$donation = array(
 			'name'     => $donation->post_title,
 			'id'       => $donation->ID,
-			'price'    => round( $total, give_currency_decimal_filter() ),
-			'subtotal' => round( $total, give_currency_decimal_filter() ),
+			'price'    => round($total, give_currency_decimal_filter()),
+			'subtotal' => round($total, give_currency_decimal_filter()),
 			'fees'     => $args['fees'],
 			'price_id' => $args['price_id'],
 			'action'   => 'add',
@@ -945,7 +945,7 @@ 
 
 		$this->pending['donations'][] = $donation;
 
-		$this->increase_subtotal( $total );
+		$this->increase_subtotal($total);
 
 		return true;
 
@@ -961,7 +961,7 @@ 
 	 *
 	 * @return bool If the item was removed or not
 	 */
-	public function remove_donation( $form_id, $args = array() ) {
+	public function remove_donation($form_id, $args = array()) {
 
 		// Set some defaults
 		$defaults = array(
@@ -969,12 +969,12 @@ 
 			'price'    => false,
 			'price_id' => false,
 		);
-		$args     = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$form = new Give_Donate_Form( $form_id );
+		$form = new Give_Donate_Form($form_id);
 
 		// Bail if this post isn't a valid give donation form
-		if ( ! $form || $form->post_type !== 'give_forms' ) {
+		if ( ! $form || $form->post_type !== 'give_forms') {
 			return false;
 		}
 
@@ -987,7 +987,7 @@ 
 
 		$this->pending['donations'][] = $pending_args;
 
-		$this->decrease_subtotal( $this->total );
+		$this->decrease_subtotal($this->total);
 
 		return true;
 	}
@@ -1002,7 +1002,7 @@ 
 	 *
 	 * @return bool If the fee was added
 	 */
-	public function add_fee( $args, $global = true ) {
+	public function add_fee($args, $global = true) {
 
 		$default_args = array(
 			'label'    => '',
@@ -1012,16 +1012,16 @@ 
 			'price_id' => 0,
 		);
 
-		$fee          = wp_parse_args( $args, $default_args );
+		$fee          = wp_parse_args($args, $default_args);
 		$this->fees[] = $fee;
 
 
 		$added_fee               = $fee;
 		$added_fee['action']     = 'add';
 		$this->pending['fees'][] = $added_fee;
-		reset( $this->fees );
+		reset($this->fees);
 
-		$this->increase_fees( $fee['amount'] );
+		$this->increase_fees($fee['amount']);
 
 		return true;
 	}
@@ -1035,11 +1035,11 @@ 
 	 *
 	 * @return bool     If the fee was removed successfully
 	 */
-	public function remove_fee( $key ) {
+	public function remove_fee($key) {
 		$removed = false;
 
-		if ( is_numeric( $key ) ) {
-			$removed = $this->remove_fee_by( 'index', $key );
+		if (is_numeric($key)) {
+			$removed = $this->remove_fee_by('index', $key);
 		}
 
 		return $removed;
@@ -1056,47 +1056,47 @@ 
 	 *
 	 * @return boolean             If the item is removed
 	 */
-	public function remove_fee_by( $key, $value, $global = false ) {
+	public function remove_fee_by($key, $value, $global = false) {
 
-		$allowed_fee_keys = apply_filters( 'give_payment_fee_keys', array(
+		$allowed_fee_keys = apply_filters('give_payment_fee_keys', array(
 			'index',
 			'label',
 			'amount',
 			'type',
-		) );
+		));
 
-		if ( ! in_array( $key, $allowed_fee_keys ) ) {
+		if ( ! in_array($key, $allowed_fee_keys)) {
 			return false;
 		}
 
 		$removed = false;
-		if ( 'index' === $key && array_key_exists( $value, $this->fees ) ) {
+		if ('index' === $key && array_key_exists($value, $this->fees)) {
 
-			$removed_fee             = $this->fees[ $value ];
+			$removed_fee             = $this->fees[$value];
 			$removed_fee['action']   = 'remove';
 			$this->pending['fees'][] = $removed_fee;
 
-			$this->decrease_fees( $removed_fee['amount'] );
+			$this->decrease_fees($removed_fee['amount']);
 
-			unset( $this->fees[ $value ] );
+			unset($this->fees[$value]);
 			$removed = true;
 
-		} else if ( 'index' !== $key ) {
+		} else if ('index' !== $key) {
 
-			foreach ( $this->fees as $index => $fee ) {
+			foreach ($this->fees as $index => $fee) {
 
-				if ( isset( $fee[ $key ] ) && $fee[ $key ] == $value ) {
+				if (isset($fee[$key]) && $fee[$key] == $value) {
 
 					$removed_fee             = $fee;
 					$removed_fee['action']   = 'remove';
 					$this->pending['fees'][] = $removed_fee;
 
-					$this->decrease_fees( $removed_fee['amount'] );
+					$this->decrease_fees($removed_fee['amount']);
 
-					unset( $this->fees[ $index ] );
+					unset($this->fees[$index]);
 					$removed = true;
 
-					if ( false === $global ) {
+					if (false === $global) {
 						break;
 					}
 
@@ -1106,8 +1106,8 @@ 
 
 		}
 
-		if ( true === $removed ) {
-			$this->fees = array_values( $this->fees );
+		if (true === $removed) {
+			$this->fees = array_values($this->fees);
 		}
 
 		return $removed;
@@ -1122,14 +1122,14 @@ 
 	 *
 	 * @return array        The Fees for the type specified
 	 */
-	public function get_fees( $type = 'all' ) {
+	public function get_fees($type = 'all') {
 		$fees = array();
 
-		if ( ! empty( $this->fees ) && is_array( $this->fees ) ) {
+		if ( ! empty($this->fees) && is_array($this->fees)) {
 
-			foreach ( $this->fees as $fee_id => $fee ) {
+			foreach ($this->fees as $fee_id => $fee) {
 
-				if ( 'all' != $type && ! empty( $fee['type'] ) && $type != $fee['type'] ) {
+				if ('all' != $type && ! empty($fee['type']) && $type != $fee['type']) {
 					continue;
 				}
 
@@ -1139,7 +1139,7 @@ 
 			}
 		}
 
-		return apply_filters( 'give_get_payment_fees', $fees, $this->ID, $this );
+		return apply_filters('give_get_payment_fees', $fees, $this->ID, $this);
 	}
 
 	/**
@@ -1151,13 +1151,13 @@ 
 	 *
 	 * @return void
 	 */
-	public function add_note( $note = false ) {
+	public function add_note($note = false) {
 		// Bail if no note specified
-		if ( ! $note ) {
+		if ( ! $note) {
 			return false;
 		}
 
-		give_insert_payment_note( $this->ID, $note );
+		give_insert_payment_note($this->ID, $note);
 	}
 
 	/**
@@ -1169,7 +1169,7 @@ 
 	 *
 	 * @return void
 	 */
-	private function increase_subtotal( $amount = 0.00 ) {
+	private function increase_subtotal($amount = 0.00) {
 		$amount = (float) $amount;
 		$this->subtotal += $amount;
 
@@ -1185,11 +1185,11 @@ 
 	 *
 	 * @return void
 	 */
-	private function decrease_subtotal( $amount = 0.00 ) {
+	private function decrease_subtotal($amount = 0.00) {
 		$amount = (float) $amount;
 		$this->subtotal -= $amount;
 
-		if ( $this->subtotal < 0 ) {
+		if ($this->subtotal < 0) {
 			$this->subtotal = 0;
 		}
 
@@ -1205,7 +1205,7 @@ 
 	 *
 	 * @return void
 	 */
-	private function increase_fees( $amount = 0.00 ) {
+	private function increase_fees($amount = 0.00) {
 		$amount = (float) $amount;
 		$this->fees_total += $amount;
 
@@ -1221,11 +1221,11 @@ 
 	 *
 	 * @return void
 	 */
-	private function decrease_fees( $amount = 0.00 ) {
+	private function decrease_fees($amount = 0.00) {
 		$amount = (float) $amount;
 		$this->fees_total -= $amount;
 
-		if ( $this->fees_total < 0 ) {
+		if ($this->fees_total < 0) {
 			$this->fees_total = 0;
 		}
 
@@ -1251,41 +1251,41 @@ 
 	 *
 	 * @return bool Returns if the status was successfully updated
 	 */
-	public function update_status( $status = false ) {
+	public function update_status($status = false) {
 
 		//standardize the 'complete(d)' status
-		if ( $status == 'completed' || $status == 'complete' ) {
+		if ($status == 'completed' || $status == 'complete') {
 			$status = 'publish';
 		}
 
-		$old_status = ! empty( $this->old_status ) ? $this->old_status : false;
+		$old_status = ! empty($this->old_status) ? $this->old_status : false;
 
-		if ( $old_status === $status ) {
+		if ($old_status === $status) {
 			return false; // Don't permit status changes that aren't changes
 		}
 
-		$do_change = apply_filters( 'give_should_update_payment_status', true, $this->ID, $status, $old_status );
+		$do_change = apply_filters('give_should_update_payment_status', true, $this->ID, $status, $old_status);
 
 		$updated = false;
 
 
-		if ( $do_change ) {
+		if ($do_change) {
 
-			do_action( 'give_before_payment_status_change', $this->ID, $status, $old_status );
+			do_action('give_before_payment_status_change', $this->ID, $status, $old_status);
 
 			$update_fields = array(
 				'ID'          => $this->ID,
 				'post_status' => $status,
-				'edit_date'   => current_time( 'mysql' )
+				'edit_date'   => current_time('mysql')
 			);
 
-			$updated = wp_update_post( apply_filters( 'give_update_payment_status_fields', $update_fields ) );
+			$updated = wp_update_post(apply_filters('give_update_payment_status_fields', $update_fields));
 
 			$all_payment_statuses  = give_get_payment_statuses();
-			$this->status_nicename = array_key_exists( $status, $all_payment_statuses ) ? $all_payment_statuses[ $status ] : ucfirst( $status );
+			$this->status_nicename = array_key_exists($status, $all_payment_statuses) ? $all_payment_statuses[$status] : ucfirst($status);
 
 			// Process any specific status functions
-			switch ( $status ) {
+			switch ($status) {
 				case 'refunded':
 					$this->process_refund();
 					break;
@@ -1297,7 +1297,7 @@ 
 					break;
 			}
 
-			do_action( 'give_update_payment_status', $this->ID, $status, $old_status );
+			do_action('give_update_payment_status', $this->ID, $status, $old_status);
 
 		}
 
@@ -1329,32 +1329,32 @@ 
 	 *
 	 * @return mixed             The value from the post meta
 	 */
-	public function get_meta( $meta_key = '_give_payment_meta', $single = true ) {
+	public function get_meta($meta_key = '_give_payment_meta', $single = true) {
 
-		$meta = get_post_meta( $this->ID, $meta_key, $single );
+		$meta = get_post_meta($this->ID, $meta_key, $single);
 
-		if ( $meta_key === '_give_payment_meta' ) {
+		if ($meta_key === '_give_payment_meta') {
 
-			if ( empty( $meta['key'] ) ) {
+			if (empty($meta['key'])) {
 				$meta['key'] = $this->setup_payment_key();
 			}
 
-			if ( empty( $meta['form_title'] ) ) {
+			if (empty($meta['form_title'])) {
 				$meta['form_title'] = $this->setup_form_title();
 			}
 
-			if ( empty( $meta['email'] ) ) {
+			if (empty($meta['email'])) {
 				$meta['email'] = $this->setup_email();
 			}
 
-			if ( empty( $meta['date'] ) ) {
-				$meta['date'] = get_post_field( 'post_date', $this->ID );
+			if (empty($meta['date'])) {
+				$meta['date'] = get_post_field('post_date', $this->ID);
 			}
 		}
 
-		$meta = apply_filters( 'give_get_payment_meta_' . $meta_key, $meta, $this->ID );
+		$meta = apply_filters('give_get_payment_meta_'.$meta_key, $meta, $this->ID);
 
-		return apply_filters( 'give_get_payment_meta', $meta, $this->ID, $meta_key );
+		return apply_filters('give_get_payment_meta', $meta, $this->ID, $meta_key);
 	}
 
 	/**
@@ -1368,23 +1368,23 @@ 
 	 *
 	 * @return int|bool Meta ID if the key didn't exist, true on successful update, false on failure
 	 */
-	public function update_meta( $meta_key = '', $meta_value = '', $prev_value = '' ) {
-		if ( empty( $meta_key ) ) {
+	public function update_meta($meta_key = '', $meta_value = '', $prev_value = '') {
+		if (empty($meta_key)) {
 			return false;
 		}
 
-		if ( $meta_key == 'key' || $meta_key == 'date' ) {
+		if ($meta_key == 'key' || $meta_key == 'date') {
 
 			$current_meta              = $this->get_meta();
-			$current_meta[ $meta_key ] = $meta_value;
+			$current_meta[$meta_key] = $meta_value;
 
 			$meta_key   = '_give_payment_meta';
 			$meta_value = $current_meta;
 
-		} else if ( $meta_key == 'email' || $meta_key == '_give_payment_user_email' ) {
+		} else if ($meta_key == 'email' || $meta_key == '_give_payment_user_email') {
 
-			$meta_value = apply_filters( 'give_give_update_payment_meta_' . $meta_key, $meta_value, $this->ID );
-			update_post_meta( $this->ID, '_give_payment_user_email', $meta_value );
+			$meta_value = apply_filters('give_give_update_payment_meta_'.$meta_key, $meta_value, $this->ID);
+			update_post_meta($this->ID, '_give_payment_user_email', $meta_value);
 
 			$current_meta                       = $this->get_meta();
 			$current_meta['user_info']['email'] = $meta_value;
@@ -1394,9 +1394,9 @@ 
 
 		}
 
-		$meta_value = apply_filters( 'give_update_payment_meta_' . $meta_key, $meta_value, $this->ID );
+		$meta_value = apply_filters('give_update_payment_meta_'.$meta_key, $meta_value, $this->ID);
 
-		return update_post_meta( $this->ID, $meta_key, $meta_value, $prev_value );
+		return update_post_meta($this->ID, $meta_key, $meta_value, $prev_value);
 	}
 
 	/**
@@ -1410,30 +1410,30 @@ 
 		$process_refund = true;
 
 		// If the payment was not in publish or revoked status, don't decrement stats as they were never incremented
-		if ( ( 'publish' != $this->old_status && 'revoked' != $this->old_status ) || 'refunded' != $this->status ) {
+		if (('publish' != $this->old_status && 'revoked' != $this->old_status) || 'refunded' != $this->status) {
 			$process_refund = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments
-		$process_refund = apply_filters( 'give_should_process_refund', $process_refund, $this );
+		$process_refund = apply_filters('give_should_process_refund', $process_refund, $this);
 
-		if ( false === $process_refund ) {
+		if (false === $process_refund) {
 			return;
 		}
 
-		do_action( 'give_pre_refund_payment', $this );
+		do_action('give_pre_refund_payment', $this);
 
-		$decrease_store_earnings = apply_filters( 'give_decrease_store_earnings_on_refund', true, $this );
-		$decrease_customer_value = apply_filters( 'give_decrease_customer_value_on_refund', true, $this );
-		$decrease_purchase_count = apply_filters( 'give_decrease_customer_purchase_count_on_refund', true, $this );
+		$decrease_store_earnings = apply_filters('give_decrease_store_earnings_on_refund', true, $this);
+		$decrease_customer_value = apply_filters('give_decrease_customer_value_on_refund', true, $this);
+		$decrease_purchase_count = apply_filters('give_decrease_customer_purchase_count_on_refund', true, $this);
 
-		$this->maybe_alter_stats( $decrease_store_earnings, $decrease_customer_value, $decrease_purchase_count );
+		$this->maybe_alter_stats($decrease_store_earnings, $decrease_customer_value, $decrease_purchase_count);
 		$this->delete_sales_logs();
 
 		// Clear the This Month earnings (this_monththis_month is NOT a typo)
-		delete_transient( md5( 'give_earnings_this_monththis_month' ) );
+		delete_transient(md5('give_earnings_this_monththis_month'));
 
-		do_action( 'give_post_refund_payment', $this );
+		do_action('give_post_refund_payment', $this);
 	}
 
 	/**
@@ -1457,29 +1457,29 @@ 
 		$process_pending = true;
 
 		// If the payment was not in publish or revoked status, don't decrement stats as they were never incremented
-		if ( ( 'publish' != $this->old_status && 'revoked' != $this->old_status ) || 'pending' != $this->status ) {
+		if (('publish' != $this->old_status && 'revoked' != $this->old_status) || 'pending' != $this->status) {
 			$process_pending = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments
-		$process_pending = apply_filters( 'give_should_process_pending', $process_pending, $this );
+		$process_pending = apply_filters('give_should_process_pending', $process_pending, $this);
 
-		if ( false === $process_pending ) {
+		if (false === $process_pending) {
 			return;
 		}
 
-		$decrease_store_earnings = apply_filters( 'give_decrease_store_earnings_on_pending', true, $this );
-		$decrease_customer_value = apply_filters( 'give_decrease_customer_value_on_pending', true, $this );
-		$decrease_purchase_count = apply_filters( 'give_decrease_customer_purchase_count_on_pending', true, $this );
+		$decrease_store_earnings = apply_filters('give_decrease_store_earnings_on_pending', true, $this);
+		$decrease_customer_value = apply_filters('give_decrease_customer_value_on_pending', true, $this);
+		$decrease_purchase_count = apply_filters('give_decrease_customer_purchase_count_on_pending', true, $this);
 
-		$this->maybe_alter_stats( $decrease_store_earnings, $decrease_customer_value, $decrease_purchase_count );
+		$this->maybe_alter_stats($decrease_store_earnings, $decrease_customer_value, $decrease_purchase_count);
 		$this->delete_sales_logs();
 
 		$this->completed_date = false;
-		$this->update_meta( '_give_completed_date', '' );
+		$this->update_meta('_give_completed_date', '');
 
 		// Clear the This Month earnings (this_monththis_month is NOT a typo)
-		delete_transient( md5( 'give_earnings_this_monththis_month' ) );
+		delete_transient(md5('give_earnings_this_monththis_month'));
 	}
 
 	/**
@@ -1493,25 +1493,25 @@ 
 	 *
 	 * @return void
 	 */
-	private function maybe_alter_stats( $alter_store_earnings, $alter_customer_value, $alter_customer_purchase_count ) {
+	private function maybe_alter_stats($alter_store_earnings, $alter_customer_value, $alter_customer_purchase_count) {
 
-		give_undo_purchase( false, $this->ID );
+		give_undo_purchase(false, $this->ID);
 
 		// Decrease store earnings
-		if ( true === $alter_store_earnings ) {
-			give_decrease_total_earnings( $this->total );
+		if (true === $alter_store_earnings) {
+			give_decrease_total_earnings($this->total);
 		}
 
 		// Decrement the stats for the customer
-		if ( ! empty( $this->customer_id ) ) {
+		if ( ! empty($this->customer_id)) {
 
-			$customer = new Give_Customer( $this->customer_id );
+			$customer = new Give_Customer($this->customer_id);
 
-			if ( true === $alter_customer_value ) {
-				$customer->decrease_value( $this->total );
+			if (true === $alter_customer_value) {
+				$customer->decrease_value($this->total);
 			}
 
-			if ( true === $alter_customer_purchase_count ) {
+			if (true === $alter_customer_purchase_count) {
 				$customer->decrease_purchase_count();
 			}
 
@@ -1557,13 +1557,13 @@ 
 	 * @return string The date the payment was completed
 	 */
 	private function setup_completed_date() {
-		$payment = get_post( $this->ID );
+		$payment = get_post($this->ID);
 
-		if ( 'pending' == $payment->post_status || 'preapproved' == $payment->post_status ) {
+		if ('pending' == $payment->post_status || 'preapproved' == $payment->post_status) {
 			return false; // This payment was never completed
 		}
 
-		$date = ( $date = $this->get_meta( '_give_completed_date', true ) ) ? $date : $payment->modified_date;
+		$date = ($date = $this->get_meta('_give_completed_date', true)) ? $date : $payment->modified_date;
 
 		return $date;
 	}
@@ -1575,7 +1575,7 @@ 
 	 * @return string The payment mode
 	 */
 	private function setup_mode() {
-		return $this->get_meta( '_give_payment_mode' );
+		return $this->get_meta('_give_payment_mode');
 	}
 
 	/**
@@ -1585,13 +1585,13 @@ 
 	 * @return float The payment total
 	 */
 	private function setup_total() {
-		$amount = $this->get_meta( '_give_payment_total', true );
+		$amount = $this->get_meta('_give_payment_total', true);
 
-		if ( empty( $amount ) && '0.00' != $amount ) {
-			$meta = $this->get_meta( '_give_payment_meta', true );
-			$meta = maybe_unserialize( $meta );
+		if (empty($amount) && '0.00' != $amount) {
+			$meta = $this->get_meta('_give_payment_meta', true);
+			$meta = maybe_unserialize($meta);
 
-			if ( isset( $meta['amount'] ) ) {
+			if (isset($meta['amount'])) {
 				$amount = $meta['amount'];
 			}
 		}
@@ -1620,9 +1620,9 @@ 
 	private function setup_fees_total() {
 		$fees_total = (float) 0.00;
 
-		$payment_fees = isset( $this->payment_meta['fees'] ) ? $this->payment_meta['fees'] : array();
-		if ( ! empty( $payment_fees ) ) {
-			foreach ( $payment_fees as $fee ) {
+		$payment_fees = isset($this->payment_meta['fees']) ? $this->payment_meta['fees'] : array();
+		if ( ! empty($payment_fees)) {
+			foreach ($payment_fees as $fee) {
 				$fees_total += (float) $fee['amount'];
 			}
 		}
@@ -1638,7 +1638,7 @@ 
 	 * @return string              The currency for the payment
 	 */
 	private function setup_currency() {
-		$currency = isset( $this->payment_meta['currency'] ) ? $this->payment_meta['currency'] : apply_filters( 'give_payment_currency_default', give_get_currency(), $this );
+		$currency = isset($this->payment_meta['currency']) ? $this->payment_meta['currency'] : apply_filters('give_payment_currency_default', give_get_currency(), $this);
 
 		return $currency;
 	}
@@ -1650,7 +1650,7 @@ 
 	 * @return array The Fees
 	 */
 	private function setup_fees() {
-		$payment_fees = isset( $this->payment_meta['fees'] ) ? $this->payment_meta['fees'] : array();
+		$payment_fees = isset($this->payment_meta['fees']) ? $this->payment_meta['fees'] : array();
 
 		return $payment_fees;
 	}
@@ -1662,7 +1662,7 @@ 
 	 * @return string The gateway
 	 */
 	private function setup_gateway() {
-		$gateway = $this->get_meta( '_give_payment_gateway', true );
+		$gateway = $this->get_meta('_give_payment_gateway', true);
 
 		return $gateway;
 	}
@@ -1674,12 +1674,12 @@ 
 	 * @return string The transaction ID for the payment
 	 */
 	private function setup_transaction_id() {
-		$transaction_id = $this->get_meta( '_give_payment_transaction_id', true );
+		$transaction_id = $this->get_meta('_give_payment_transaction_id', true);
 
-		if ( empty( $transaction_id ) || (int) $transaction_id === (int) $this->ID ) {
+		if (empty($transaction_id) || (int) $transaction_id === (int) $this->ID) {
 
 			$gateway        = $this->gateway;
-			$transaction_id = apply_filters( 'give_get_payment_transaction_id-' . $gateway, $this->ID );
+			$transaction_id = apply_filters('give_get_payment_transaction_id-'.$gateway, $this->ID);
 
 		}
 
@@ -1693,7 +1693,7 @@ 
 	 * @return string The IP address for the payment
 	 */
 	private function setup_ip() {
-		$ip = $this->get_meta( '_give_payment_user_ip', true );
+		$ip = $this->get_meta('_give_payment_user_ip', true);
 
 		return $ip;
 	}
@@ -1705,7 +1705,7 @@ 
 	 * @return int The Customer ID
 	 */
 	private function setup_customer_id() {
-		$customer_id = $this->get_meta( '_give_payment_customer_id', true );
+		$customer_id = $this->get_meta('_give_payment_customer_id', true);
 
 		return $customer_id;
 	}
@@ -1717,7 +1717,7 @@ 
 	 * @return int The User ID
 	 */
 	private function setup_user_id() {
-		$user_id = $this->get_meta( '_give_payment_user_id', true );
+		$user_id = $this->get_meta('_give_payment_user_id', true);
 
 		return $user_id;
 	}
@@ -1729,10 +1729,10 @@ 
 	 * @return string The email address for the payment
 	 */
 	private function setup_email() {
-		$email = $this->get_meta( '_give_payment_user_email', true );
+		$email = $this->get_meta('_give_payment_user_email', true);
 
-		if ( empty( $email ) ) {
-			$email = Give()->customers->get_column( 'email', $this->customer_id );
+		if (empty($email)) {
+			$email = Give()->customers->get_column('email', $this->customer_id);
 		}
 
 		return $email;
@@ -1750,15 +1750,15 @@ 
 			'last_name'  => $this->last_name,
 		);
 
-		$user_info = isset( $this->payment_meta['user_info'] ) ? maybe_unserialize( $this->payment_meta['user_info'] ) : array();
-		$user_info = wp_parse_args( $user_info, $defaults );
+		$user_info = isset($this->payment_meta['user_info']) ? maybe_unserialize($this->payment_meta['user_info']) : array();
+		$user_info = wp_parse_args($user_info, $defaults);
 
-		if ( empty( $user_info ) ) {
+		if (empty($user_info)) {
 			// Get the customer, but only if it's been created
-			$customer = new Give_Customer( $this->customer_id );
+			$customer = new Give_Customer($this->customer_id);
 
-			if ( $customer->id > 0 ) {
-				$name      = explode( ' ', $customer->name, 2 );
+			if ($customer->id > 0) {
+				$name      = explode(' ', $customer->name, 2);
 				$user_info = array(
 					'first_name' => $name[0],
 					'last_name'  => $name[1],
@@ -1768,29 +1768,29 @@ 
 			}
 		} else {
 			// Get the customer, but only if it's been created
-			$customer = new Give_Customer( $this->customer_id );
-			if ( $customer->id > 0 ) {
-				foreach ( $user_info as $key => $value ) {
-					if ( ! empty( $value ) ) {
+			$customer = new Give_Customer($this->customer_id);
+			if ($customer->id > 0) {
+				foreach ($user_info as $key => $value) {
+					if ( ! empty($value)) {
 						continue;
 					}
 
-					switch ( $key ) {
+					switch ($key) {
 						case 'first_name':
-							$name = explode( ' ', $customer->name, 2 );
+							$name = explode(' ', $customer->name, 2);
 
-							$user_info[ $key ] = $name[0];
+							$user_info[$key] = $name[0];
 							break;
 
 						case 'last_name':
-							$name      = explode( ' ', $customer->name, 2 );
-							$last_name = ! empty( $name[1] ) ? $name[1] : '';
+							$name      = explode(' ', $customer->name, 2);
+							$last_name = ! empty($name[1]) ? $name[1] : '';
 
-							$user_info[ $key ] = $last_name;
+							$user_info[$key] = $last_name;
 							break;
 
 						case 'email':
-							$user_info[ $key ] = $customer->email;
+							$user_info[$key] = $customer->email;
 							break;
 					}
 				}
@@ -1810,7 +1810,7 @@ 
 	 */
 	private function setup_address() {
 
-		$address = ! empty( $this->payment_meta['user_info']['address'] ) ? $this->payment_meta['user_info']['address'] : array(
+		$address = ! empty($this->payment_meta['user_info']['address']) ? $this->payment_meta['user_info']['address'] : array(
 			'line1'   => '',
 			'line2'   => '',
 			'city'    => '',
@@ -1830,7 +1830,7 @@ 
 	 */
 	private function setup_form_title() {
 
-		$form_id = $this->get_meta( '_give_payment_form_title', true );
+		$form_id = $this->get_meta('_give_payment_form_title', true);
 
 		return $form_id;
 	}
@@ -1843,7 +1843,7 @@ 
 	 */
 	private function setup_form_id() {
 
-		$form_id = $this->get_meta( '_give_payment_form_id', true );
+		$form_id = $this->get_meta('_give_payment_form_id', true);
 
 		return $form_id;
 	}
@@ -1855,7 +1855,7 @@ 
 	 * @return int The Form Price ID
 	 */
 	private function setup_price_id() {
-		$price_id = $this->get_meta( '_give_payment_price_id', true );
+		$price_id = $this->get_meta('_give_payment_price_id', true);
 
 		return $price_id;
 	}
@@ -1867,7 +1867,7 @@ 
 	 * @return string The Payment Key
 	 */
 	private function setup_payment_key() {
-		$key = $this->get_meta( '_give_payment_purchase_key', true );
+		$key = $this->get_meta('_give_payment_purchase_key', true);
 
 		return $key;
 	}
@@ -1881,11 +1881,11 @@ 
 	private function setup_payment_number() {
 		$number = $this->ID;
 
-		if ( give_get_option( 'enable_sequential' ) ) {
+		if (give_get_option('enable_sequential')) {
 
-			$number = $this->get_meta( '_give_payment_number', true );
+			$number = $this->get_meta('_give_payment_number', true);
 
-			if ( ! $number ) {
+			if ( ! $number) {
 
 				$number = $this->ID;
 
@@ -1902,7 +1902,7 @@ 
 	 * @return array The payment object as an array
 	 */
 	public function array_convert() {
-		return get_object_vars( $this );
+		return get_object_vars($this);
 	}
 
 	/**
@@ -1912,7 +1912,7 @@ 
 	 * @return string Date payment was completed
 	 */
 	private function get_completed_date() {
-		return apply_filters( 'give_payment_completed_date', $this->completed_date, $this->ID, $this );
+		return apply_filters('give_payment_completed_date', $this->completed_date, $this->ID, $this);
 	}
 
 	/**
@@ -1922,7 +1922,7 @@ 
 	 * @return float Payment subtotal
 	 */
 	private function get_subtotal() {
-		return apply_filters( 'give_get_payment_subtotal', $this->subtotal, $this->ID, $this );
+		return apply_filters('give_get_payment_subtotal', $this->subtotal, $this->ID, $this);
 	}
 
 	/**
@@ -1932,7 +1932,7 @@ 
 	 * @return string Payment currency code
 	 */
 	private function get_currency() {
-		return apply_filters( 'give_payment_currency_code', $this->currency, $this->ID, $this );
+		return apply_filters('give_payment_currency_code', $this->currency, $this->ID, $this);
 	}
 
 	/**
@@ -1942,7 +1942,7 @@ 
 	 * @return string Gateway used
 	 */
 	private function get_gateway() {
-		return apply_filters( 'give_payment_gateway', $this->gateway, $this->ID, $this );
+		return apply_filters('give_payment_gateway', $this->gateway, $this->ID, $this);
 	}
 
 	/**
@@ -1952,7 +1952,7 @@ 
 	 * @return string Transaction ID from merchant processor
 	 */
 	private function get_transaction_id() {
-		return apply_filters( 'give_get_payment_transaction_id', $this->transaction_id, $this->ID, $this );
+		return apply_filters('give_get_payment_transaction_id', $this->transaction_id, $this->ID, $this);
 	}
 
 	/**
@@ -1962,7 +1962,7 @@ 
 	 * @return string Payment IP address
 	 */
 	private function get_ip() {
-		return apply_filters( 'give_payment_user_ip', $this->ip, $this->ID, $this );
+		return apply_filters('give_payment_user_ip', $this->ip, $this->ID, $this);
 	}
 
 	/**
@@ -1972,7 +1972,7 @@ 
 	 * @return int Payment customer ID
 	 */
 	private function get_customer_id() {
-		return apply_filters( 'give_payment_customer_id', $this->customer_id, $this->ID, $this );
+		return apply_filters('give_payment_customer_id', $this->customer_id, $this->ID, $this);
 	}
 
 	/**
@@ -1982,7 +1982,7 @@ 
 	 * @return int Payment user ID
 	 */
 	private function get_user_id() {
-		return apply_filters( 'give_payment_user_id', $this->user_id, $this->ID, $this );
+		return apply_filters('give_payment_user_id', $this->user_id, $this->ID, $this);
 	}
 
 	/**
@@ -1992,7 +1992,7 @@ 
 	 * @return string Payment customer email
 	 */
 	private function get_email() {
-		return apply_filters( 'give_payment_user_email', $this->email, $this->ID, $this );
+		return apply_filters('give_payment_user_email', $this->email, $this->ID, $this);
 	}
 
 	/**
@@ -2002,7 +2002,7 @@ 
 	 * @return array Payment user info
 	 */
 	private function get_user_info() {
-		return apply_filters( 'give_payment_meta_user_info', $this->user_info, $this->ID, $this );
+		return apply_filters('give_payment_meta_user_info', $this->user_info, $this->ID, $this);
 	}
 
 	/**
@@ -2012,7 +2012,7 @@ 
 	 * @return array Payment billing address
 	 */
 	private function get_address() {
-		return apply_filters( 'give_payment_address', $this->address, $this->ID, $this );
+		return apply_filters('give_payment_address', $this->address, $this->ID, $this);
 	}
 
 	/**
@@ -2022,7 +2022,7 @@ 
 	 * @return string Payment key
 	 */
 	private function get_key() {
-		return apply_filters( 'give_payment_key', $this->key, $this->ID, $this );
+		return apply_filters('give_payment_key', $this->key, $this->ID, $this);
 	}
 
 	/**
@@ -2032,7 +2032,7 @@ 
 	 * @return string Payment key
 	 */
 	private function get_form_id() {
-		return apply_filters( 'give_payment_form_id', $this->form_id, $this->ID, $this );
+		return apply_filters('give_payment_form_id', $this->form_id, $this->ID, $this);
 	}
 
 	/**
@@ -2042,7 +2042,7 @@ 
 	 * @return int|string Payment number
 	 */
 	private function get_number() {
-		return apply_filters( 'give_payment_number', $this->number, $this->ID, $this );
+		return apply_filters('give_payment_number', $this->number, $this->ID, $this);
 	}
 
 }

includes/payments/functions.php

Doc Comments +6 added lines, -6 removed lines

@@ -607,7 +607,7 @@ 
  * @param int $year Year
  * @param int $hour Hour
  *
- * @return int $earnings Earnings
+ * @return double $earnings Earnings
  */
 function give_get_earnings_by_date( $day = null, $month_num, $year = null, $hour = null ) {
 
@@ -906,7 +906,7 @@ 
  *
  * @param int $payment_id Payment ID
  *
- * @return array $user_info User Info Meta Values
+ * @return string $user_info User Info Meta Values
  */
 function give_get_payment_meta_user_info( $payment_id ) {
 	$payment = new Give_Payment( $payment_id );
@@ -922,7 +922,7 @@ 
  *
  * @param int $payment_id Payment ID
  *
- * @return int $form_id
+ * @return string $form_id
  */
 function give_get_payment_form_id( $payment_id ) {
 	$payment = new Give_Payment( $payment_id );
@@ -968,7 +968,7 @@ 
  *
  * @param int $payment_id Payment ID
  *
- * @return string $user_id User ID
+ * @return integer $user_id User ID
  */
 function give_get_payment_user_id( $payment_id ) {
 	$payment = new Give_Payment( $payment_id );
@@ -983,7 +983,7 @@ 
  *
  * @param int $payment_id Payment ID
  *
- * @return string $customer_id Customer ID
+ * @return integer $customer_id Customer ID
  */
 function give_get_payment_customer_id( $payment_id ) {
 	$payment = new Give_Payment( $payment_id );
@@ -1580,7 +1580,7 @@ 
  * @param array $where
  * @param object $wp_comment_query WordPress Comment Query Object
  *
- * @return array $where
+ * @return string $where
  */
 function give_hide_payment_notes_from_feeds( $where, $wp_comment_query ) {
 	global $wpdb;

Spacing +380 added lines, -380 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -32,15 +32,15 @@ 
  *
  * @return object $payments Payments retrieved from the database
  */
-function give_get_payments( $args = array() ) {
+function give_get_payments($args = array()) {
 
 	// Fallback to post objects to ensure backwards compatibility
-	if ( ! isset( $args['output'] ) ) {
+	if ( ! isset($args['output'])) {
 		$args['output'] = 'posts';
 	}
 
-	$args     = apply_filters( 'give_get_payments_args', $args );
-	$payments = new Give_Payments_Query( $args );
+	$args     = apply_filters('give_get_payments_args', $args);
+	$payments = new Give_Payments_Query($args);
 
 	return $payments->get_payments();
 }
@@ -55,48 +55,48 @@ 
  *
  * @return      mixed
  */
-function give_get_payment_by( $field = '', $value = '' ) {
+function give_get_payment_by($field = '', $value = '') {
 
-	if ( empty( $field ) || empty( $value ) ) {
+	if (empty($field) || empty($value)) {
 		return false;
 	}
 
-	switch ( strtolower( $field ) ) {
+	switch (strtolower($field)) {
 
 		case 'id':
-			$payment = new Give_Payment( $value );
+			$payment = new Give_Payment($value);
 			$id      = $payment->ID;
 
-			if ( empty( $id ) ) {
+			if (empty($id)) {
 				return false;
 			}
 
 			break;
 
 		case 'key':
-			$payment = give_get_payments( array(
+			$payment = give_get_payments(array(
 				'meta_key'       => '_give_payment_purchase_key',
 				'meta_value'     => $value,
 				'posts_per_page' => 1,
 				'fields'         => 'ids',
-			) );
+			));
 
-			if ( $payment ) {
-				$payment = new Give_Payment( $payment[0] );
+			if ($payment) {
+				$payment = new Give_Payment($payment[0]);
 			}
 
 			break;
 
 		case 'payment_number':
-			$payment = give_get_payments( array(
+			$payment = give_get_payments(array(
 				'meta_key'       => '_give_payment_number',
 				'meta_value'     => $value,
 				'posts_per_page' => 1,
 				'fields'         => 'ids',
-			) );
+			));
 
-			if ( $payment ) {
-				$payment = new Give_Payment( $payment[0] );
+			if ($payment) {
+				$payment = new Give_Payment($payment[0]);
 			}
 
 			break;
@@ -105,7 +105,7 @@ 
 			return false;
 	}
 
-	if ( $payment ) {
+	if ($payment) {
 		return $payment;
 	}
 
@@ -121,23 +121,23 @@ 
  *
  * @return int|bool Payment ID if payment is inserted, false otherwise
  */
-function give_insert_payment( $payment_data = array() ) {
+function give_insert_payment($payment_data = array()) {
 
-	if ( empty( $payment_data ) ) {
+	if (empty($payment_data)) {
 		return false;
 	}
 
 	$payment    = new Give_Payment();
-	$gateway    = ! empty( $payment_data['gateway'] ) ? $payment_data['gateway'] : '';
-	$gateway    = empty( $gateway ) && isset( $_POST['give-gateway'] ) ? $_POST['give-gateway'] : $gateway;
-	$form_id    = isset( $payment_data['give_form_id'] ) ? $payment_data['give_form_id'] : 0;
-	$price_id   = isset( $payment_data['give_price_id'] ) ? $payment_data['give_price_id'] : give_get_price_id( $payment_data['give_form_id'], $payment_data['price'] );
-	$form_title = isset( $payment_data['give_form_title'] ) ? $payment_data['give_form_title'] : get_the_title( $form_id );
+	$gateway    = ! empty($payment_data['gateway']) ? $payment_data['gateway'] : '';
+	$gateway    = empty($gateway) && isset($_POST['give-gateway']) ? $_POST['give-gateway'] : $gateway;
+	$form_id    = isset($payment_data['give_form_id']) ? $payment_data['give_form_id'] : 0;
+	$price_id   = isset($payment_data['give_price_id']) ? $payment_data['give_price_id'] : give_get_price_id($payment_data['give_form_id'], $payment_data['price']);
+	$form_title = isset($payment_data['give_form_title']) ? $payment_data['give_form_title'] : get_the_title($form_id);
 
 	//Set properties
 	$payment->total          = $payment_data['price'];
-	$payment->status         = ! empty( $payment_data['status'] ) ? $payment_data['status'] : 'pending';
-	$payment->currency       = ! empty( $payment_data['currency'] ) ? $payment_data['currency'] : give_get_currency();
+	$payment->status         = ! empty($payment_data['status']) ? $payment_data['status'] : 'pending';
+	$payment->currency       = ! empty($payment_data['currency']) ? $payment_data['currency'] : give_get_currency();
 	$payment->user_info      = $payment_data['user_info'];
 	$payment->gateway        = $gateway;
 	$payment->form_title     = $form_title;
@@ -151,40 +151,40 @@ 
 	$payment->ip             = give_get_ip();
 	$payment->key            = $payment_data['purchase_key'];
 	$payment->mode           = give_is_test_mode() ? 'test' : 'live';
-	$payment->parent_payment = ! empty( $payment_data['parent'] ) ? absint( $payment_data['parent'] ) : '';
+	$payment->parent_payment = ! empty($payment_data['parent']) ? absint($payment_data['parent']) : '';
 
 	//Add the donation
 	$args = array(
 		'price'    => $payment->total,
 		'price_id' => $payment->price_id,
-		'fees'     => isset( $payment_data['fees'] ) ? $payment_data['fees'] : array()
+		'fees'     => isset($payment_data['fees']) ? $payment_data['fees'] : array()
 	);
 
-	$payment->add_donation( $payment->form_id, $args );
+	$payment->add_donation($payment->form_id, $args);
 
 	//Set date if present
-	if ( isset( $payment_data['post_date'] ) ) {
+	if (isset($payment_data['post_date'])) {
 		$payment->date = $payment_data['post_date'];
 	}
 
 	//Handle sequential payments
-	if ( give_get_option( 'enable_sequential' ) ) {
+	if (give_get_option('enable_sequential')) {
 		$number          = give_get_next_payment_number();
-		$payment->number = give_format_payment_number( $number );
-		update_option( 'give_last_payment_number', $number );
+		$payment->number = give_format_payment_number($number);
+		update_option('give_last_payment_number', $number);
 	}
 
 	// Clear the user's purchased cache
-	delete_transient( 'give_user_' . $payment_data['user_info']['id'] . '_purchases' );
+	delete_transient('give_user_'.$payment_data['user_info']['id'].'_purchases');
 
 	//Save payment
 	$payment->save();
 
 	//Hook it
-	do_action( 'give_insert_payment', $payment->ID, $payment_data );
+	do_action('give_insert_payment', $payment->ID, $payment_data);
 
 	//Return payment ID upon success
-	if ( ! empty( $payment->ID ) ) {
+	if ( ! empty($payment->ID)) {
 		return $payment->ID;
 	}
 
@@ -203,9 +203,9 @@ 
  *
  * @return bool
  */
-function give_update_payment_status( $payment_id, $new_status = 'publish' ) {
+function give_update_payment_status($payment_id, $new_status = 'publish') {
 
-	$payment         = new Give_Payment( $payment_id );
+	$payment         = new Give_Payment($payment_id);
 	$payment->status = $new_status;
 	$updated         = $payment->save();
 
@@ -225,52 +225,52 @@ 
  *
  * @return void
  */
-function give_delete_purchase( $payment_id = 0, $update_customer = true ) {
+function give_delete_purchase($payment_id = 0, $update_customer = true) {
 	global $give_logs;
 
-	$payment     = new Give_Payment( $payment_id );
-	$amount      = give_get_payment_amount( $payment_id );
+	$payment     = new Give_Payment($payment_id);
+	$amount      = give_get_payment_amount($payment_id);
 	$status      = $payment->post_status;
-	$customer_id = give_get_payment_customer_id( $payment_id );
-	$customer    = new Give_Customer( $customer_id );
+	$customer_id = give_get_payment_customer_id($payment_id);
+	$customer    = new Give_Customer($customer_id);
 
 	//Only undo purchases that aren't these statuses
-	$dont_undo_statuses = apply_filters( 'give_undo_purchase_statuses', array(
+	$dont_undo_statuses = apply_filters('give_undo_purchase_statuses', array(
 		'pending',
 		'cancelled'
-	) );
+	));
 
-	if ( ! in_array( $status, $dont_undo_statuses ) ) {
-		give_undo_purchase( false, $payment_id );
+	if ( ! in_array($status, $dont_undo_statuses)) {
+		give_undo_purchase(false, $payment_id);
 	}
 
-	if ( $status == 'publish' ) {
+	if ($status == 'publish') {
 
 		// Only decrease earnings if they haven't already been decreased (or were never increased for this payment)
-		give_decrease_total_earnings( $amount );
+		give_decrease_total_earnings($amount);
 		// Clear the This Month earnings (this_monththis_month is NOT a typo)
-		delete_transient( md5( 'give_earnings_this_monththis_month' ) );
+		delete_transient(md5('give_earnings_this_monththis_month'));
 
-		if ( $customer->id && $update_customer ) {
+		if ($customer->id && $update_customer) {
 
 			// Decrement the stats for the customer
 			$customer->decrease_purchase_count();
-			$customer->decrease_value( $amount );
+			$customer->decrease_value($amount);
 
 		}
 	}
 
-	do_action( 'give_payment_delete', $payment_id );
+	do_action('give_payment_delete', $payment_id);
 
-	if ( $customer->id && $update_customer ) {
+	if ($customer->id && $update_customer) {
 
 		// Remove the payment ID from the customer
-		$customer->remove_payment( $payment_id );
+		$customer->remove_payment($payment_id);
 
 	}
 
 	// Remove the payment
-	wp_delete_post( $payment_id, true );
+	wp_delete_post($payment_id, true);
 
 	// Remove related sale log entries
 	$give_logs->delete_logs(
@@ -284,7 +284,7 @@ 
 		)
 	);
 
-	do_action( 'give_payment_deleted', $payment_id );
+	do_action('give_payment_deleted', $payment_id);
 }
 
 /**
@@ -297,25 +297,25 @@ 
  *
  * @return void
  */
-function give_undo_purchase( $form_id = false, $payment_id ) {
+function give_undo_purchase($form_id = false, $payment_id) {
 
-	if ( ! empty( $form_id ) ) {
+	if ( ! empty($form_id)) {
 		$form_id = false;
-		_give_deprected_argument( 'form_id', 'give_undo_purchase', '1.5' );
+		_give_deprected_argument('form_id', 'give_undo_purchase', '1.5');
 	}
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	$maybe_decrease_earnings = apply_filters( 'give_decrease_earnings_on_undo', true, $payment, $payment->form_id );
-	if ( true === $maybe_decrease_earnings ) {
+	$maybe_decrease_earnings = apply_filters('give_decrease_earnings_on_undo', true, $payment, $payment->form_id);
+	if (true === $maybe_decrease_earnings) {
 		// decrease earnings
-		give_decrease_earnings( $payment->form_id, $payment->total );
+		give_decrease_earnings($payment->form_id, $payment->total);
 	}
 
-	$maybe_decrease_sales = apply_filters( 'give_decrease_sales_on_undo', true, $payment, $payment->form_id );
-	if ( true === $maybe_decrease_sales ) {
+	$maybe_decrease_sales = apply_filters('give_decrease_sales_on_undo', true, $payment, $payment->form_id);
+	if (true === $maybe_decrease_sales) {
 		// decrease purchase count
-		give_decrease_purchase_count( $payment->form_id );
+		give_decrease_purchase_count($payment->form_id);
 	}
 
 }
@@ -332,7 +332,7 @@ 
  *
  * @return array $count Number of payments sorted by payment status
  */
-function give_count_payments( $args = array() ) {
+function give_count_payments($args = array()) {
 
 	global $wpdb;
 
@@ -344,18 +344,18 @@ 
 		'form_id'    => null,
 	);
 
-	$args = wp_parse_args( $args, $defaults );
+	$args = wp_parse_args($args, $defaults);
 
 	$select = "SELECT p.post_status,count( * ) AS num_posts";
 	$join   = '';
 	$where  = "WHERE p.post_type = 'give_payment'";
 
 	// Count payments for a specific user
-	if ( ! empty( $args['user'] ) ) {
+	if ( ! empty($args['user'])) {
 
-		if ( is_email( $args['user'] ) ) {
+		if (is_email($args['user'])) {
 			$field = 'email';
-		} elseif ( is_numeric( $args['user'] ) ) {
+		} elseif (is_numeric($args['user'])) {
 			$field = 'id';
 		} else {
 			$field = '';
@@ -363,108 +363,108 @@ 
 
 		$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
 
-		if ( ! empty( $field ) ) {
+		if ( ! empty($field)) {
 			$where .= "
 				AND m.meta_key = '_give_payment_user_{$field}'
 				AND m.meta_value = '{$args['user']}'";
 		}
 
 		// Count payments for a search
-	} elseif ( ! empty( $args['s'] ) ) {
+	} elseif ( ! empty($args['s'])) {
 
-		if ( is_email( $args['s'] ) || strlen( $args['s'] ) == 32 ) {
+		if (is_email($args['s']) || strlen($args['s']) == 32) {
 
-			if ( is_email( $args['s'] ) ) {
+			if (is_email($args['s'])) {
 				$field = '_give_payment_user_email';
 			} else {
 				$field = '_give_payment_purchase_key';
 			}
 
 			$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
-			$where .= $wpdb->prepare( "
+			$where .= $wpdb->prepare("
 				AND m.meta_key = %s
 				AND m.meta_value = %s",
 				$field,
 				$args['s']
 			);
 
-		} elseif ( '#' == substr( $args['s'], 0, 1 ) ) {
+		} elseif ('#' == substr($args['s'], 0, 1)) {
 
-			$search = str_replace( '#:', '', $args['s'] );
-			$search = str_replace( '#', '', $search );
+			$search = str_replace('#:', '', $args['s']);
+			$search = str_replace('#', '', $search);
 
 			$select = "SELECT p2.post_status,count( * ) AS num_posts ";
 			$join   = "LEFT JOIN $wpdb->postmeta m ON m.meta_key = '_give_log_payment_id' AND m.post_id = p.ID ";
 			$join .= "INNER JOIN $wpdb->posts p2 ON m.meta_value = p2.ID ";
 			$where = "WHERE p.post_type = 'give_log' ";
-			$where .= $wpdb->prepare( "AND p.post_parent = %d} ", $search );
+			$where .= $wpdb->prepare("AND p.post_parent = %d} ", $search);
 
-		} elseif ( is_numeric( $args['s'] ) ) {
+		} elseif (is_numeric($args['s'])) {
 
 			$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
-			$where .= $wpdb->prepare( "
+			$where .= $wpdb->prepare("
 				AND m.meta_key = '_give_payment_user_id'
 				AND m.meta_value = %d",
 				$args['s']
 			);
 
 		} else {
-			$search = $wpdb->esc_like( $args['s'] );
-			$search = '%' . $search . '%';
+			$search = $wpdb->esc_like($args['s']);
+			$search = '%'.$search.'%';
 
-			$where .= $wpdb->prepare( "AND ((p.post_title LIKE %s) OR (p.post_content LIKE %s))", $search, $search );
+			$where .= $wpdb->prepare("AND ((p.post_title LIKE %s) OR (p.post_content LIKE %s))", $search, $search);
 		}
 
 	}
 
-	if ( ! empty( $args['form_id'] ) && is_numeric( $args['form_id'] ) ) {
+	if ( ! empty($args['form_id']) && is_numeric($args['form_id'])) {
 
-		$where .= $wpdb->prepare( " AND p.post_parent = %d", $args['form_id'] );
+		$where .= $wpdb->prepare(" AND p.post_parent = %d", $args['form_id']);
 
 	}
 	// Limit payments count by date
-	if ( ! empty( $args['start-date'] ) && false !== strpos( $args['start-date'], '/' ) ) {
+	if ( ! empty($args['start-date']) && false !== strpos($args['start-date'], '/')) {
 
-		$date_parts = explode( '/', $args['start-date'] );
-		$month      = ! empty( $date_parts[0] ) && is_numeric( $date_parts[0] ) ? $date_parts[0] : 0;
-		$day        = ! empty( $date_parts[1] ) && is_numeric( $date_parts[1] ) ? $date_parts[1] : 0;
-		$year       = ! empty( $date_parts[2] ) && is_numeric( $date_parts[2] ) ? $date_parts[2] : 0;
+		$date_parts = explode('/', $args['start-date']);
+		$month      = ! empty($date_parts[0]) && is_numeric($date_parts[0]) ? $date_parts[0] : 0;
+		$day        = ! empty($date_parts[1]) && is_numeric($date_parts[1]) ? $date_parts[1] : 0;
+		$year       = ! empty($date_parts[2]) && is_numeric($date_parts[2]) ? $date_parts[2] : 0;
 
-		$is_date = checkdate( $month, $day, $year );
-		if ( false !== $is_date ) {
+		$is_date = checkdate($month, $day, $year);
+		if (false !== $is_date) {
 
-			$date = new DateTime( $args['start-date'] );
-			$where .= $wpdb->prepare( " AND p.post_date >= '%s'", $date->format( 'Y-m-d' ) );
+			$date = new DateTime($args['start-date']);
+			$where .= $wpdb->prepare(" AND p.post_date >= '%s'", $date->format('Y-m-d'));
 
 		}
 
 		// Fixes an issue with the payments list table counts when no end date is specified (partiy with stats class)
-		if ( empty( $args['end-date'] ) ) {
+		if (empty($args['end-date'])) {
 			$args['end-date'] = $args['start-date'];
 		}
 
 	}
 
-	if ( ! empty ( $args['end-date'] ) && false !== strpos( $args['end-date'], '/' ) ) {
+	if ( ! empty ($args['end-date']) && false !== strpos($args['end-date'], '/')) {
 
-		$date_parts = explode( '/', $args['end-date'] );
+		$date_parts = explode('/', $args['end-date']);
 
-		$month = ! empty( $date_parts[0] ) ? $date_parts[0] : 0;
-		$day   = ! empty( $date_parts[1] ) ? $date_parts[1] : 0;
-		$year  = ! empty( $date_parts[2] ) ? $date_parts[2] : 0;
+		$month = ! empty($date_parts[0]) ? $date_parts[0] : 0;
+		$day   = ! empty($date_parts[1]) ? $date_parts[1] : 0;
+		$year  = ! empty($date_parts[2]) ? $date_parts[2] : 0;
 
-		$is_date = checkdate( $month, $day, $year );
-		if ( false !== $is_date ) {
+		$is_date = checkdate($month, $day, $year);
+		if (false !== $is_date) {
 
-			$date = new DateTime( $args['end-date'] );
-			$where .= $wpdb->prepare( " AND p.post_date <= '%s'", $date->format( 'Y-m-d' ) );
+			$date = new DateTime($args['end-date']);
+			$where .= $wpdb->prepare(" AND p.post_date <= '%s'", $date->format('Y-m-d'));
 
 		}
 
 	}
 
-	$where = apply_filters( 'give_count_payments_where', $where );
-	$join  = apply_filters( 'give_count_payments_join', $join );
+	$where = apply_filters('give_count_payments_where', $where);
+	$join  = apply_filters('give_count_payments_join', $join);
 
 	$query = "$select
 		FROM $wpdb->posts p
@@ -473,36 +473,36 @@ 
 		GROUP BY p.post_status
 	";
 
-	$cache_key = md5( $query );
+	$cache_key = md5($query);
 
-	$count = wp_cache_get( $cache_key, 'counts' );
-	if ( false !== $count ) {
+	$count = wp_cache_get($cache_key, 'counts');
+	if (false !== $count) {
 		return $count;
 	}
 
-	$count = $wpdb->get_results( $query, ARRAY_A );
+	$count = $wpdb->get_results($query, ARRAY_A);
 
 	$stats    = array();
 	$statuses = get_post_stati();
-	if ( isset( $statuses['private'] ) && empty( $args['s'] ) ) {
-		unset( $statuses['private'] );
+	if (isset($statuses['private']) && empty($args['s'])) {
+		unset($statuses['private']);
 	}
 
-	foreach ( $statuses as $state ) {
-		$stats[ $state ] = 0;
+	foreach ($statuses as $state) {
+		$stats[$state] = 0;
 	}
 
-	foreach ( (array) $count as $row ) {
+	foreach ((array) $count as $row) {
 
-		if ( 'private' == $row['post_status'] && empty( $args['s'] ) ) {
+		if ('private' == $row['post_status'] && empty($args['s'])) {
 			continue;
 		}
 
-		$stats[ $row['post_status'] ] = $row['num_posts'];
+		$stats[$row['post_status']] = $row['num_posts'];
 	}
 
 	$stats = (object) $stats;
-	wp_cache_set( $cache_key, $stats, 'counts' );
+	wp_cache_set($cache_key, $stats, 'counts');
 
 	return $stats;
 }
@@ -517,12 +517,12 @@ 
  *
  * @return bool true if payment exists, false otherwise
  */
-function give_check_for_existing_payment( $payment_id ) {
+function give_check_for_existing_payment($payment_id) {
 	$exists  = false;
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
 
-	if ( $payment_id === $payment->ID && 'publish' === $payment->status ) {
+	if ($payment_id === $payment->ID && 'publish' === $payment->status) {
 		$exists = true;
 	}
 
@@ -539,29 +539,29 @@ 
  *
  * @return bool|mixed if payment status exists, false otherwise
  */
-function give_get_payment_status( $payment, $return_label = false ) {
+function give_get_payment_status($payment, $return_label = false) {
 
-	if ( ! is_object( $payment ) || ! isset( $payment->post_status ) ) {
+	if ( ! is_object($payment) || ! isset($payment->post_status)) {
 		return false;
 	}
 
 	$statuses = give_get_payment_statuses();
 
-	if ( ! is_array( $statuses ) || empty( $statuses ) ) {
+	if ( ! is_array($statuses) || empty($statuses)) {
 		return false;
 	}
 
-	$payment = new Give_Payment( $payment->ID );
+	$payment = new Give_Payment($payment->ID);
 
-	if ( array_key_exists( $payment->status, $statuses ) ) {
-		if ( true === $return_label ) {
-			return $statuses[ $payment->status ];
+	if (array_key_exists($payment->status, $statuses)) {
+		if (true === $return_label) {
+			return $statuses[$payment->status];
 		} else {
 			// Account that our 'publish' status is labeled 'Complete'
 			$post_status = 'publish' == $payment->status ? 'Complete' : $payment->post_status;
 
 			// Make sure we're matching cases, since they matter
-			return array_search( strtolower( $post_status ), array_map( 'strtolower', $statuses ) );
+			return array_search(strtolower($post_status), array_map('strtolower', $statuses));
 		}
 	}
 
@@ -576,17 +576,17 @@ 
  */
 function give_get_payment_statuses() {
 	$payment_statuses = array(
-		'pending'     => __( 'Pending', 'give' ),
-		'publish'     => __( 'Complete', 'give' ),
-		'refunded'    => __( 'Refunded', 'give' ),
-		'failed'      => __( 'Failed', 'give' ),
-		'cancelled'   => __( 'Cancelled', 'give' ),
-		'abandoned'   => __( 'Abandoned', 'give' ),
-		'preapproval' => __( 'Pre-Approved', 'give' ),
-		'revoked'     => __( 'Revoked', 'give' )
+		'pending'     => __('Pending', 'give'),
+		'publish'     => __('Complete', 'give'),
+		'refunded'    => __('Refunded', 'give'),
+		'failed'      => __('Failed', 'give'),
+		'cancelled'   => __('Cancelled', 'give'),
+		'abandoned'   => __('Abandoned', 'give'),
+		'preapproval' => __('Pre-Approved', 'give'),
+		'revoked'     => __('Revoked', 'give')
 	);
 
-	return apply_filters( 'give_payment_statuses', $payment_statuses );
+	return apply_filters('give_payment_statuses', $payment_statuses);
 }
 
 /**
@@ -598,10 +598,10 @@ 
  * @return array $payment_status All the available payment statuses
  */
 function give_get_payment_status_keys() {
-	$statuses = array_keys( give_get_payment_statuses() );
-	asort( $statuses );
+	$statuses = array_keys(give_get_payment_statuses());
+	asort($statuses);
 
-	return array_values( $statuses );
+	return array_values($statuses);
 }
 
 /**
@@ -616,7 +616,7 @@ 
  *
  * @return int $earnings Earnings
  */
-function give_get_earnings_by_date( $day = null, $month_num, $year = null, $hour = null ) {
+function give_get_earnings_by_date($day = null, $month_num, $year = null, $hour = null) {
 
 	// This is getting deprecated soon. Use Give_Payment_Stats with the get_earnings() method instead
 
@@ -627,41 +627,41 @@ 
 		'nopaging'               => true,
 		'year'                   => $year,
 		'monthnum'               => $month_num,
-		'post_status'            => array( 'publish', 'revoked' ),
+		'post_status'            => array('publish', 'revoked'),
 		'fields'                 => 'ids',
 		'update_post_term_cache' => false
 	);
-	if ( ! empty( $day ) ) {
+	if ( ! empty($day)) {
 		$args['day'] = $day;
 	}
 
-	if ( ! empty( $hour ) ) {
+	if ( ! empty($hour)) {
 		$args['hour'] = $hour;
 	}
 
-	$args = apply_filters( 'give_get_earnings_by_date_args', $args );
-	$key  = 'give_stats_' . substr( md5( serialize( $args ) ), 0, 15 );
+	$args = apply_filters('give_get_earnings_by_date_args', $args);
+	$key  = 'give_stats_'.substr(md5(serialize($args)), 0, 15);
 
-	if ( ! empty( $_GET['_wpnonce'] ) && wp_verify_nonce( $_GET['_wpnonce'], 'give-refresh-reports' ) ) {
+	if ( ! empty($_GET['_wpnonce']) && wp_verify_nonce($_GET['_wpnonce'], 'give-refresh-reports')) {
 		$earnings = false;
 	} else {
-		$earnings = get_transient( $key );
+		$earnings = get_transient($key);
 	}
 
-	if ( false === $earnings ) {
-		$sales    = get_posts( $args );
+	if (false === $earnings) {
+		$sales    = get_posts($args);
 		$earnings = 0;
-		if ( $sales ) {
-			$sales = implode( ',', $sales );
+		if ($sales) {
+			$sales = implode(',', $sales);
 
-			$earnings = $wpdb->get_var( "SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN ({$sales})" );
+			$earnings = $wpdb->get_var("SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN ({$sales})");
 
 		}
 		// Cache the results for one hour
-		set_transient( $key, $earnings, HOUR_IN_SECONDS );
+		set_transient($key, $earnings, HOUR_IN_SECONDS);
 	}
 
-	return round( $earnings, 2 );
+	return round($earnings, 2);
 }
 
 /**
@@ -676,7 +676,7 @@ 
  *
  * @return int $count Sales
  */
-function give_get_sales_by_date( $day = null, $month_num = null, $year = null, $hour = null ) {
+function give_get_sales_by_date($day = null, $month_num = null, $year = null, $hour = null) {
 
 	// This is getting deprecated soon. Use Give_Payment_Stats with the get_sales() method instead
 	$args = array(
@@ -684,14 +684,14 @@ 
 		'nopaging'               => true,
 		'year'                   => $year,
 		'fields'                 => 'ids',
-		'post_status'            => array( 'publish', 'revoked' ),
+		'post_status'            => array('publish', 'revoked'),
 		'update_post_meta_cache' => false,
 		'update_post_term_cache' => false
 	);
 
-	$show_free = apply_filters( 'give_sales_by_date_show_free', true, $args );
+	$show_free = apply_filters('give_sales_by_date_show_free', true, $args);
 
-	if ( false === $show_free ) {
+	if (false === $show_free) {
 		$args['meta_query'] = array(
 			array(
 				'key'     => '_give_payment_total',
@@ -702,33 +702,33 @@ 
 		);
 	}
 
-	if ( ! empty( $month_num ) ) {
+	if ( ! empty($month_num)) {
 		$args['monthnum'] = $month_num;
 	}
 
-	if ( ! empty( $day ) ) {
+	if ( ! empty($day)) {
 		$args['day'] = $day;
 	}
 
-	if ( ! empty( $hour ) ) {
+	if ( ! empty($hour)) {
 		$args['hour'] = $hour;
 	}
 
-	$args = apply_filters( 'give_get_sales_by_date_args', $args );
+	$args = apply_filters('give_get_sales_by_date_args', $args);
 
-	$key = 'give_stats_' . substr( md5( serialize( $args ) ), 0, 15 );
+	$key = 'give_stats_'.substr(md5(serialize($args)), 0, 15);
 
-	if ( ! empty( $_GET['_wpnonce'] ) && wp_verify_nonce( $_GET['_wpnonce'], 'give-refresh-reports' ) ) {
+	if ( ! empty($_GET['_wpnonce']) && wp_verify_nonce($_GET['_wpnonce'], 'give-refresh-reports')) {
 		$count = false;
 	} else {
-		$count = get_transient( $key );
+		$count = get_transient($key);
 	}
 
-	if ( false === $count ) {
-		$sales = new WP_Query( $args );
+	if (false === $count) {
+		$sales = new WP_Query($args);
 		$count = (int) $sales->post_count;
 		// Cache the results for one hour
-		set_transient( $key, $count, HOUR_IN_SECONDS );
+		set_transient($key, $count, HOUR_IN_SECONDS);
 	}
 
 	return $count;
@@ -743,20 +743,20 @@ 
  *
  * @return bool true if complete, false otherwise
  */
-function give_is_payment_complete( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_is_payment_complete($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	$ret = false;
 
-	if ( $payment->ID > 0 ) {
+	if ($payment->ID > 0) {
 
-		if ( (int) $payment_id === (int) $payment->ID && 'publish' == $payment->status ) {
+		if ((int) $payment_id === (int) $payment->ID && 'publish' == $payment->status) {
 			$ret = true;
 		}
 
 	}
 
-	return apply_filters( 'give_is_payment_complete', $ret, $payment_id, $payment->post_status );
+	return apply_filters('give_is_payment_complete', $ret, $payment_id, $payment->post_status);
 }
 
 /**
@@ -780,29 +780,29 @@ 
  */
 function give_get_total_earnings() {
 
-	$total = get_option( 'give_earnings_total', false );
+	$total = get_option('give_earnings_total', false);
 
 	// If no total stored in DB, use old method of calculating total earnings
-	if ( false === $total ) {
+	if (false === $total) {
 
 		global $wpdb;
 
-		$total = get_transient( 'give_earnings_total' );
+		$total = get_transient('give_earnings_total');
 
-		if ( false === $total ) {
+		if (false === $total) {
 
 			$total = (float) 0;
 
-			$args = apply_filters( 'give_get_total_earnings_args', array(
+			$args = apply_filters('give_get_total_earnings_args', array(
 				'offset' => 0,
-				'number' => - 1,
-				'status' => array( 'publish', 'revoked' ),
+				'number' => -1,
+				'status' => array('publish', 'revoked'),
 				'fields' => 'ids'
-			) );
+			));
 
 
-			$payments = give_get_payments( $args );
-			if ( $payments ) {
+			$payments = give_get_payments($args);
+			if ($payments) {
 
 				/*
 				 * If performing a purchase, we need to skip the very last payment in the database, since it calls
@@ -810,30 +810,30 @@ 
 				 * first purchase
 				 */
 
-				if ( did_action( 'give_update_payment_status' ) ) {
-					array_pop( $payments );
+				if (did_action('give_update_payment_status')) {
+					array_pop($payments);
 				}
 
-				if ( ! empty( $payments ) ) {
-					$payments = implode( ',', $payments );
-					$total += $wpdb->get_var( "SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$payments})" );
+				if ( ! empty($payments)) {
+					$payments = implode(',', $payments);
+					$total += $wpdb->get_var("SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$payments})");
 				}
 
 			}
 
 			// Cache results for 1 day. This cache is cleared automatically when a payment is made
-			set_transient( 'give_earnings_total', $total, 86400 );
+			set_transient('give_earnings_total', $total, 86400);
 
 			// Store the total for the first time
-			update_option( 'give_earnings_total', $total );
+			update_option('give_earnings_total', $total);
 		}
 	}
 
-	if ( $total < 0 ) {
+	if ($total < 0) {
 		$total = 0; // Don't ever show negative earnings
 	}
 
-	return apply_filters( 'give_total_earnings', round( $total, give_currency_decimal_filter() ) );
+	return apply_filters('give_total_earnings', round($total, give_currency_decimal_filter()));
 }
 
 /**
@@ -845,10 +845,10 @@ 
  *
  * @return float $total Total earnings
  */
-function give_increase_total_earnings( $amount = 0 ) {
+function give_increase_total_earnings($amount = 0) {
 	$total = give_get_total_earnings();
 	$total += $amount;
-	update_option( 'give_earnings_total', $total );
+	update_option('give_earnings_total', $total);
 
 	return $total;
 }
@@ -862,13 +862,13 @@ 
  *
  * @return float $total Total earnings
  */
-function give_decrease_total_earnings( $amount = 0 ) {
+function give_decrease_total_earnings($amount = 0) {
 	$total = give_get_total_earnings();
 	$total -= $amount;
-	if ( $total < 0 ) {
+	if ($total < 0) {
 		$total = 0;
 	}
-	update_option( 'give_earnings_total', $total );
+	update_option('give_earnings_total', $total);
 
 	return $total;
 }
@@ -884,10 +884,10 @@ 
  *
  * @return mixed $meta Payment Meta
  */
-function give_get_payment_meta( $payment_id = 0, $meta_key = '_give_payment_meta', $single = true ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_meta($payment_id = 0, $meta_key = '_give_payment_meta', $single = true) {
+	$payment = new Give_Payment($payment_id);
 
-	return $payment->get_meta( $meta_key, $single );
+	return $payment->get_meta($meta_key, $single);
 }
 
 /**
@@ -900,10 +900,10 @@ 
  *
  * @return mixed               Meta ID if successful, false if unsuccessful
  */
-function give_update_payment_meta( $payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '' ) {
-	$payment = new Give_Payment( $payment_id );
+function give_update_payment_meta($payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '') {
+	$payment = new Give_Payment($payment_id);
 
-	return $payment->update_meta( $meta_key, $meta_value, $prev_value );
+	return $payment->update_meta($meta_key, $meta_value, $prev_value);
 }
 
 /**
@@ -915,8 +915,8 @@ 
  *
  * @return array $user_info User Info Meta Values
  */
-function give_get_payment_meta_user_info( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_meta_user_info($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->user_info;
 }
@@ -931,8 +931,8 @@ 
  *
  * @return int $form_id
  */
-function give_get_payment_form_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_form_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->form_id;
 }
@@ -946,8 +946,8 @@ 
  *
  * @return string $email User Email
  */
-function give_get_payment_user_email( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_email($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->email;
 }
@@ -961,11 +961,11 @@ 
  *
  * @return bool            If the payment is associated with a user (false) or not (true)
  */
-function give_is_guest_payment( $payment_id ) {
-	$payment_user_id  = give_get_payment_user_id( $payment_id );
-	$is_guest_payment = ! empty( $payment_user_id ) && $payment_user_id > 0 ? false : true;
+function give_is_guest_payment($payment_id) {
+	$payment_user_id  = give_get_payment_user_id($payment_id);
+	$is_guest_payment = ! empty($payment_user_id) && $payment_user_id > 0 ? false : true;
 
-	return (bool) apply_filters( 'give_is_guest_payment', $is_guest_payment, $payment_id );
+	return (bool) apply_filters('give_is_guest_payment', $is_guest_payment, $payment_id);
 }
 
 /**
@@ -977,8 +977,8 @@ 
  *
  * @return string $user_id User ID
  */
-function give_get_payment_user_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->user_id;
 }
@@ -992,8 +992,8 @@ 
  *
  * @return string $customer_id Customer ID
  */
-function give_get_payment_customer_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_customer_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->customer_id;
 }
@@ -1007,8 +1007,8 @@ 
  *
  * @return string $ip User IP
  */
-function give_get_payment_user_ip( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_ip($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->ip;
 }
@@ -1022,8 +1022,8 @@ 
  *
  * @return string $date The date the payment was completed
  */
-function give_get_payment_completed_date( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_completed_date($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->completed_date;
 }
@@ -1037,8 +1037,8 @@ 
  *
  * @return string $gateway Gateway
  */
-function give_get_payment_gateway( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_gateway($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->gateway;
 }
@@ -1052,8 +1052,8 @@ 
  *
  * @return string $currency The currency code
  */
-function give_get_payment_currency_code( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_currency_code($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->currency;
 }
@@ -1067,10 +1067,10 @@ 
  *
  * @return string $currency The currency name
  */
-function give_get_payment_currency( $payment_id = 0 ) {
-	$currency = give_get_payment_currency_code( $payment_id );
+function give_get_payment_currency($payment_id = 0) {
+	$currency = give_get_payment_currency_code($payment_id);
 
-	return apply_filters( 'give_payment_currency', give_get_currency_name( $currency ), $payment_id );
+	return apply_filters('give_payment_currency', give_get_currency_name($currency), $payment_id);
 }
 
 /**
@@ -1082,8 +1082,8 @@ 
  *
  * @return string $key Purchase key
  */
-function give_get_payment_key( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_key($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->key;
 }
@@ -1099,8 +1099,8 @@ 
  *
  * @return string $number Payment order number
  */
-function give_get_payment_number( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_number($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->number;
 }
@@ -1114,23 +1114,23 @@ 
  *
  * @return string      The formatted payment number
  */
-function give_format_payment_number( $number ) {
+function give_format_payment_number($number) {
 
-	if ( ! give_get_option( 'enable_sequential' ) ) {
+	if ( ! give_get_option('enable_sequential')) {
 		return $number;
 	}
 
-	if ( ! is_numeric( $number ) ) {
+	if ( ! is_numeric($number)) {
 		return $number;
 	}
 
-	$prefix  = give_get_option( 'sequential_prefix' );
-	$number  = absint( $number );
-	$postfix = give_get_option( 'sequential_postfix' );
+	$prefix  = give_get_option('sequential_prefix');
+	$number  = absint($number);
+	$postfix = give_get_option('sequential_postfix');
 
-	$formatted_number = $prefix . $number . $postfix;
+	$formatted_number = $prefix.$number.$postfix;
 
-	return apply_filters( 'give_format_payment_number', $formatted_number, $prefix, $number, $postfix );
+	return apply_filters('give_format_payment_number', $formatted_number, $prefix, $number, $postfix);
 }
 
 /**
@@ -1143,17 +1143,17 @@ 
  */
 function give_get_next_payment_number() {
 
-	if ( ! give_get_option( 'enable_sequential' ) ) {
+	if ( ! give_get_option('enable_sequential')) {
 		return false;
 	}
 
-	$number           = get_option( 'give_last_payment_number' );
-	$start            = give_get_option( 'sequential_start', 1 );
+	$number           = get_option('give_last_payment_number');
+	$start            = give_get_option('sequential_start', 1);
 	$increment_number = true;
 
-	if ( false !== $number ) {
+	if (false !== $number) {
 
-		if ( empty( $number ) ) {
+		if (empty($number)) {
 
 			$number           = $start;
 			$increment_number = false;
@@ -1163,24 +1163,24 @@ 
 	} else {
 
 		// This case handles the first addition of the new option, as well as if it get's deleted for any reason
-		$payments     = new Give_Payments_Query( array(
+		$payments = new Give_Payments_Query(array(
 			'number'  => 1,
 			'order'   => 'DESC',
 			'orderby' => 'ID',
 			'output'  => 'posts',
 			'fields'  => 'ids'
-		) );
+		));
 		$last_payment = $payments->get_payments();
 
-		if ( ! empty( $last_payment ) ) {
+		if ( ! empty($last_payment)) {
 
-			$number = give_get_payment_number( $last_payment[0] );
+			$number = give_get_payment_number($last_payment[0]);
 
 		}
 
-		if ( ! empty( $number ) && $number !== (int) $last_payment[0] ) {
+		if ( ! empty($number) && $number !== (int) $last_payment[0]) {
 
-			$number = give_remove_payment_prefix_postfix( $number );
+			$number = give_remove_payment_prefix_postfix($number);
 
 		} else {
 
@@ -1190,13 +1190,13 @@ 
 
 	}
 
-	$increment_number = apply_filters( 'give_increment_payment_number', $increment_number, $number );
+	$increment_number = apply_filters('give_increment_payment_number', $increment_number, $number);
 
-	if ( $increment_number ) {
-		$number ++;
+	if ($increment_number) {
+		$number++;
 	}
 
-	return apply_filters( 'give_get_next_payment_number', $number );
+	return apply_filters('give_get_next_payment_number', $number);
 }
 
 /**
@@ -1208,25 +1208,25 @@ 
  *
  * @return string          The new Payment number without prefix and postfix
  */
-function give_remove_payment_prefix_postfix( $number ) {
+function give_remove_payment_prefix_postfix($number) {
 
-	$prefix  = give_get_option( 'sequential_prefix' );
-	$postfix = give_get_option( 'sequential_postfix' );
+	$prefix  = give_get_option('sequential_prefix');
+	$postfix = give_get_option('sequential_postfix');
 
 	// Remove prefix
-	$number = preg_replace( '/' . $prefix . '/', '', $number, 1 );
+	$number = preg_replace('/'.$prefix.'/', '', $number, 1);
 
 	// Remove the postfix
-	$length      = strlen( $number );
-	$postfix_pos = strrpos( $number, $postfix );
-	if ( false !== $postfix_pos ) {
-		$number = substr_replace( $number, '', $postfix_pos, $length );
+	$length      = strlen($number);
+	$postfix_pos = strrpos($number, $postfix);
+	if (false !== $postfix_pos) {
+		$number = substr_replace($number, '', $postfix_pos, $length);
 	}
 
 	// Ensure it's a whole number
-	$number = intval( $number );
+	$number = intval($number);
 
-	return apply_filters( 'give_remove_payment_prefix_postfix', $number, $prefix, $postfix );
+	return apply_filters('give_remove_payment_prefix_postfix', $number, $prefix, $postfix);
 
 }
 
@@ -1242,10 +1242,10 @@ 
  *
  * @return string $amount Fully formatted payment amount
  */
-function give_payment_amount( $payment_id = 0 ) {
-	$amount = give_get_payment_amount( $payment_id );
+function give_payment_amount($payment_id = 0) {
+	$amount = give_get_payment_amount($payment_id);
 
-	return give_currency_filter( give_format_amount( $amount ), give_get_payment_currency_code( $payment_id ) );
+	return give_currency_filter(give_format_amount($amount), give_get_payment_currency_code($payment_id));
 }
 
 /**
@@ -1258,11 +1258,11 @@ 
  *
  * @return mixed|void
  */
-function give_get_payment_amount( $payment_id ) {
+function give_get_payment_amount($payment_id) {
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	return apply_filters( 'give_payment_amount', floatval( $payment->total ), $payment_id );
+	return apply_filters('give_payment_amount', floatval($payment->total), $payment_id);
 }
 
 /**
@@ -1278,10 +1278,10 @@ 
  *
  * @return array Fully formatted payment subtotal
  */
-function give_payment_subtotal( $payment_id = 0 ) {
-	$subtotal = give_get_payment_subtotal( $payment_id );
+function give_payment_subtotal($payment_id = 0) {
+	$subtotal = give_get_payment_subtotal($payment_id);
 
-	return give_currency_filter( give_format_amount( $subtotal ), give_get_payment_currency_code( $payment_id ) );
+	return give_currency_filter(give_format_amount($subtotal), give_get_payment_currency_code($payment_id));
 }
 
 /**
@@ -1295,8 +1295,8 @@ 
  *
  * @return float $subtotal Subtotal for payment (non formatted)
  */
-function give_get_payment_subtotal( $payment_id = 0 ) {
-	$payment = new G_Payment( $payment_id );
+function give_get_payment_subtotal($payment_id = 0) {
+	$payment = new G_Payment($payment_id);
 
 	return $payment->subtotal;
 }
@@ -1311,10 +1311,10 @@ 
  *
  * @return mixed array if payment fees found, false otherwise
  */
-function give_get_payment_fees( $payment_id = 0, $type = 'all' ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_fees($payment_id = 0, $type = 'all') {
+	$payment = new Give_Payment($payment_id);
 
-	return $payment->get_fees( $type );
+	return $payment->get_fees($type);
 }
 
 /**
@@ -1326,8 +1326,8 @@ 
  *
  * @return string The Transaction ID
  */
-function give_get_payment_transaction_id( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_transaction_id($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->transaction_id;
 }
@@ -1342,15 +1342,15 @@ 
  *
  * @return bool|mixed
  */
-function give_set_payment_transaction_id( $payment_id = 0, $transaction_id = '' ) {
+function give_set_payment_transaction_id($payment_id = 0, $transaction_id = '') {
 
-	if ( empty( $payment_id ) || empty( $transaction_id ) ) {
+	if (empty($payment_id) || empty($transaction_id)) {
 		return false;
 	}
 
-	$transaction_id = apply_filters( 'give_set_payment_transaction_id', $transaction_id, $payment_id );
+	$transaction_id = apply_filters('give_set_payment_transaction_id', $transaction_id, $payment_id);
 
-	return give_update_payment_meta( $payment_id, '_give_payment_transaction_id', $transaction_id );
+	return give_update_payment_meta($payment_id, '_give_payment_transaction_id', $transaction_id);
 }
 
 /**
@@ -1364,12 +1364,12 @@ 
  *
  * @return int $purchase Purchase ID
  */
-function give_get_purchase_id_by_key( $key ) {
+function give_get_purchase_id_by_key($key) {
 	global $wpdb;
 
-	$purchase = $wpdb->get_var( $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_purchase_key' AND meta_value = %s LIMIT 1", $key ) );
+	$purchase = $wpdb->get_var($wpdb->prepare("SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_purchase_key' AND meta_value = %s LIMIT 1", $key));
 
-	if ( $purchase != null ) {
+	if ($purchase != null) {
 		return $purchase;
 	}
 
@@ -1388,12 +1388,12 @@ 
  *
  * @return int $purchase Purchase ID
  */
-function give_get_purchase_id_by_transaction_id( $key ) {
+function give_get_purchase_id_by_transaction_id($key) {
 	global $wpdb;
 
-	$purchase = $wpdb->get_var( $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_transaction_id' AND meta_value = %s LIMIT 1", $key ) );
+	$purchase = $wpdb->get_var($wpdb->prepare("SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_transaction_id' AND meta_value = %s LIMIT 1", $key));
 
-	if ( $purchase != null ) {
+	if ($purchase != null) {
 		return $purchase;
 	}
 
@@ -1410,19 +1410,19 @@ 
  *
  * @return array $notes Payment Notes
  */
-function give_get_payment_notes( $payment_id = 0, $search = '' ) {
+function give_get_payment_notes($payment_id = 0, $search = '') {
 
-	if ( empty( $payment_id ) && empty( $search ) ) {
+	if (empty($payment_id) && empty($search)) {
 		return false;
 	}
 
-	remove_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
-	remove_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+	remove_action('pre_get_comments', 'give_hide_payment_notes', 10);
+	remove_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
-	$notes = get_comments( array( 'post_id' => $payment_id, 'order' => 'ASC', 'search' => $search ) );
+	$notes = get_comments(array('post_id' => $payment_id, 'order' => 'ASC', 'search' => $search));
 
-	add_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
-	add_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+	add_action('pre_get_comments', 'give_hide_payment_notes', 10);
+	add_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
 	return $notes;
 }
@@ -1438,19 +1438,19 @@ 
  *
  * @return int The new note ID
  */
-function give_insert_payment_note( $payment_id = 0, $note = '' ) {
-	if ( empty( $payment_id ) ) {
+function give_insert_payment_note($payment_id = 0, $note = '') {
+	if (empty($payment_id)) {
 		return false;
 	}
 
-	do_action( 'give_pre_insert_payment_note', $payment_id, $note );
+	do_action('give_pre_insert_payment_note', $payment_id, $note);
 
-	$note_id = wp_insert_comment( wp_filter_comment( array(
+	$note_id = wp_insert_comment(wp_filter_comment(array(
 		'comment_post_ID'      => $payment_id,
 		'comment_content'      => $note,
 		'user_id'              => is_admin() ? get_current_user_id() : 0,
-		'comment_date'         => current_time( 'mysql' ),
-		'comment_date_gmt'     => current_time( 'mysql', 1 ),
+		'comment_date'         => current_time('mysql'),
+		'comment_date_gmt'     => current_time('mysql', 1),
 		'comment_approved'     => 1,
 		'comment_parent'       => 0,
 		'comment_author'       => '',
@@ -1459,9 +1459,9 @@ 
 		'comment_author_email' => '',
 		'comment_type'         => 'give_payment_note'
 
-	) ) );
+	)));
 
-	do_action( 'give_insert_payment_note', $note_id, $payment_id, $note );
+	do_action('give_insert_payment_note', $note_id, $payment_id, $note);
 
 	return $note_id;
 }
@@ -1476,14 +1476,14 @@ 
  *
  * @return bool True on success, false otherwise
  */
-function give_delete_payment_note( $comment_id = 0, $payment_id = 0 ) {
-	if ( empty( $comment_id ) ) {
+function give_delete_payment_note($comment_id = 0, $payment_id = 0) {
+	if (empty($comment_id)) {
 		return false;
 	}
 
-	do_action( 'give_pre_delete_payment_note', $comment_id, $payment_id );
-	$ret = wp_delete_comment( $comment_id, true );
-	do_action( 'give_post_delete_payment_note', $comment_id, $payment_id );
+	do_action('give_pre_delete_payment_note', $comment_id, $payment_id);
+	$ret = wp_delete_comment($comment_id, true);
+	do_action('give_post_delete_payment_note', $comment_id, $payment_id);
 
 	return $ret;
 }
@@ -1498,32 +1498,32 @@ 
  *
  * @return string
  */
-function give_get_payment_note_html( $note, $payment_id = 0 ) {
+function give_get_payment_note_html($note, $payment_id = 0) {
 
-	if ( is_numeric( $note ) ) {
-		$note = get_comment( $note );
+	if (is_numeric($note)) {
+		$note = get_comment($note);
 	}
 
-	if ( ! empty( $note->user_id ) ) {
-		$user = get_userdata( $note->user_id );
+	if ( ! empty($note->user_id)) {
+		$user = get_userdata($note->user_id);
 		$user = $user->display_name;
 	} else {
-		$user = __( 'System', 'give' );
+		$user = __('System', 'give');
 	}
 
-	$date_format = get_option( 'date_format' ) . ', ' . get_option( 'time_format' );
+	$date_format = get_option('date_format').', '.get_option('time_format');
 
-	$delete_note_url = wp_nonce_url( add_query_arg( array(
+	$delete_note_url = wp_nonce_url(add_query_arg(array(
 		'give-action' => 'delete_payment_note',
 		'note_id'     => $note->comment_ID,
 		'payment_id'  => $payment_id
-	) ), 'give_delete_payment_note_' . $note->comment_ID );
+	)), 'give_delete_payment_note_'.$note->comment_ID);
 
-	$note_html = '<div class="give-payment-note" id="give-payment-note-' . $note->comment_ID . '">';
+	$note_html = '<div class="give-payment-note" id="give-payment-note-'.$note->comment_ID.'">';
 	$note_html .= '<p>';
-	$note_html .= '<strong>' . $user . '</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">' . date_i18n( $date_format, strtotime( $note->comment_date ) ) . '</span><br/>';
+	$note_html .= '<strong>'.$user.'</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">'.date_i18n($date_format, strtotime($note->comment_date)).'</span><br/>';
 	$note_html .= $note->comment_content;
-	$note_html .= '&nbsp;&ndash;&nbsp;<a href="' . esc_url( $delete_note_url ) . '" class="give-delete-payment-note" data-note-id="' . absint( $note->comment_ID ) . '" data-payment-id="' . absint( $payment_id ) . '" title="' . __( 'Delete this payment note', 'give' ) . '">' . __( 'Delete', 'give' ) . '</a>';
+	$note_html .= '&nbsp;&ndash;&nbsp;<a href="'.esc_url($delete_note_url).'" class="give-delete-payment-note" data-note-id="'.absint($note->comment_ID).'" data-payment-id="'.absint($payment_id).'" title="'.__('Delete this payment note', 'give').'">'.__('Delete', 'give').'</a>';
 	$note_html .= '</p>';
 	$note_html .= '</div>';
 
@@ -1541,20 +1541,20 @@ 
  *
  * @return void
  */
-function give_hide_payment_notes( $query ) {
+function give_hide_payment_notes($query) {
 	global $wp_version;
 
-	if ( version_compare( floatval( $wp_version ), '4.1', '>=' ) ) {
-		$types = isset( $query->query_vars['type__not_in'] ) ? $query->query_vars['type__not_in'] : array();
-		if ( ! is_array( $types ) ) {
-			$types = array( $types );
+	if (version_compare(floatval($wp_version), '4.1', '>=')) {
+		$types = isset($query->query_vars['type__not_in']) ? $query->query_vars['type__not_in'] : array();
+		if ( ! is_array($types)) {
+			$types = array($types);
 		}
 		$types[]                           = 'give_payment_note';
 		$query->query_vars['type__not_in'] = $types;
 	}
 }
 
-add_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
+add_action('pre_get_comments', 'give_hide_payment_notes', 10);
 
 /**
  * Exclude notes (comments) on give_payment post type from showing in Recent Comments widgets
@@ -1566,17 +1566,17 @@ 
  *
  * @return array $clauses Updated comment clauses
  */
-function give_hide_payment_notes_pre_41( $clauses, $wp_comment_query ) {
+function give_hide_payment_notes_pre_41($clauses, $wp_comment_query) {
 	global $wpdb, $wp_version;
 
-	if ( version_compare( floatval( $wp_version ), '4.1', '<' ) ) {
+	if (version_compare(floatval($wp_version), '4.1', '<')) {
 		$clauses['where'] .= ' AND comment_type != "give_payment_note"';
 	}
 
 	return $clauses;
 }
 
-add_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+add_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
 
 /**
@@ -1589,15 +1589,15 @@ 
  *
  * @return array $where
  */
-function give_hide_payment_notes_from_feeds( $where, $wp_comment_query ) {
+function give_hide_payment_notes_from_feeds($where, $wp_comment_query) {
 	global $wpdb;
 
-	$where .= $wpdb->prepare( " AND comment_type != %s", 'give_payment_note' );
+	$where .= $wpdb->prepare(" AND comment_type != %s", 'give_payment_note');
 
 	return $where;
 }
 
-add_filter( 'comment_feed_where', 'give_hide_payment_notes_from_feeds', 10, 2 );
+add_filter('comment_feed_where', 'give_hide_payment_notes_from_feeds', 10, 2);
 
 
 /**
@@ -1611,32 +1611,32 @@ 
  *
  * @return array Array of comment counts
  */
-function give_remove_payment_notes_in_comment_counts( $stats, $post_id ) {
+function give_remove_payment_notes_in_comment_counts($stats, $post_id) {
 	global $wpdb, $pagenow;
 
-	if ( 'index.php' != $pagenow ) {
+	if ('index.php' != $pagenow) {
 		return $stats;
 	}
 
 	$post_id = (int) $post_id;
 
-	if ( apply_filters( 'give_count_payment_notes_in_comments', false ) ) {
+	if (apply_filters('give_count_payment_notes_in_comments', false)) {
 		return $stats;
 	}
 
-	$stats = wp_cache_get( "comments-{$post_id}", 'counts' );
+	$stats = wp_cache_get("comments-{$post_id}", 'counts');
 
-	if ( false !== $stats ) {
+	if (false !== $stats) {
 		return $stats;
 	}
 
 	$where = 'WHERE comment_type != "give_payment_note"';
 
-	if ( $post_id > 0 ) {
-		$where .= $wpdb->prepare( " AND comment_post_ID = %d", $post_id );
+	if ($post_id > 0) {
+		$where .= $wpdb->prepare(" AND comment_post_ID = %d", $post_id);
 	}
 
-	$count = $wpdb->get_results( "SELECT comment_approved, COUNT( * ) AS num_comments FROM {$wpdb->comments} {$where} GROUP BY comment_approved", ARRAY_A );
+	$count = $wpdb->get_results("SELECT comment_approved, COUNT( * ) AS num_comments FROM {$wpdb->comments} {$where} GROUP BY comment_approved", ARRAY_A);
 
 	$total    = 0;
 	$approved = array(
@@ -1646,30 +1646,30 @@ 
 		'trash'        => 'trash',
 		'post-trashed' => 'post-trashed'
 	);
-	foreach ( (array) $count as $row ) {
+	foreach ((array) $count as $row) {
 		// Don't count post-trashed toward totals
-		if ( 'post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved'] ) {
+		if ('post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved']) {
 			$total += $row['num_comments'];
 		}
-		if ( isset( $approved[ $row['comment_approved'] ] ) ) {
-			$stats[ $approved[ $row['comment_approved'] ] ] = $row['num_comments'];
+		if (isset($approved[$row['comment_approved']])) {
+			$stats[$approved[$row['comment_approved']]] = $row['num_comments'];
 		}
 	}
 
 	$stats['total_comments'] = $total;
-	foreach ( $approved as $key ) {
-		if ( empty( $stats[ $key ] ) ) {
-			$stats[ $key ] = 0;
+	foreach ($approved as $key) {
+		if (empty($stats[$key])) {
+			$stats[$key] = 0;
 		}
 	}
 
 	$stats = (object) $stats;
-	wp_cache_set( "comments-{$post_id}", $stats, 'counts' );
+	wp_cache_set("comments-{$post_id}", $stats, 'counts');
 
 	return $stats;
 }
 
-add_filter( 'wp_count_comments', 'give_remove_payment_notes_in_comment_counts', 10, 2 );
+add_filter('wp_count_comments', 'give_remove_payment_notes_in_comment_counts', 10, 2);
 
 
 /**
@@ -1682,9 +1682,9 @@ 
  *
  * @return string $where Modified where clause
  */
-function give_filter_where_older_than_week( $where = '' ) {
+function give_filter_where_older_than_week($where = '') {
 	// Payments older than one week
-	$start = date( 'Y-m-d', strtotime( '-7 days' ) );
+	$start = date('Y-m-d', strtotime('-7 days'));
 	$where .= " AND post_date <= '{$start}'";
 
 	return $where;
@@ -1703,37 +1703,37 @@ 
  *
  * @return string $form_title Returns the full title if $level_title false, otherwise returns the levels title
  */
-function give_get_payment_form_title( $payment_meta, $level_title = false, $separator = '' ) {
+function give_get_payment_form_title($payment_meta, $level_title = false, $separator = '') {
 
-	$form_id    = isset( $payment_meta['form_id'] ) ? $payment_meta['form_id'] : 0;
-	$form_title = isset( $payment_meta['form_title'] ) ? $payment_meta['form_title'] : '';
-	$price_id   = isset( $payment_meta['price_id'] ) ? $payment_meta['price_id'] : null;
+	$form_id    = isset($payment_meta['form_id']) ? $payment_meta['form_id'] : 0;
+	$form_title = isset($payment_meta['form_title']) ? $payment_meta['form_title'] : '';
+	$price_id   = isset($payment_meta['price_id']) ? $payment_meta['price_id'] : null;
 
-	if ( $level_title == true ) {
+	if ($level_title == true) {
 		$form_title = '';
 	}
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		if ( ! empty( $separator ) ) {
-			$form_title .= ' ' . $separator;
+		if ( ! empty($separator)) {
+			$form_title .= ' '.$separator;
 		}
 		$form_title .= ' <span class="donation-level-text-wrap">';
 
-		if ( $price_id == 'custom' ) {
+		if ($price_id == 'custom') {
 
-			$custom_amount_text = get_post_meta( $form_id, '_give_custom_amount_text', true );
-			$form_title .= ! empty( $custom_amount_text ) ? $custom_amount_text : __( 'Custom Amount', 'give' );
+			$custom_amount_text = get_post_meta($form_id, '_give_custom_amount_text', true);
+			$form_title .= ! empty($custom_amount_text) ? $custom_amount_text : __('Custom Amount', 'give');
 
 		} else {
-			$form_title .= give_get_price_option_name( $form_id, $price_id );
+			$form_title .= give_get_price_option_name($form_id, $price_id);
 		}
 
 		$form_title .= '</span>';
 
 	}
 
-	return apply_filters( 'give_get_payment_form_title', $form_title, $payment_meta );
+	return apply_filters('give_get_payment_form_title', $form_title, $payment_meta);
 
 }
 
@@ -1747,20 +1747,20 @@ 
  *
  * @return string $price_id
  */
-function give_get_price_id( $form_id, $price ) {
+function give_get_price_id($form_id, $price) {
 
 	$price_id = 0;
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		$levels = maybe_unserialize( get_post_meta( $form_id, '_give_donation_levels', true ) );
+		$levels = maybe_unserialize(get_post_meta($form_id, '_give_donation_levels', true));
 
-		foreach ( $levels as $level ) {
+		foreach ($levels as $level) {
 
-			$level_amount = (float) give_sanitize_amount( $level['_give_amount'] );
+			$level_amount = (float) give_sanitize_amount($level['_give_amount']);
 
 			//check that this indeed the recurring price
-			if ( $level_amount == $price ) {
+			if ($level_amount == $price) {
 
 				$price_id = $level['_give_id']['level_id'];
 

includes/payments/class-payments-query.php

Spacing +112 added lines, -112 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -54,10 +54,10 @@ 
 	 *
 	 * @param $args array The array of arguments that can be passed in and used for setting up this payment query.
 	 */
-	public function __construct( $args = array() ) {
+	public function __construct($args = array()) {
 		$defaults = array(
 			'output'          => 'payments', // Use 'posts' to get standard post objects
-			'post_type'       => array( 'give_payment' ),
+			'post_type'       => array('give_payment'),
 			'start_date'      => false,
 			'end_date'        => false,
 			'number'          => 20,
@@ -77,7 +77,7 @@ 
 			'give_forms'      => null
 		);
 
-		$this->args = wp_parse_args( $args, $defaults );
+		$this->args = wp_parse_args($args, $defaults);
 
 		$this->init();
 	}
@@ -88,11 +88,11 @@ 
 	 * @access public
 	 * @since  1.0
 	 */
-	public function __set( $query_var, $value ) {
-		if ( in_array( $query_var, array( 'meta_query', 'tax_query' ) ) ) {
-			$this->args[ $query_var ][] = $value;
+	public function __set($query_var, $value) {
+		if (in_array($query_var, array('meta_query', 'tax_query'))) {
+			$this->args[$query_var][] = $value;
 		} else {
-			$this->args[ $query_var ] = $value;
+			$this->args[$query_var] = $value;
 		}
 	}
 
@@ -102,8 +102,8 @@ 
 	 * @access public
 	 * @since  1.0
 	 */
-	public function __unset( $query_var ) {
-		unset( $this->args[ $query_var ] );
+	public function __unset($query_var) {
+		unset($this->args[$query_var]);
 	}
 
 	/**
@@ -115,19 +115,19 @@ 
 	 */
 	public function init() {
 
-		add_action( 'give_pre_get_payments', array( $this, 'date_filter_pre' ) );
-		add_action( 'give_post_get_payments', array( $this, 'date_filter_post' ) );
-
-		add_action( 'give_pre_get_payments', array( $this, 'orderby' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'status' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'month' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'per_page' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'page' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'user' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'search' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'mode' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'children' ) );
-		add_action( 'give_pre_get_payments', array( $this, 'give_forms' ) );
+		add_action('give_pre_get_payments', array($this, 'date_filter_pre'));
+		add_action('give_post_get_payments', array($this, 'date_filter_post'));
+
+		add_action('give_pre_get_payments', array($this, 'orderby'));
+		add_action('give_pre_get_payments', array($this, 'status'));
+		add_action('give_pre_get_payments', array($this, 'month'));
+		add_action('give_pre_get_payments', array($this, 'per_page'));
+		add_action('give_pre_get_payments', array($this, 'page'));
+		add_action('give_pre_get_payments', array($this, 'user'));
+		add_action('give_pre_get_payments', array($this, 'search'));
+		add_action('give_pre_get_payments', array($this, 'mode'));
+		add_action('give_pre_get_payments', array($this, 'children'));
+		add_action('give_pre_get_payments', array($this, 'give_forms'));
 	}
 
 	/**
@@ -143,33 +143,33 @@ 
 	 */
 	public function get_payments() {
 
-		do_action( 'give_pre_get_payments', $this );
+		do_action('give_pre_get_payments', $this);
 
-		$query = new WP_Query( $this->args );
+		$query = new WP_Query($this->args);
 
 		$custom_output = array(
 			'payments',
 			'give_payments',
 		);
 
-		if ( ! in_array( $this->args['output'], $custom_output ) ) {
+		if ( ! in_array($this->args['output'], $custom_output)) {
 			return $query->posts;
 		}
 
-		if ( $query->have_posts() ) {
-			while ( $query->have_posts() ) {
+		if ($query->have_posts()) {
+			while ($query->have_posts()) {
 				$query->the_post();
 
 				$payment_id = get_post()->ID;
-				$payment    = new Give_Payment( $payment_id );
+				$payment    = new Give_Payment($payment_id);
 
-				$this->payments[] = apply_filters( 'give_payment', $payment, $payment_id, $this );
+				$this->payments[] = apply_filters('give_payment', $payment, $payment_id, $this);
 			}
 
 			wp_reset_postdata();
 		}
 
-		do_action( 'give_post_get_payments', $this );
+		do_action('give_post_get_payments', $this);
 
 		return $this->payments;
 	}
@@ -182,13 +182,13 @@ 
 	 * @return void
 	 */
 	public function date_filter_pre() {
-		if ( ! ( $this->args['start_date'] || $this->args['end_date'] ) ) {
+		if ( ! ($this->args['start_date'] || $this->args['end_date'])) {
 			return;
 		}
 
-		$this->setup_dates( $this->args['start_date'], $this->args['end_date'] );
+		$this->setup_dates($this->args['start_date'], $this->args['end_date']);
 
-		add_filter( 'posts_where', array( $this, 'payments_where' ) );
+		add_filter('posts_where', array($this, 'payments_where'));
 	}
 
 	/**
@@ -200,11 +200,11 @@ 
 	 * @return void
 	 */
 	public function date_filter_post() {
-		if ( ! ( $this->args['start_date'] || $this->args['end_date'] ) ) {
+		if ( ! ($this->args['start_date'] || $this->args['end_date'])) {
 			return;
 		}
 
-		remove_filter( 'posts_where', array( $this, 'payments_where' ) );
+		remove_filter('posts_where', array($this, 'payments_where'));
 	}
 
 	/**
@@ -215,12 +215,12 @@ 
 	 * @return void
 	 */
 	public function status() {
-		if ( ! isset ( $this->args['status'] ) ) {
+		if ( ! isset ($this->args['status'])) {
 			return;
 		}
 
-		$this->__set( 'post_status', $this->args['status'] );
-		$this->__unset( 'status' );
+		$this->__set('post_status', $this->args['status']);
+		$this->__unset('status');
 	}
 
 	/**
@@ -231,12 +231,12 @@ 
 	 * @return void
 	 */
 	public function page() {
-		if ( ! isset ( $this->args['page'] ) ) {
+		if ( ! isset ($this->args['page'])) {
 			return;
 		}
 
-		$this->__set( 'paged', $this->args['page'] );
-		$this->__unset( 'page' );
+		$this->__set('paged', $this->args['page']);
+		$this->__unset('page');
 	}
 
 	/**
@@ -248,17 +248,17 @@ 
 	 */
 	public function per_page() {
 
-		if ( ! isset( $this->args['number'] ) ) {
+		if ( ! isset($this->args['number'])) {
 			return;
 		}
 
-		if ( $this->args['number'] == - 1 ) {
-			$this->__set( 'nopaging', true );
+		if ($this->args['number'] == - 1) {
+			$this->__set('nopaging', true);
 		} else {
-			$this->__set( 'posts_per_page', $this->args['number'] );
+			$this->__set('posts_per_page', $this->args['number']);
 		}
 
-		$this->__unset( 'number' );
+		$this->__unset('number');
 	}
 
 	/**
@@ -269,12 +269,12 @@ 
 	 * @return void
 	 */
 	public function month() {
-		if ( ! isset ( $this->args['month'] ) ) {
+		if ( ! isset ($this->args['month'])) {
 			return;
 		}
 
-		$this->__set( 'monthnum', $this->args['month'] );
-		$this->__unset( 'month' );
+		$this->__set('monthnum', $this->args['month']);
+		$this->__unset('month');
 	}
 
 	/**
@@ -285,13 +285,13 @@ 
 	 * @return void
 	 */
 	public function orderby() {
-		switch ( $this->args['orderby'] ) {
+		switch ($this->args['orderby']) {
 			case 'amount' :
-				$this->__set( 'orderby', 'meta_value_num' );
-				$this->__set( 'meta_key', '_give_payment_total' );
+				$this->__set('orderby', 'meta_value_num');
+				$this->__set('meta_key', '_give_payment_total');
 				break;
 			default :
-				$this->__set( 'orderby', $this->args['orderby'] );
+				$this->__set('orderby', $this->args['orderby']);
 				break;
 		}
 	}
@@ -304,20 +304,20 @@ 
 	 * @return void
 	 */
 	public function user() {
-		if ( is_null( $this->args['user'] ) ) {
+		if (is_null($this->args['user'])) {
 			return;
 		}
 
-		if ( is_numeric( $this->args['user'] ) ) {
+		if (is_numeric($this->args['user'])) {
 			$user_key = '_give_payment_user_id';
 		} else {
 			$user_key = '_give_payment_user_email';
 		}
 
-		$this->__set( 'meta_query', array(
+		$this->__set('meta_query', array(
 			'key'   => $user_key,
 			'value' => $this->args['user']
-		) );
+		));
 	}
 
 	/**
@@ -329,33 +329,33 @@ 
 	 */
 	public function search() {
 
-		if ( ! isset( $this->args['s'] ) ) {
+		if ( ! isset($this->args['s'])) {
 			return;
 		}
 
-		$search = trim( $this->args['s'] );
+		$search = trim($this->args['s']);
 
-		if ( empty( $search ) ) {
+		if (empty($search)) {
 			return;
 		}
 
-		$is_email = is_email( $search ) || strpos( $search, '@' ) !== false;
-		$is_user  = strpos( $search, strtolower( 'user:' ) ) !== false;
+		$is_email = is_email($search) || strpos($search, '@') !== false;
+		$is_user  = strpos($search, strtolower('user:')) !== false;
 
-		if ( ! empty( $this->args['search_in_notes'] ) ) {
+		if ( ! empty($this->args['search_in_notes'])) {
 
-			$notes = give_get_payment_notes( 0, $search );
+			$notes = give_get_payment_notes(0, $search);
 
-			if ( ! empty( $notes ) ) {
+			if ( ! empty($notes)) {
 
-				$payment_ids = wp_list_pluck( (array) $notes, 'comment_post_ID' );
+				$payment_ids = wp_list_pluck((array) $notes, 'comment_post_ID');
 
-				$this->__set( 'post__in', $payment_ids );
+				$this->__set('post__in', $payment_ids);
 			}
 
-			$this->__unset( 's' );
+			$this->__unset('s');
 
-		} elseif ( $is_email || strlen( $search ) == 32 ) {
+		} elseif ($is_email || strlen($search) == 32) {
 
 			$key         = $is_email ? '_give_payment_user_email' : '_give_payment_purchase_key';
 			$search_meta = array(
@@ -364,19 +364,19 @@ 
 				'compare' => 'LIKE'
 			);
 
-			$this->__set( 'meta_query', $search_meta );
-			$this->__unset( 's' );
+			$this->__set('meta_query', $search_meta);
+			$this->__unset('s');
 
-		} elseif ( $is_user ) {
+		} elseif ($is_user) {
 
 			$search_meta = array(
 				'key'   => '_give_payment_user_id',
-				'value' => trim( str_replace( 'user:', '', strtolower( $search ) ) )
+				'value' => trim(str_replace('user:', '', strtolower($search)))
 			);
 
-			$this->__set( 'meta_query', $search_meta );
+			$this->__set('meta_query', $search_meta);
 
-			if ( give_get_option( 'enable_sequential' ) ) {
+			if (give_get_option('enable_sequential')) {
 
 				$search_meta = array(
 					'key'     => '_give_payment_number',
@@ -384,19 +384,19 @@ 
 					'compare' => 'LIKE'
 				);
 
-				$this->__set( 'meta_query', $search_meta );
+				$this->__set('meta_query', $search_meta);
 
 				$this->args['meta_query']['relation'] = 'OR';
 
 			}
 
-			$this->__unset( 's' );
+			$this->__unset('s');
 
 		} elseif (
-			give_get_option( 'enable_sequential' ) &&
+			give_get_option('enable_sequential') &&
 			(
-				false !== strpos( $search, give_get_option( 'sequential_prefix' ) ) ||
-				false !== strpos( $search, give_get_option( 'sequential_postfix' ) )
+				false !== strpos($search, give_get_option('sequential_prefix')) ||
+				false !== strpos($search, give_get_option('sequential_postfix'))
 			)
 		) {
 
@@ -406,30 +406,30 @@ 
 				'compare' => 'LIKE'
 			);
 
-			$this->__set( 'meta_query', $search_meta );
-			$this->__unset( 's' );
+			$this->__set('meta_query', $search_meta);
+			$this->__unset('s');
 
-		} elseif ( is_numeric( $search ) ) {
+		} elseif (is_numeric($search)) {
 
-			$post = get_post( $search );
+			$post = get_post($search);
 
-			if ( is_object( $post ) && $post->post_type == 'give_payment' ) {
+			if (is_object($post) && $post->post_type == 'give_payment') {
 
 				$arr   = array();
 				$arr[] = $search;
-				$this->__set( 'post__in', $arr );
-				$this->__unset( 's' );
+				$this->__set('post__in', $arr);
+				$this->__unset('s');
 			}
 
-		} elseif ( '#' == substr( $search, 0, 1 ) ) {
+		} elseif ('#' == substr($search, 0, 1)) {
 
-			$search = str_replace( '#:', '', $search );
-			$search = str_replace( '#', '', $search );
-			$this->__set( 'give_forms', $search );
-			$this->__unset( 's' );
+			$search = str_replace('#:', '', $search);
+			$search = str_replace('#', '', $search);
+			$this->__set('give_forms', $search);
+			$this->__unset('s');
 
 		} else {
-			$this->__set( 's', $search );
+			$this->__set('s', $search);
 
 		}
 
@@ -443,16 +443,16 @@ 
 	 * @return void
 	 */
 	public function mode() {
-		if ( empty( $this->args['mode'] ) || $this->args['mode'] == 'all' ) {
-			$this->__unset( 'mode' );
+		if (empty($this->args['mode']) || $this->args['mode'] == 'all') {
+			$this->__unset('mode');
 
 			return;
 		}
 
-		$this->__set( 'meta_query', array(
+		$this->__set('meta_query', array(
 			'key'   => '_give_payment_mode',
 			'value' => $this->args['mode']
-		) );
+		));
 	}
 
 	/**
@@ -463,10 +463,10 @@ 
 	 * @return void
 	 */
 	public function children() {
-		if ( empty( $this->args['children'] ) ) {
-			$this->__set( 'post_parent', 0 );
+		if (empty($this->args['children'])) {
+			$this->__set('post_parent', 0);
 		}
-		$this->__unset( 'children' );
+		$this->__unset('children');
 	}
 
 	/**
@@ -478,7 +478,7 @@ 
 	 */
 	public function give_forms() {
 
-		if ( empty( $this->args['give_forms'] ) ) {
+		if (empty($this->args['give_forms'])) {
 			return;
 		}
 
@@ -487,7 +487,7 @@ 
 		$args = array(
 			'post_parent'            => $this->args['give_forms'],
 			'log_type'               => 'sale',
-			'post_status'            => array( 'publish' ),
+			'post_status'            => array('publish'),
 			'nopaging'               => true,
 			'no_found_rows'          => true,
 			'update_post_term_cache' => false,
@@ -496,31 +496,31 @@ 
 			'fields'                 => 'ids'
 		);
 
-		if ( is_array( $this->args['give_forms'] ) ) {
-			unset( $args['post_parent'] );
+		if (is_array($this->args['give_forms'])) {
+			unset($args['post_parent']);
 			$args['post_parent__in'] = $this->args['give_forms'];
 		}
 
-		$sales = $give_logs->get_connected_logs( $args );
+		$sales = $give_logs->get_connected_logs($args);
 
-		if ( ! empty( $sales ) ) {
+		if ( ! empty($sales)) {
 
 			$payments = array();
 
-			foreach ( $sales as $sale ) {
-				$payments[] = get_post_meta( $sale, '_give_log_payment_id', true );
+			foreach ($sales as $sale) {
+				$payments[] = get_post_meta($sale, '_give_log_payment_id', true);
 			}
 
-			$this->__set( 'post__in', $payments );
+			$this->__set('post__in', $payments);
 
 		} else {
 
 			// Set post_parent to something crazy so it doesn't find anything
-			$this->__set( 'post_parent', 999999999999999 );
+			$this->__set('post_parent', 999999999999999);
 
 		}
 
-		$this->__unset( 'give_forms' );
+		$this->__unset('give_forms');
 
 	}
 }