WordImpress / Give

includes/class-give-db-donors.php

Doc Comments +2 added lines, -2 removed lines

@@ -207,7 +207,7 @@ 
 	 * NOTE: This should not be called directly as it does not make necessary changes to
 	 * the payment meta and logs. Use give_donor_delete() instead.
 	 *
-	 * @param  bool|string|int $_id_or_email ID or Email of Donor.
+	 * @param  integer $_id_or_email ID or Email of Donor.
 	 *
 	 * @since  1.0
 	 * @access public
@@ -407,7 +407,7 @@ 
 	 * @access public
 	 *
 	 * @param  string $field ID or email. Default is 'id'.
-	 * @param  mixed  $value The Customer ID or email to search. Default is 0.
+	 * @param  integer  $value The Customer ID or email to search. Default is 0.
 	 *
 	 * @return mixed         Upon success, an object of the donor. Upon failure, NULL
 	 */

Spacing +107 added lines, -107 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -42,7 +42,7 @@ 
 		$this->bc_200_params();
 
 		// Set hooks and register table only if instance loading first time.
-		if ( ! ( Give()->donors instanceof Give_DB_Donors ) ) {
+		if ( ! (Give()->donors instanceof Give_DB_Donors)) {
 			// Install table.
 			$this->register_table();
 		}
@@ -91,7 +91,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'),
 			'token'           => '',
 			'verify_key'      => '',
 			'verify_throttle' => '',
@@ -108,40 +108,40 @@ 
 	 *
 	 * @return int|bool
 	 */
-	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'])));
 		}
 
-		$donor = $this->get_donor_by( 'email', $args['email'] );
+		$donor = $this->get_donor_by('email', $args['email']);
 
 		// update an existing donor.
-		if ( $donor ) {
+		if ($donor) {
 
 			// Update the payment IDs attached to the donor
-			if ( ! empty( $args['payment_ids'] ) ) {
+			if ( ! empty($args['payment_ids'])) {
 
-				if ( empty( $donor->payment_ids ) ) {
+				if (empty($donor->payment_ids)) {
 
 					$donor->payment_ids = $args['payment_ids'];
 
 				} else {
 
-					$existing_ids       = array_map( 'absint', explode( ',', $donor->payment_ids ) );
-					$payment_ids        = array_map( 'absint', explode( ',', $args['payment_ids'] ) );
-					$payment_ids        = array_merge( $payment_ids, $existing_ids );
-					$donor->payment_ids = implode( ',', array_unique( array_values( $payment_ids ) ) );
+					$existing_ids       = array_map('absint', explode(',', $donor->payment_ids));
+					$payment_ids        = array_map('absint', explode(',', $args['payment_ids']));
+					$payment_ids        = array_merge($payment_ids, $existing_ids);
+					$donor->payment_ids = implode(',', array_unique(array_values($payment_ids)));
 
 				}
 
@@ -149,13 +149,13 @@ 
 
 			}
 
-			$this->update( $donor->id, $args );
+			$this->update($donor->id, $args);
 
 			return $donor->id;
 
 		} else {
 
-			return $this->insert( $args, 'donor' );
+			return $this->insert($args, 'donor');
 
 		}
 
@@ -172,12 +172,12 @@ 
 	 *
 	 * @return bool
 	 */
-	public function update( $row_id, $data = array(), $where = '' ) {
+	public function update($row_id, $data = array(), $where = '') {
 
-		$status = parent::update( $row_id, $data, $where );
+		$status = parent::update($row_id, $data, $where);
 
-		if ( $status ) {
-			Give_Cache::delete_group( $row_id, 'give-donors' );
+		if ($status) {
+			Give_Cache::delete_group($row_id, 'give-donors');
 		}
 
 		return $status;
@@ -191,11 +191,11 @@ 
 	 *
 	 * @return int
 	 */
-	public function insert( $data, $type = '' ) {
-		$donor_id = parent::insert( $data, $type );
+	public function insert($data, $type = '') {
+		$donor_id = parent::insert($data, $type);
 
-		if ( $donor_id ) {
-			Give_Cache::delete_group( $donor_id, 'give-donors' );
+		if ($donor_id) {
+			Give_Cache::delete_group($donor_id, 'give-donors');
 		}
 
 		return $donor_id;
@@ -214,16 +214,16 @@ 
 	 *
 	 * @return bool|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';
-		$donor  = $this->get_donor_by( $column, $_id_or_email );
+		$column = is_email($_id_or_email) ? 'email' : 'id';
+		$donor  = $this->get_donor_by($column, $_id_or_email);
 
-		if ( $donor->id > 0 ) {
+		if ($donor->id > 0) {
 
 			global $wpdb;
 
@@ -232,11 +232,11 @@ 
 			 *
 			 * @since 1.8.14
 			 */
-			Give()->donor_meta->delete_all_meta( $donor->id );
+			Give()->donor_meta->delete_all_meta($donor->id);
 
 			// Cache already deleted in delete_all_meta fn.
 
-			return $wpdb->delete( $this->table_name, array( 'id' => $donor->id ), array( '%d' ) );
+			return $wpdb->delete($this->table_name, array('id' => $donor->id), array('%d'));
 
 		} else {
 			return false;
@@ -257,10 +257,10 @@ 
 	 *
 	 * @return bool|int
 	 */
-	public function delete_by_user_id( $user_id = false ) {
+	public function delete_by_user_id($user_id = false) {
 		global $wpdb;
 
-		if ( empty( $user_id ) ) {
+		if (empty($user_id)) {
 			return false;
 		}
 
@@ -269,14 +269,14 @@ 
 		 *
 		 * @since 1.8.14
 		 */
-		$donor = new Give_Donor( $user_id, true );
-		if ( ! empty( $donor->id ) ) {
-			Give()->donor_meta->delete_all_meta( $donor->id );
+		$donor = new Give_Donor($user_id, true);
+		if ( ! empty($donor->id)) {
+			Give()->donor_meta->delete_all_meta($donor->id);
 		}
 
 		// Cache is already deleted in delete_all_meta fn.
 
-		return $wpdb->delete( $this->table_name, array( 'user_id' => $user_id ), array( '%d' ) );
+		return $wpdb->delete($this->table_name, array('user_id' => $user_id), array('%d'));
 	}
 
 	/**
@@ -290,14 +290,14 @@ 
 	 *
 	 * @return bool          True is exists, false otherwise.
 	 */
-	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);
 
 	}
 
@@ -312,16 +312,16 @@ 
 	 *
 	 * @return bool
 	 */
-	public function attach_payment( $donor_id = 0, $payment_id = 0 ) {
+	public function attach_payment($donor_id = 0, $payment_id = 0) {
 
-		$donor = new Give_Donor( $donor_id );
+		$donor = new Give_Donor($donor_id);
 
-		if ( empty( $donor->id ) ) {
+		if (empty($donor->id)) {
 			return false;
 		}
 
 		// Attach the payment, but don't increment stats, as this function previously did not
-		return $donor->attach_payment( $payment_id, false );
+		return $donor->attach_payment($payment_id, false);
 
 	}
 
@@ -336,16 +336,16 @@ 
 	 *
 	 * @return bool
 	 */
-	public function remove_payment( $donor_id = 0, $payment_id = 0 ) {
+	public function remove_payment($donor_id = 0, $payment_id = 0) {
 
-		$donor = new Give_Donor( $donor_id );
+		$donor = new Give_Donor($donor_id);
 
-		if ( ! $donor ) {
+		if ( ! $donor) {
 			return false;
 		}
 
 		// Remove the payment, but don't decrease stats, as this function previously did not
-		return $donor->remove_payment( $payment_id, false );
+		return $donor->remove_payment($payment_id, false);
 
 	}
 
@@ -359,18 +359,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function increment_stats( $donor_id = 0, $amount = 0.00 ) {
+	public function increment_stats($donor_id = 0, $amount = 0.00) {
 
-		$donor = new Give_Donor( $donor_id );
+		$donor = new Give_Donor($donor_id);
 
-		if ( empty( $donor->id ) ) {
+		if (empty($donor->id)) {
 			return false;
 		}
 
 		$increased_count = $donor->increase_purchase_count();
-		$increased_value = $donor->increase_value( $amount );
+		$increased_value = $donor->increase_value($amount);
 
-		return ( $increased_count && $increased_value ) ? true : false;
+		return ($increased_count && $increased_value) ? true : false;
 
 	}
 
@@ -385,18 +385,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function decrement_stats( $donor_id = 0, $amount = 0.00 ) {
+	public function decrement_stats($donor_id = 0, $amount = 0.00) {
 
-		$donor = new Give_Donor( $donor_id );
+		$donor = new Give_Donor($donor_id);
 
-		if ( ! $donor ) {
+		if ( ! $donor) {
 			return false;
 		}
 
 		$decreased_count = $donor->decrease_donation_count();
-		$decreased_value = $donor->decrease_value( $amount );
+		$decreased_value = $donor->decrease_value($amount);
 
-		return ( $decreased_count && $decreased_value ) ? true : false;
+		return ($decreased_count && $decreased_value) ? true : false;
 
 	}
 
@@ -411,44 +411,44 @@ 
 	 *
 	 * @return mixed         Upon success, an object of the donor. Upon failure, NULL
 	 */
-	public function get_donor_by( $field = 'id', $value = 0 ) {
-		$value = sanitize_text_field( $value );
+	public function get_donor_by($field = 'id', $value = 0) {
+		$value = sanitize_text_field($value);
 
 		// Bailout.
-		if ( empty( $field ) || empty( $value ) ) {
+		if (empty($field) || empty($value)) {
 			return null;
 		}
 
 		// Verify values.
-		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 = absint( $value );
+			$value = absint($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);
 		}
 
 		// Bailout
-		if ( ! $value ) {
+		if ( ! $value) {
 			return false;
 		}
 
 		// Set query params.
-		switch ( $field ) {
+		switch ($field) {
 			case 'id':
 				$args['donor'] = $value;
 				break;
@@ -463,9 +463,9 @@ 
 		}
 
 		// Get donors.
-		$donor = new Give_Donors_Query( $args );
+		$donor = new Give_Donors_Query($args);
 
-		if ( ! $donor = $donor->get_donors() ) {
+		if ( ! $donor = $donor->get_donors()) {
 			// Look for donor from an additional email.
 			$args = array(
 				'meta_query' => array(
@@ -476,15 +476,15 @@ 
 				),
 			);
 
-			$donor = new Give_Donors_Query( $args );
+			$donor = new Give_Donors_Query($args);
 			$donor = $donor->get_donors();
 
-			if ( empty( $donor ) ) {
+			if (empty($donor)) {
 				return false;
 			}
 		}
 
-		return current( $donor );
+		return current($donor);
 	}
 
 	/**
@@ -497,10 +497,10 @@ 
 	 *
 	 * @return array|object|null Donors array or object. Null if not found.
 	 */
-	public function get_donors( $args = array() ) {
-		$this->bc_1814_params( $args );
+	public function get_donors($args = array()) {
+		$this->bc_1814_params($args);
 
-		$donors = new Give_Donors_Query( $args );
+		$donors = new Give_Donors_Query($args);
 
 		return $donors->get_donors();
 
@@ -517,21 +517,21 @@ 
 	 *
 	 * @return int         Total number of donors.
 	 */
-	public function count( $args = array() ) {
-		$this->bc_1814_params( $args );
+	public function count($args = array()) {
+		$this->bc_1814_params($args);
 		$args['count'] = true;
 
-		$cache_key = md5( 'give_donors_count' . serialize( $args ) );
-		$count     = Give_Cache::get_group( $cache_key, 'donors' );
+		$cache_key = md5('give_donors_count'.serialize($args));
+		$count     = Give_Cache::get_group($cache_key, 'donors');
 
-		if ( is_null( $count ) ) {
-			$donors = new Give_Donors_Query( $args );
+		if (is_null($count)) {
+			$donors = new Give_Donors_Query($args);
 			$count  = $donors->get_donors();
 
-			Give_Cache::set_group( $cache_key, $count, 'donors', 3600 );
+			Give_Cache::set_group($cache_key, $count, 'donors', 3600);
 		}
 
-		return absint( $count );
+		return absint($count);
 
 	}
 
@@ -545,9 +545,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,
@@ -565,9 +565,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);
 	}
 
 	/**
@@ -582,8 +582,8 @@ 
 		global $wpdb;
 
 		if (
-			! give_has_upgrade_completed( 'v20_rename_donor_tables' ) &&
-			$wpdb->query( $wpdb->prepare( "SHOW TABLES LIKE %s", "{$wpdb->prefix}give_customers" ) )
+			! give_has_upgrade_completed('v20_rename_donor_tables') &&
+			$wpdb->query($wpdb->prepare("SHOW TABLES LIKE %s", "{$wpdb->prefix}give_customers"))
 		) {
 			$wpdb->donors = $this->table_name = "{$wpdb->prefix}give_customers";
 		}
@@ -597,41 +597,41 @@ 
 	 *
 	 * @param $args
 	 */
-	private function bc_1814_params( &$args ) {
+	private function bc_1814_params(&$args) {
 		// Backward compatibility: user_id
-		if ( ! empty( $args['user_id'] ) ) {
+		if ( ! empty($args['user_id'])) {
 			$args['user'] = $args['user_id'];
 		}
 
 		// Backward compatibility: id
-		if ( ! empty( $args['id'] ) ) {
+		if ( ! empty($args['id'])) {
 			$args['donor'] = $args['id'];
 		}
 
 		// Backward compatibility: name
-		if ( ! empty( $args['name'] ) ) {
+		if ( ! empty($args['name'])) {
 			$args['s'] = "name:{$args['name']}";
 		}
 
 		// Backward compatibility: date
 		// Donors 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'] ) ) {
-					$args['date_query']['after'] = date( 'Y-m-d H:i:s', strtotime( $args['date']['start'] ) );
+				if ( ! empty($args['date']['start'])) {
+					$args['date_query']['after'] = date('Y-m-d H:i:s', strtotime($args['date']['start']));
 				}
 
-				if ( ! empty( $args['date']['end'] ) ) {
-					$args['date_query']['before'] = date( 'Y-m-d H:i:s', strtotime( $args['date']['end'] ) );
+				if ( ! empty($args['date']['end'])) {
+					$args['date_query']['before'] = date('Y-m-d H:i:s', strtotime($args['date']['end']));
 				}
 
 			} else {
 
-				$args['date_query']['year']  = date( 'Y', strtotime( $args['date'] ) );
-				$args['date_query']['month'] = date( 'm', strtotime( $args['date'] ) );
-				$args['date_query']['day']   = date( 'd', strtotime( $args['date'] ) );
+				$args['date_query']['year']  = date('Y', strtotime($args['date']));
+				$args['date_query']['month'] = date('m', strtotime($args['date']));
+				$args['date_query']['day']   = date('d', strtotime($args['date']));
 			}
 		}
 	}

includes/class-give-logging.php

Doc Comments +1 added lines, -1 removed lines

@@ -512,7 +512,7 @@
 	 * @since  1.7
 	 * @access public
 	 *
-	 * @return bool
+	 * @return false|null
 	 */
 	public function delete_cache() {
 		// Add log related keys to delete.

Spacing +137 added lines, -137 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -53,8 +53,8 @@ 
 		 * Setup properties
 		 */
 
-		require_once GIVE_PLUGIN_DIR . 'includes/class-give-db-logs.php';
-		require_once GIVE_PLUGIN_DIR . 'includes/class-give-db-logs-meta.php';
+		require_once GIVE_PLUGIN_DIR.'includes/class-give-db-logs.php';
+		require_once GIVE_PLUGIN_DIR.'includes/class-give-db-logs-meta.php';
 		$this->log_db     = new Give_DB_Logs();
 		$this->logmeta_db = new Give_DB_Log_Meta();
 
@@ -62,22 +62,22 @@ 
 		 * Setup hooks.
 		 */
 
-		add_action( 'save_post_give_payment', array( $this, 'background_process_delete_cache' ) );
-		add_action( 'save_post_give_forms', array( $this, 'background_process_delete_cache' ) );
-		add_action( 'save_post_give_log', array( $this, 'background_process_delete_cache' ) );
-		add_action( 'give_delete_log_cache', array( $this, 'delete_cache' ) );
-		add_action( 'update_log_metadata', array( $this, 'bc_200_set_payment_as_log_parent' ), 10, 4 );
+		add_action('save_post_give_payment', array($this, 'background_process_delete_cache'));
+		add_action('save_post_give_forms', array($this, 'background_process_delete_cache'));
+		add_action('save_post_give_log', array($this, 'background_process_delete_cache'));
+		add_action('give_delete_log_cache', array($this, 'delete_cache'));
+		add_action('update_log_metadata', array($this, 'bc_200_set_payment_as_log_parent'), 10, 4);
 
 		// Backward compatibility.
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Create the log post type
-			add_action( 'init', array( $this, 'register_post_type' ), -2 );
+			add_action('init', array($this, 'register_post_type'), -2);
 		}
 
 		// Create types taxonomy and default types
 		// @todo: remove this taxonomy, some addon use this taxonomy with there custom log post type for example: recurring
 		// Do not use this taxonomy with your log type because we will remove it in future releases.
-		add_action( 'init', array( $this, 'register_taxonomy' ), -2 );
+		add_action('init', array($this, 'register_taxonomy'), -2);
 	}
 
 
@@ -95,7 +95,7 @@ 
 		/* Logs post type */
 		$log_args = array(
 			'labels'              => array(
-				'name' => esc_html__( 'Logs', 'give' ),
+				'name' => esc_html__('Logs', 'give'),
 			),
 			'public'              => false,
 			'exclude_from_search' => true,
@@ -104,11 +104,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);
 	}
 
 	/**
@@ -122,9 +122,9 @@ 
 	 * @return void
 	 */
 	public function register_taxonomy() {
-		register_taxonomy( 'give_log_type', 'give_log', array(
+		register_taxonomy('give_log_type', 'give_log', array(
 			'public' => false,
-		) );
+		));
 	}
 
 	/**
@@ -145,7 +145,7 @@ 
 			'update',
 		);
 
-		return apply_filters( 'give_log_types', $terms );
+		return apply_filters('give_log_types', $terms);
 	}
 
 	/**
@@ -160,8 +160,8 @@ 
 	 *
 	 * @return bool         Whether log type is valid.
 	 */
-	public function valid_type( $type ) {
-		return in_array( $type, $this->log_types() );
+	public function valid_type($type) {
+		return in_array($type, $this->log_types());
 	}
 
 	/**
@@ -180,7 +180,7 @@ 
 	 *
 	 * @return int             Log ID.
 	 */
-	public function add( $title = '', $message = '', $parent = 0, $type = '' ) {
+	public function add($title = '', $message = '', $parent = 0, $type = '') {
 		$log_data = array(
 			'post_title'   => $title,
 			'post_content' => $message,
@@ -188,7 +188,7 @@ 
 			'log_type'     => $type,
 		);
 
-		return $this->insert_log( $log_data );
+		return $this->insert_log($log_data);
 	}
 
 	/**
@@ -205,12 +205,12 @@ 
 	 *
 	 * @return array             An array of the connected logs.
 	 */
-	public function get_logs( $object_id = 0, $type = '', $paged = null ) {
-		return $this->get_connected_logs( array(
+	public function get_logs($object_id = 0, $type = '', $paged = null) {
+		return $this->get_connected_logs(array(
 			'log_parent' => $object_id,
 			'paged'      => $paged,
 			'log_type'   => $type,
-		) );
+		));
 	}
 
 	/**
@@ -224,7 +224,7 @@ 
 	 *
 	 * @return int             The ID of the newly created log item.
 	 */
-	public function insert_log( $log_data = array(), $log_meta = array() ) {
+	public function insert_log($log_data = array(), $log_meta = array()) {
 		$log_id = 0;
 
 		$defaults = array(
@@ -237,28 +237,28 @@ 
 			'post_status' => 'publish',
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
-		$this->bc_200_validate_params( $args, $log_meta );
+		$args = wp_parse_args($log_data, $defaults);
+		$this->bc_200_validate_params($args, $log_meta);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			global $wpdb;
 
 			// Backward Compatibility.
-			if ( ! $wpdb->get_var( "SELECT ID from {$this->log_db->table_name} ORDER BY id DESC LIMIT 1" ) ) {
-				$latest_log_id = $wpdb->get_var( "SELECT ID from $wpdb->posts ORDER BY id DESC LIMIT 1" );
-				$latest_log_id = empty( $latest_log_id ) ? 1 : ++ $latest_log_id;
+			if ( ! $wpdb->get_var("SELECT ID from {$this->log_db->table_name} ORDER BY id DESC LIMIT 1")) {
+				$latest_log_id = $wpdb->get_var("SELECT ID from $wpdb->posts ORDER BY id DESC LIMIT 1");
+				$latest_log_id = empty($latest_log_id) ? 1 : ++ $latest_log_id;
 
 				$args['ID'] = $latest_log_id;
-				$this->log_db->insert( $args );
+				$this->log_db->insert($args);
 			}
 		}
 
-		$log_id = $this->log_db->add( $args );
+		$log_id = $this->log_db->add($args);
 
 		// Set log meta, if any
-		if ( $log_id && ! empty( $log_meta ) ) {
-			foreach ( (array) $log_meta as $key => $meta ) {
-				$this->logmeta_db->update_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				$this->logmeta_db->update_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 			}
 		}
 
@@ -280,7 +280,7 @@ 
 	 *
 	 * @return bool|null       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()) {
 		$log_id = 0;
 
 		/**
@@ -291,7 +291,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry meta.
 		 */
-		do_action( 'give_pre_update_log', $log_data, $log_meta );
+		do_action('give_pre_update_log', $log_data, $log_meta);
 
 		$defaults = array(
 			'log_parent'  => 0,
@@ -301,28 +301,28 @@ 
 			'post_status' => 'publish',
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
-		$this->bc_200_validate_params( $args, $log_meta );
+		$args = wp_parse_args($log_data, $defaults);
+		$this->bc_200_validate_params($args, $log_meta);
 
 		// Store the log entry
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Backward compatibility.
-			$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 ) ) {
-						give_update_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)) {
+						give_update_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 					}
 				}
 			}
 		} else {
-			$log_id = $this->log_db->add( $args );
+			$log_id = $this->log_db->add($args);
 
-			if ( $log_id && ! empty( $log_meta ) ) {
-				foreach ( (array) $log_meta as $key => $meta ) {
-					if ( ! empty( $meta ) ) {
-						$this->logmeta_db->update_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)) {
+						$this->logmeta_db->update_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 					}
 				}
 			}
@@ -337,7 +337,7 @@ 
 		 * @param array $log_data Log entry data.
 		 * @param array $log_meta Log entry 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);
 	}
 
 	/**
@@ -354,30 +354,30 @@ 
 	 *
 	 * @return array|false Array if logs were found, false otherwise.
 	 */
-	public function get_connected_logs( $args = array() ) {
+	public function get_connected_logs($args = array()) {
 		$logs = array();
 
-		$defaults   = array(
+		$defaults = array(
 			'number'      => 20,
-			'paged'       => get_query_var( 'paged' ),
+			'paged'       => get_query_var('paged'),
 			'log_type'    => false,
 
 			// Backward compatibility.
 			'post_type'   => 'give_log',
 			'post_status' => 'publish',
 		);
-		$query_args = wp_parse_args( $args, $defaults );
-		$this->bc_200_validate_params( $query_args );
+		$query_args = wp_parse_args($args, $defaults);
+		$this->bc_200_validate_params($query_args);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Backward compatibility.
-			$logs = get_posts( $query_args );
-			$this->bc_200_add_new_properties( $logs );
+			$logs = get_posts($query_args);
+			$this->bc_200_add_new_properties($logs);
 		} else {
-			$logs = $this->log_db->get_logs( $query_args );
+			$logs = $this->log_db->get_logs($query_args);
 		}
 
-		return ( ! empty( $logs ) ? $logs : false );
+		return ( ! empty($logs) ? $logs : false);
 	}
 
 	/**
@@ -395,41 +395,41 @@ 
 	 *
 	 * @return int                Log count.
 	 */
-	public function get_log_count( $object_id = 0, $type = '', $meta_query = null, $date_query = null ) {
+	public function get_log_count($object_id = 0, $type = '', $meta_query = null, $date_query = null) {
 		$logs_count = 0;
 
 		$query_args = array(
-			'number'      => - 1,
+			'number'      => -1,
 
 			// Backward comatibility.
 			'post_type'   => 'give_log',
 			'post_status' => 'publish',
 		);
 
-		if ( $object_id ) {
+		if ($object_id) {
 			$query_args['log_parent'] = $object_id;
 		}
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['log_type'] = $type;
 		}
 
-		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;
 		}
 
-		$this->bc_200_validate_params( $query_args );
+		$this->bc_200_validate_params($query_args);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Backward compatibility.
-			$logs       = new WP_Query( $query_args );
+			$logs       = new WP_Query($query_args);
 			$logs_count = (int) $logs->post_count;
 		} else {
-			$logs_count = $this->log_db->count( $query_args );
+			$logs_count = $this->log_db->count($query_args);
 		}
 
 		return $logs_count;
@@ -449,10 +449,10 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_logs( $object_id = 0, $type = '', $meta_query = null ) {
+	public function delete_logs($object_id = 0, $type = '', $meta_query = null) {
 		$query_args = array(
 			'log_parent'  => $object_id,
-			'number'      => - 1,
+			'number'      => -1,
 			'fields'      => 'ID',
 
 			// Backward compatibility.
@@ -460,32 +460,32 @@ 
 			'post_status' => 'publish',
 		);
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['log_type'] = $type;
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		$this->bc_200_validate_params( $query_args );
+		$this->bc_200_validate_params($query_args);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Backward compatibility.
-			$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);
 				}
 			}
 		} else {
-			$logs = $this->log_db->get_logs( $query_args );
+			$logs = $this->log_db->get_logs($query_args);
 
-			if ( $logs ) {
-				foreach ( $logs as $log ) {
-					if ( $this->log_db->delete( $log->ID ) ) {
-						$this->logmeta_db->delete_row( $log->ID );
+			if ($logs) {
+				foreach ($logs as $log) {
+					if ($this->log_db->delete($log->ID)) {
+						$this->logmeta_db->delete_row($log->ID);
 					}
 				}
 			}
@@ -502,9 +502,9 @@ 
 	 *
 	 * @param int $post_id
 	 */
-	public function background_process_delete_cache( $post_id ) {
+	public function background_process_delete_cache($post_id) {
 		// Delete log cache immediately
-		wp_schedule_single_event( time() - 5, 'give_delete_log_cache' );
+		wp_schedule_single_event(time() - 5, 'give_delete_log_cache');
 	}
 
 	/**
@@ -517,17 +517,17 @@ 
 	 */
 	public function delete_cache() {
 		// Add log related keys to delete.
-		$cache_give_logs      = Give_Cache::get_options_like( 'give_logs' );
-		$cache_give_log_count = Give_Cache::get_options_like( 'log_count' );
+		$cache_give_logs      = Give_Cache::get_options_like('give_logs');
+		$cache_give_log_count = Give_Cache::get_options_like('log_count');
 
-		$cache_option_names = array_merge( $cache_give_logs, $cache_give_log_count );
+		$cache_option_names = array_merge($cache_give_logs, $cache_give_log_count);
 
 		// Bailout.
-		if ( empty( $cache_option_names ) ) {
+		if (empty($cache_option_names)) {
 			return false;
 		}
 
-		Give_Cache::delete( $cache_option_names );
+		Give_Cache::delete($cache_option_names);
 	}
 
 	/**
@@ -539,7 +539,7 @@ 
 	 * @param array $log_query
 	 * @param array $log_meta
 	 */
-	private function bc_200_validate_params( &$log_query, &$log_meta = array() ) {
+	private function bc_200_validate_params(&$log_query, &$log_meta = array()) {
 		$query_params = array(
 			'log_title'    => 'post_title',
 			'log_parent'   => 'post_parent',
@@ -551,41 +551,41 @@ 
 			'meta_query'   => 'meta_query',
 		);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
 			// Set old params.
-			foreach ( $query_params as $new_query_param => $old_query_param ) {
+			foreach ($query_params as $new_query_param => $old_query_param) {
 
-				if ( isset( $log_query[ $old_query_param ] ) && empty( $log_query[ $new_query_param ] ) ) {
-					$log_query[ $new_query_param ] = $log_query[ $old_query_param ];
+				if (isset($log_query[$old_query_param]) && empty($log_query[$new_query_param])) {
+					$log_query[$new_query_param] = $log_query[$old_query_param];
 					continue;
-				} elseif ( ! isset( $log_query[ $new_query_param ] ) ) {
+				} elseif ( ! isset($log_query[$new_query_param])) {
 					continue;
-				} elseif( empty( $log_query[ $new_query_param ] ) ) {
+				} elseif (empty($log_query[$new_query_param])) {
 					continue;
 				}
 
-				switch ( $new_query_param ) {
+				switch ($new_query_param) {
 					case 'log_type':
 						$log_query['tax_query'] = array(
 							array(
 								'taxonomy' => 'give_log_type',
 								'field'    => 'slug',
-								'terms'    => $log_query[ $new_query_param ],
+								'terms'    => $log_query[$new_query_param],
 							),
 						);
 						break;
 
 					case 'meta_query':
-						if( ! empty( $log_query['meta_query'] ) && empty( $log_query['post_parent'] ) ) {
-							foreach ( $log_query['meta_query'] as $index => $meta_query ){
-								if( ! is_array( $meta_query ) || empty( $meta_query['key'] ) ) {
+						if ( ! empty($log_query['meta_query']) && empty($log_query['post_parent'])) {
+							foreach ($log_query['meta_query'] as $index => $meta_query) {
+								if ( ! is_array($meta_query) || empty($meta_query['key'])) {
 									continue;
 								}
 
-								switch ( $meta_query['key'] ) {
+								switch ($meta_query['key']) {
 									case '_give_log_form_id':
 										$log_query['post_parent'] = $meta_query['value'];
-										unset( $log_query['meta_query'][$index] );
+										unset($log_query['meta_query'][$index]);
 										break;
 								}
 							}
@@ -593,40 +593,40 @@ 
 						break;
 
 					default:
-						switch( $new_query_param ){
+						switch ($new_query_param) {
 							case 'log_parent':
 								$log_query['meta_query'][] = array(
 									'key' => '_give_log_payment_id',
-									'value' => $log_query[ $new_query_param ]
+									'value' => $log_query[$new_query_param]
 								);
 
 								break;
 
 							default:
-								$log_query[ $old_query_param ] = $log_query[ $new_query_param ];
+								$log_query[$old_query_param] = $log_query[$new_query_param];
 						}
 				}
 			}
 		} else {
 			// Set only old params.
-			$query_params = array_flip( $query_params );
-			foreach ( $query_params as $old_query_param => $new_query_param ) {
-				if ( isset( $log_query[ $new_query_param ] ) && empty( $log_query[ $old_query_param ] ) ) {
-					$log_query[ $old_query_param ] = $log_query[ $new_query_param ];
+			$query_params = array_flip($query_params);
+			foreach ($query_params as $old_query_param => $new_query_param) {
+				if (isset($log_query[$new_query_param]) && empty($log_query[$old_query_param])) {
+					$log_query[$old_query_param] = $log_query[$new_query_param];
 					continue;
-				} elseif ( ! isset( $log_query[ $old_query_param ] ) ) {
+				} elseif ( ! isset($log_query[$old_query_param])) {
 					continue;
 				}
 
-				switch ( $old_query_param ) {
+				switch ($old_query_param) {
 					case 'tax_query':
-						if ( isset( $log_query[ $old_query_param ][0]['terms'] ) ) {
-							$log_query[ $new_query_param ] = $log_query[ $old_query_param ][0]['terms'];
+						if (isset($log_query[$old_query_param][0]['terms'])) {
+							$log_query[$new_query_param] = $log_query[$old_query_param][0]['terms'];
 						}
 						break;
 
 					default:
-						$log_query[ $new_query_param ] = $log_query[ $old_query_param ];
+						$log_query[$new_query_param] = $log_query[$old_query_param];
 				}
 			}
 		}
@@ -640,8 +640,8 @@ 
 	 *
 	 * @param  array $logs
 	 */
-	private function bc_200_add_new_properties( &$logs ) {
-		if ( empty( $logs ) ) {
+	private function bc_200_add_new_properties(&$logs) {
+		if (empty($logs)) {
 			return;
 		}
 
@@ -654,30 +654,30 @@ 
 			'log_type'     => 'give_log_type',
 		);
 
-		if ( ! give_has_upgrade_completed( 'v20_logs_upgrades' ) ) {
-			foreach ( $logs as $index => $log ) {
-				foreach ( $query_params as $new_query_param => $old_query_param ) {
-					if ( ! property_exists( $log, $old_query_param ) ) {
+		if ( ! give_has_upgrade_completed('v20_logs_upgrades')) {
+			foreach ($logs as $index => $log) {
+				foreach ($query_params as $new_query_param => $old_query_param) {
+					if ( ! property_exists($log, $old_query_param)) {
 						/**
 						 *  Set unmatched properties.
 						 */
 
 						// 1. log_type
-						$term = get_the_terms( $log->ID, 'give_log_type' );
-						$term = ! is_wp_error( $term ) && ! empty( $term ) ? $term[0] : array();
+						$term = get_the_terms($log->ID, 'give_log_type');
+						$term = ! is_wp_error($term) && ! empty($term) ? $term[0] : array();
 
-						$logs[ $index ]->{$new_query_param} = ! empty( $term ) ? $term->slug : '';
+						$logs[$index]->{$new_query_param} = ! empty($term) ? $term->slug : '';
 
 						continue;
 					}
 
-					switch ( $old_query_param ) {
+					switch ($old_query_param) {
 						case 'post_parent':
-							$logs[ $index ]->{$new_query_param} = give_get_meta( $log->ID, '_give_log_payment_id', true );
+							$logs[$index]->{$new_query_param} = give_get_meta($log->ID, '_give_log_payment_id', true);
 							break;
 
 						default:
-							$logs[ $index ]->{$new_query_param} = $log->{$old_query_param};
+							$logs[$index]->{$new_query_param} = $log->{$old_query_param};
 					}
 				}
 			}
@@ -697,10 +697,10 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function bc_200_set_payment_as_log_parent( $check, $log_id, $meta_key, $meta_value ) {
+	public function bc_200_set_payment_as_log_parent($check, $log_id, $meta_key, $meta_value) {
 		global $wpdb;
 		$update_status = false;
-		$post_type     = get_post_type( $log_id );
+		$post_type     = get_post_type($log_id);
 
 		// Bailout.
 		if (
@@ -720,9 +720,9 @@ 
 			)
 		);
 
-		if ( $form_id ) {
-			$this->logmeta_db->delete_meta( $log_id, '_give_log_payment_id' );
-			$this->logmeta_db->update_meta( $log_id, '_give_log_form_id', $form_id );
+		if ($form_id) {
+			$this->logmeta_db->delete_meta($log_id, '_give_log_payment_id');
+			$this->logmeta_db->update_meta($log_id, '_give_log_form_id', $form_id);
 
 			$update_status = $wpdb->update(
 				$this->log_db->table_name,

includes/install.php

Doc Comments +1 added lines, -1 removed lines

@@ -397,7 +397,7 @@
  *
  * @since 1.8.11
  *
- * @return void
+ * @return false|null
  */
 function give_create_pages() {
 

Spacing +60 added lines, -60 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -28,15 +28,15 @@ 
  * @global     $wpdb
  * @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();
 
@@ -63,31 +63,31 @@ 
 	give_setup_post_types();
 
 	// 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();
 
 	//Fresh Install? Setup Test Mode, Base Country (US), Test Gateway, Currency.
-	if ( empty( $current_version ) ) {
-		$options = array_merge( $options, give_get_default_settings() );
+	if (empty($current_version)) {
+		$options = array_merge($options, give_get_default_settings());
 	}
 
 	// Populate the default values.
-	update_option( 'give_settings', array_merge( $give_options, $options ) );
+	update_option('give_settings', array_merge($give_options, $options));
 
 	/**
 	 * Run plugin upgrades.
 	 *
 	 * @since 1.8
 	 */
-	do_action( 'give_upgrades' );
+	do_action('give_upgrades');
 
-	if ( GIVE_VERSION !== get_option( 'give_version' ) ) {
-		update_option( 'give_version', GIVE_VERSION );
+	if (GIVE_VERSION !== get_option('give_version')) {
+		update_option('give_version', GIVE_VERSION);
 	}
 
 	// Create Give roles.
@@ -98,7 +98,7 @@ 
 	// Set api version, end point and refresh permalink.
 	$api = new Give_API();
 	$api->add_endpoint();
-	update_option( 'give_default_api_version', 'v' . $api->get_version() );
+	update_option('give_default_api_version', 'v'.$api->get_version());
 
 	flush_rewrite_rules();
 
@@ -113,11 +113,11 @@ 
 	$give_sessions->use_php_sessions();
 
 	// Add a temporary option to note that Give pages have been created.
-	Give_Cache::set( '_give_installed', $options, 30, true );
+	Give_Cache::set('_give_installed', $options, 30, true);
 
-	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(
@@ -149,18 +149,18 @@ 
 			'v210_verify_form_status_upgrades',
 		);
 
-		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.
-	Give_Cache::set( '_give_activation_redirect', true, 30, true );
+	Give_Cache::set('_give_activation_redirect', true, 30, true);
 }
 
 /**
@@ -178,11 +178,11 @@ 
  * @param  int    $site_id The Site ID.
  * @param  array  $meta    Blog Meta.
  */
-function give_on_create_blog( $blog_id, $user_id, $domain, $path, $site_id, $meta ) {
+function give_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();
 
@@ -190,7 +190,7 @@ 
 
 }
 
-add_action( 'wpmu_new_blog', 'give_on_create_blog', 10, 6 );
+add_action('wpmu_new_blog', 'give_on_create_blog', 10, 6);
 
 
 /**
@@ -203,13 +203,13 @@ 
  *
  * @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);
 	$donors_db     = new Give_DB_Donors();
 	$donor_meta_db = new Give_DB_Donor_Meta();
 
-	if ( $donors_db->installed() ) {
+	if ($donors_db->installed()) {
 		$tables[] = $donors_db->table_name;
 		$tables[] = $donor_meta_db->table_name;
 	}
@@ -219,7 +219,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
@@ -231,16 +231,16 @@ 
  */
 function give_after_install() {
 
-	if ( ! is_admin() ) {
+	if ( ! is_admin()) {
 		return;
 	}
 
-	$give_options     = Give_Cache::get( '_give_installed', true );
-	$give_table_check = get_option( '_give_table_check', false );
+	$give_options     = Give_Cache::get('_give_installed', true);
+	$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()->donor_meta->installed() ) {
+		if ( ! @Give()->donor_meta->installed()) {
 
 			// Create the donor meta database.
 			// (this ensures it creates it on multisite instances where it is network activated).
@@ -248,7 +248,7 @@ 
 
 		}
 
-		if ( ! @Give()->donors->installed() ) {
+		if ( ! @Give()->donors->installed()) {
 			// Create the donor database.
 			// (this ensures it creates it on multisite instances where it is network activated).
 			@Give()->donors->create_table();
@@ -260,22 +260,22 @@ 
 			 *
 			 * @param array $give_options Give plugin options.
 			 */
-			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 ) {
-		Give_Cache::delete( Give_Cache::get_key( '_give_installed' ) );
+	if (false !== $give_options) {
+		Give_Cache::delete(Give_Cache::get_key('_give_installed'));
 	}
 
 
 }
 
-add_action( 'admin_init', 'give_after_install' );
+add_action('admin_init', 'give_after_install');
 
 
 /**
@@ -290,11 +290,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();
@@ -305,7 +305,7 @@ 
 
 }
 
-add_action( 'admin_init', 'give_install_roles_on_network' );
+add_action('admin_init', 'give_install_roles_on_network');
 
 /**
  * Default core setting values.
@@ -346,14 +346,14 @@ 
 		'uninstall_on_delete'                         => 'disabled',
 		'the_content_filter'                          => 'enabled',
 		'scripts_footer'                              => 'disabled',
-		'agree_to_terms_label'                        => __( 'Agree to Terms?', 'give' ),
+		'agree_to_terms_label'                        => __('Agree to Terms?', 'give'),
 		'agreement_text'                              => give_get_default_agreement_text(),
 
 		// Paypal IPN verification.
 		'paypal_verification'                         => 'enabled',
 
 		// Default is manual gateway.
-		'gateways'                                    => array( 'manual' => 1, 'offline' => 1 ),
+		'gateways'                                    => array('manual' => 1, 'offline' => 1),
 		'default_gateway'                             => 'manual',
 
 		// Offline gateway setup.
@@ -381,7 +381,7 @@ 
  */
 function give_get_default_agreement_text() {
 
-	$org_name = get_bloginfo( 'name' );
+	$org_name = get_bloginfo('name');
 
 	$agreement = sprintf(
 		'<p>Acceptance of any contribution, gift or grant is at the discretion of the %1$s. The  %1$s will not accept any gift unless it can be used or expended consistently with the purpose and mission of the  %1$s.</p>
@@ -395,7 +395,7 @@ 
 		$org_name
 	);
 
-	return apply_filters( 'give_get_default_agreement_text', $agreement, $org_name );
+	return apply_filters('give_get_default_agreement_text', $agreement, $org_name);
 }
 
 
@@ -409,19 +409,19 @@ 
 function give_create_pages() {
 
 	// Bailout if pages already created.
-	if ( get_option( 'give_install_pages_created' ) ) {
+	if (get_option('give_install_pages_created')) {
 		return false;
 	}
 
 	$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'))) {
 
 		// Donation Confirmation (Success) Page
 		$success = wp_insert_post(
 			array(
-				'post_title'     => esc_html__( 'Donation Confirmation', 'give' ),
+				'post_title'     => esc_html__('Donation Confirmation', 'give'),
 				'post_content'   => '[give_receipt]',
 				'post_status'    => 'publish',
 				'post_author'    => 1,
@@ -435,13 +435,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 Donation Page
 		$failed = wp_insert_post(
 			array(
-				'post_title'     => esc_html__( 'Donation Failed', 'give' ),
-				'post_content'   => esc_html__( 'We\'re sorry, your donation failed to process. Please try again or contact site support.', 'give' ),
+				'post_title'     => esc_html__('Donation Failed', 'give'),
+				'post_content'   => esc_html__('We\'re sorry, your donation failed to process. Please try again or contact site support.', 'give'),
 				'post_status'    => 'publish',
 				'post_author'    => 1,
 				'post_type'      => 'page',
@@ -453,11 +453,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'))) {
 		// Donation History Page
 		$history = wp_insert_post(
 			array(
-				'post_title'     => esc_html__( 'Donation History', 'give' ),
+				'post_title'     => esc_html__('Donation History', 'give'),
 				'post_content'   => '[donation_history]',
 				'post_status'    => 'publish',
 				'post_author'    => 1,
@@ -469,11 +469,11 @@ 
 		$options['history_page'] = $history;
 	}
 
-	if ( ! empty( $options ) ) {
-		update_option( 'give_settings', array_merge( give_get_settings(), $options ) );
+	if ( ! empty($options)) {
+		update_option('give_settings', array_merge(give_get_settings(), $options));
 	}
 
-	add_option( 'give_install_pages_created', 1, '', 'no' );
+	add_option('give_install_pages_created', 1, '', 'no');
 }
 
-add_action( 'admin_init', 'give_create_pages', - 1 );
+add_action('admin_init', 'give_create_pages', - 1);

includes/payments/backward-compatibility.php

Doc Comments +1 added lines, -1 removed lines

@@ -207,7 +207,7 @@
  *                              metadata entries with the specified value.
  *                              Otherwise, update all entries.
  *
- * @return mixed
+ * @return null|boolean
  */
 function _give_20_bc_saving_old_payment_meta( $check, $object_id, $meta_key, $meta_value, $prev_value ) {
 	// Bailout.

Spacing +166 added lines, -171 removed lines

@@ -9,74 +9,74 @@ 
  *
  * @return void
  */
-function _give_20_bc_split_and_save_give_payment_meta( $object_id, $meta_value ) {
+function _give_20_bc_split_and_save_give_payment_meta($object_id, $meta_value) {
 	// Bailout
-	if ( empty( $meta_value ) ) {
+	if (empty($meta_value)) {
 		return;
-	} elseif ( ! is_array( $meta_value ) ) {
+	} elseif ( ! is_array($meta_value)) {
 		$meta_value = array();
 	}
 
-	remove_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10 );
+	remove_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10);
 
 	// Date payment meta.
-	if ( ! empty( $meta_value['date'] ) ) {
-		give_update_meta( $object_id, '_give_payment_date', $meta_value['date'] );
+	if ( ! empty($meta_value['date'])) {
+		give_update_meta($object_id, '_give_payment_date', $meta_value['date']);
 	}
 
 	// Currency payment meta.
-	if ( ! empty( $meta_value['currency'] ) ) {
-		give_update_meta( $object_id, '_give_payment_currency', $meta_value['currency'] );
+	if ( ! empty($meta_value['currency'])) {
+		give_update_meta($object_id, '_give_payment_currency', $meta_value['currency']);
 	}
 
 	// User information.
-	if ( ! empty( $meta_value['user_info'] ) ) {
+	if ( ! empty($meta_value['user_info'])) {
 		// Donor first name.
-		if ( ! empty( $meta_value['user_info']['first_name'] ) ) {
-			give_update_meta( $object_id, '_give_donor_billing_first_name', $meta_value['user_info']['first_name'] );
+		if ( ! empty($meta_value['user_info']['first_name'])) {
+			give_update_meta($object_id, '_give_donor_billing_first_name', $meta_value['user_info']['first_name']);
 		}
 
 		// Donor last name.
-		if ( ! empty( $meta_value['user_info']['last_name'] ) ) {
-			give_update_meta( $object_id, '_give_donor_billing_last_name', $meta_value['user_info']['last_name'] );
+		if ( ! empty($meta_value['user_info']['last_name'])) {
+			give_update_meta($object_id, '_give_donor_billing_last_name', $meta_value['user_info']['last_name']);
 		}
 
 		// Donor address payment meta.
-		if ( ! empty( $meta_value['user_info']['address'] ) ) {
+		if ( ! empty($meta_value['user_info']['address'])) {
 
 			// Address1.
-			if ( ! empty( $meta_value['user_info']['address']['line1'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_address1', $meta_value['user_info']['address']['line1'] );
+			if ( ! empty($meta_value['user_info']['address']['line1'])) {
+				give_update_meta($object_id, '_give_donor_billing_address1', $meta_value['user_info']['address']['line1']);
 			}
 
 			// Address2.
-			if ( ! empty( $meta_value['user_info']['address']['line2'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_address2', $meta_value['user_info']['address']['line2'] );
+			if ( ! empty($meta_value['user_info']['address']['line2'])) {
+				give_update_meta($object_id, '_give_donor_billing_address2', $meta_value['user_info']['address']['line2']);
 			}
 
 			// City.
-			if ( ! empty( $meta_value['user_info']['address']['city'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_city', $meta_value['user_info']['address']['city'] );
+			if ( ! empty($meta_value['user_info']['address']['city'])) {
+				give_update_meta($object_id, '_give_donor_billing_city', $meta_value['user_info']['address']['city']);
 			}
 
 			// Zip.
-			if ( ! empty( $meta_value['user_info']['address']['zip'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_zip', $meta_value['user_info']['address']['zip'] );
+			if ( ! empty($meta_value['user_info']['address']['zip'])) {
+				give_update_meta($object_id, '_give_donor_billing_zip', $meta_value['user_info']['address']['zip']);
 			}
 
 			// State.
-			if ( ! empty( $meta_value['user_info']['address']['state'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_state', $meta_value['user_info']['address']['state'] );
+			if ( ! empty($meta_value['user_info']['address']['state'])) {
+				give_update_meta($object_id, '_give_donor_billing_state', $meta_value['user_info']['address']['state']);
 			}
 
 			// Country.
-			if ( ! empty( $meta_value['user_info']['address']['country'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_country', $meta_value['user_info']['address']['country'] );
+			if ( ! empty($meta_value['user_info']['address']['country'])) {
+				give_update_meta($object_id, '_give_donor_billing_country', $meta_value['user_info']['address']['country']);
 			}
 		}
 	}// End if().
 
-	add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+	add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 }
 
 /**
@@ -89,103 +89,99 @@ 
  *
  * @return array
  */
-function _give_20_bc_give_payment_meta_value( $object_id, $meta_value ) {
+function _give_20_bc_give_payment_meta_value($object_id, $meta_value) {
 	$cache_key = "_give_payment_meta_{$object_id}";
-	$cache     = Give_Cache::get_db_query( $cache_key );
+	$cache     = Give_Cache::get_db_query($cache_key);
 
-	if ( ! is_null( $cache ) ) {
+	if ( ! is_null($cache)) {
 		return $cache;
 	}
 
 	// Set default value to array.
-	if ( ! is_array( $meta_value ) ) {
+	if ( ! is_array($meta_value)) {
 		$meta_value = array();
 	}
 
 	// Donation key.
-	$meta_value['key'] = give_get_meta( $object_id, '_give_payment_purchase_key', true );
+	$meta_value['key'] = give_get_meta($object_id, '_give_payment_purchase_key', true);
 
 	// Donation form.
-	$meta_value['form_title'] = give_get_meta( $object_id, '_give_payment_form_title', true );
+	$meta_value['form_title'] = give_get_meta($object_id, '_give_payment_form_title', true);
 
 	// Donor email.
-	$meta_value['email'] = give_get_meta( $object_id, '_give_payment_donor_email', true );
-	$meta_value['email'] = ! empty( $meta_value['email'] ) ?
-		$meta_value['email'] :
-		Give()->donors->get_column( 'email', give_get_payment_donor_id( $object_id ) );
+	$meta_value['email'] = give_get_meta($object_id, '_give_payment_donor_email', true);
+	$meta_value['email'] = ! empty($meta_value['email']) ?
+		$meta_value['email'] : Give()->donors->get_column('email', give_get_payment_donor_id($object_id));
 
 	// Form id.
-	$meta_value['form_id'] = give_get_meta( $object_id, '_give_payment_form_id', true );
+	$meta_value['form_id'] = give_get_meta($object_id, '_give_payment_form_id', true);
 
 	// Price id.
-	$meta_value['price_id'] = give_get_meta( $object_id, '_give_payment_price_id', true );
+	$meta_value['price_id'] = give_get_meta($object_id, '_give_payment_price_id', true);
 
 	// Date.
-	$meta_value['date'] = give_get_meta( $object_id, '_give_payment_date', true );
-	$meta_value['date'] = ! empty( $meta_value['date'] ) ?
-		$meta_value['date'] :
-		get_post_field( 'post_date', $object_id );
+	$meta_value['date'] = give_get_meta($object_id, '_give_payment_date', true);
+	$meta_value['date'] = ! empty($meta_value['date']) ?
+		$meta_value['date'] : get_post_field('post_date', $object_id);
 
 	// Currency.
-	$meta_value['currency'] = give_get_meta( $object_id, '_give_payment_currency', true );
+	$meta_value['currency'] = give_get_meta($object_id, '_give_payment_currency', true);
 
 	// Decode donor data.
-	$donor_names = give_get_donor_name_by( give_get_meta( $object_id, '_give_payment_donor_id', true ), 'donor' );
-	$donor_names = explode( ' ', $donor_names, 2 );
+	$donor_names = give_get_donor_name_by(give_get_meta($object_id, '_give_payment_donor_id', true), 'donor');
+	$donor_names = explode(' ', $donor_names, 2);
 
 	// Donor first name.
-	$donor_data['first_name'] = give_get_meta( $object_id, '_give_donor_billing_first_name', true );
-	$donor_data['first_name'] = ! empty( $donor_data['first_name'] ) ?
-		$donor_data['first_name'] :
-		$donor_names[0];
+	$donor_data['first_name'] = give_get_meta($object_id, '_give_donor_billing_first_name', true);
+	$donor_data['first_name'] = ! empty($donor_data['first_name']) ?
+		$donor_data['first_name'] : $donor_names[0];
 
 	// Donor last name.
-	$donor_data['last_name'] = give_get_meta( $object_id, '_give_donor_billing_last_name', true );
-	$donor_data['last_name'] = ! empty( $donor_data['last_name'] ) ?
-		$donor_data['last_name'] :
-		( isset( $donor_names[1] ) ? $donor_names[1] : '' );
+	$donor_data['last_name'] = give_get_meta($object_id, '_give_donor_billing_last_name', true);
+	$donor_data['last_name'] = ! empty($donor_data['last_name']) ?
+		$donor_data['last_name'] : (isset($donor_names[1]) ? $donor_names[1] : '');
 
 	// Donor email.
 	$donor_data['email'] = $meta_value['email'];
 
 	// User ID.
-	$donor_data['id'] = give_get_payment_user_id( $object_id );
+	$donor_data['id'] = give_get_payment_user_id($object_id);
 
 	$donor_data['address'] = false;
 
 	// Address1.
-	if ( $address1 = give_get_meta( $object_id, '_give_donor_billing_address1', true ) ) {
+	if ($address1 = give_get_meta($object_id, '_give_donor_billing_address1', true)) {
 		$donor_data['address']['line1'] = $address1;
 	}
 
 	// Address2.
-	if ( $address2 = give_get_meta( $object_id, '_give_donor_billing_address2', true ) ) {
+	if ($address2 = give_get_meta($object_id, '_give_donor_billing_address2', true)) {
 		$donor_data['address']['line2'] = $address2;
 	}
 
 	// City.
-	if ( $city = give_get_meta( $object_id, '_give_donor_billing_city', true ) ) {
+	if ($city = give_get_meta($object_id, '_give_donor_billing_city', true)) {
 		$donor_data['address']['city'] = $city;
 	}
 
 	// Zip.
-	if ( $zip = give_get_meta( $object_id, '_give_donor_billing_zip', true ) ) {
+	if ($zip = give_get_meta($object_id, '_give_donor_billing_zip', true)) {
 		$donor_data['address']['zip'] = $zip;
 	}
 
 	// State.
-	if ( $state = give_get_meta( $object_id, '_give_donor_billing_state', true ) ) {
+	if ($state = give_get_meta($object_id, '_give_donor_billing_state', true)) {
 		$donor_data['address']['state'] = $state;
 	}
 
 	// Country.
-	if ( $country = give_get_meta( $object_id, '_give_donor_billing_country', true ) ) {
+	if ($country = give_get_meta($object_id, '_give_donor_billing_country', true)) {
 		$donor_data['address']['country'] = $country;
 	}
 
-	$meta_value['user_info'] = maybe_unserialize( $donor_data );
+	$meta_value['user_info'] = maybe_unserialize($donor_data);
 
-	Give_Cache::set_db_query( $cache_key, $meta_value );
+	Give_Cache::set_db_query($cache_key, $meta_value);
 
 	return $meta_value;
 }
@@ -209,37 +205,37 @@ 
  *
  * @return mixed
  */
-function _give_20_bc_saving_old_payment_meta( $check, $object_id, $meta_key, $meta_value, $prev_value ) {
+function _give_20_bc_saving_old_payment_meta($check, $object_id, $meta_key, $meta_value, $prev_value) {
 	// Bailout.
 	if (
-		'give_payment' !== get_post_type( $object_id ) ||
-		! in_array( $meta_key, array(
+		'give_payment' !== get_post_type($object_id) ||
+		! in_array($meta_key, array(
 			'_give_payment_meta',
 			'_give_payment_user_email',
 			'_give_payment_customer_id',
 			'give_payment_user_ip',
-		) )
+		))
 	) {
 		return $check;
 	}
 
-	if ( '_give_payment_meta' === $meta_key ) {
-		_give_20_bc_split_and_save_give_payment_meta( $object_id, $meta_value );
-	} elseif ( '_give_payment_user_email' === $meta_key ) {
-		give_update_meta( $object_id, '_give_payment_donor_email', $meta_value );
+	if ('_give_payment_meta' === $meta_key) {
+		_give_20_bc_split_and_save_give_payment_meta($object_id, $meta_value);
+	} elseif ('_give_payment_user_email' === $meta_key) {
+		give_update_meta($object_id, '_give_payment_donor_email', $meta_value);
 		$check = true;
-	} elseif ( '_give_payment_customer_id' === $meta_key ) {
-		give_update_meta( $object_id, '_give_payment_donor_id', $meta_value );
+	} elseif ('_give_payment_customer_id' === $meta_key) {
+		give_update_meta($object_id, '_give_payment_donor_id', $meta_value);
 		$check = true;
-	} elseif ( 'give_payment_user_ip' === $meta_key ) {
-		give_update_meta( $object_id, '_give_payment_donor_ip', $meta_value );
+	} elseif ('give_payment_user_ip' === $meta_key) {
+		give_update_meta($object_id, '_give_payment_donor_ip', $meta_value);
 		$check = true;
 	}
 
 	return $check;
 }
 
-add_filter( 'update_post_metadata', '_give_20_bc_saving_old_payment_meta', 10, 5 );
+add_filter('update_post_metadata', '_give_20_bc_saving_old_payment_meta', 10, 5);
 
 
 /**
@@ -254,7 +250,7 @@ 
  *
  * @return mixed
  */
-function _give_20_bc_get_old_payment_meta( $check, $object_id, $meta_key, $single ) {
+function _give_20_bc_get_old_payment_meta($check, $object_id, $meta_key, $single) {
 	global $wpdb;
 
 	// Deprecated meta keys.
@@ -265,32 +261,31 @@ 
 	);
 
 	// Add _give_payment_meta to backward compatibility
-	if ( ! give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if ( ! give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		$old_meta_keys[] = '_give_payment_meta';
 	}
 
 	// Bailout.
 	if (
-		'give_payment' !== get_post_type( $object_id ) ||
-		! in_array( $meta_key, $old_meta_keys )
+		'give_payment' !== get_post_type($object_id) ||
+		! in_array($meta_key, $old_meta_keys)
 	) {
 		return $check;
 	}
 
 	$cache_key = "{$meta_key}_{$object_id}";
-	$check     = Give_Cache::get_db_query( $cache_key );
+	$check     = Give_Cache::get_db_query($cache_key);
 
-	if ( is_null( $check ) ) {
-		switch ( $meta_key ) {
+	if (is_null($check)) {
+		switch ($meta_key) {
 
 			// Handle old meta keys.
 			case '_give_payment_meta':
-				remove_filter( 'get_post_metadata', '_give_20_bc_get_old_payment_meta' );
+				remove_filter('get_post_metadata', '_give_20_bc_get_old_payment_meta');
 
 				// if ( $meta_value = give_get_meta( $object_id, '_give_payment_meta' ) ) {
-				$meta_value = ! empty( $meta_value ) ?
-					current( $meta_value ) :
-					(array) maybe_unserialize(
+				$meta_value = ! empty($meta_value) ?
+					current($meta_value) : (array) maybe_unserialize(
 						$wpdb->get_var(
 							$wpdb->prepare(
 								"
@@ -304,44 +299,44 @@ 
 							)
 						)
 					);
-				$check      = _give_20_bc_give_payment_meta_value( $object_id, $meta_value );
+				$check = _give_20_bc_give_payment_meta_value($object_id, $meta_value);
 				// }
 
-				add_filter( 'get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5 );
+				add_filter('get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5);
 
 				break;
 
 			case '_give_payment_customer_id':
-				if ( $donor_id = give_get_meta( $object_id, '_give_payment_donor_id', $single ) ) {
+				if ($donor_id = give_get_meta($object_id, '_give_payment_donor_id', $single)) {
 					$check = $donor_id;
 				}
 				break;
 
 			case '_give_payment_user_email':
-				if ( $donor_email = give_get_meta( $object_id, '_give_payment_donor_email', $single ) ) {
+				if ($donor_email = give_get_meta($object_id, '_give_payment_donor_email', $single)) {
 					$check = $donor_email;
 				}
 				break;
 
 			case '_give_payment_user_ip':
-				if ( $donor_ip = give_get_meta( $object_id, '_give_payment_donor_ip', $single ) ) {
+				if ($donor_ip = give_get_meta($object_id, '_give_payment_donor_ip', $single)) {
 					$check = $donor_ip;
 				}
 				break;
 		}// End switch().
 
-		Give_Cache::set_db_query( $cache_key, $check );
+		Give_Cache::set_db_query($cache_key, $check);
 	}
 
 	// Put result in an array on zero index.
-	if ( ! is_null( $check ) ) {
-		$check = array( $check );
+	if ( ! is_null($check)) {
+		$check = array($check);
 	}
 
 	return $check;
 }
 
-add_filter( 'get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5 );
+add_filter('get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5);
 
 
 /**
@@ -356,9 +351,9 @@ 
  *
  * @return mixed
  */
-function _give_20_bc_get_new_payment_meta( $check, $object_id, $meta_key, $single ) {
+function _give_20_bc_get_new_payment_meta($check, $object_id, $meta_key, $single) {
 	// Bailout: do not apply backward compatibility if upgrade done.
-	if ( give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if (give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		return $check;
 	}
 
@@ -380,26 +375,26 @@ 
 	);
 
 	// metadata_exists fx will cause of firing get_post_metadata filter again so remove it to prevent infinite loop.
-	remove_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta' );
+	remove_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta');
 
 	// Bailout.
 	if (
-		'give_payment' !== get_post_type( $object_id ) ||
-		! in_array( $meta_key, $new_meta_keys ) ||
-		metadata_exists( 'post', $object_id, $meta_key )
+		'give_payment' !== get_post_type($object_id) ||
+		! in_array($meta_key, $new_meta_keys) ||
+		metadata_exists('post', $object_id, $meta_key)
 	) {
-		add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+		add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 
 		return $check;
 	}
 
-	add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+	add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 
 	$cache_key = "{$meta_key}_{$object_id}";
-	$check    = Give_Cache::get_db_query( $cache_key );
+	$check = Give_Cache::get_db_query($cache_key);
 
-	if ( is_null( $check ) ) {
-		switch ( $meta_key ) {
+	if (is_null($check)) {
+		switch ($meta_key) {
 
 			// Handle new meta keys.
 			case '_give_payment_donor_id':
@@ -442,9 +437,9 @@ 
 			case '_give_donor_billing_country':
 			case '_give_payment_date':
 			case '_give_payment_currency':
-				$donation_meta = Give_Cache::get_db_query( "_give_payment_meta_{$object_id}" );
+				$donation_meta = Give_Cache::get_db_query("_give_payment_meta_{$object_id}");
 
-				if ( is_null( $donation_meta ) ) {
+				if (is_null($donation_meta)) {
 					$donation_meta = $wpdb->get_var(
 						$wpdb->prepare(
 							"SELECT meta_value FROM {$wpdb->postmeta} WHERE post_id=%d AND meta_key=%s",
@@ -452,51 +447,51 @@ 
 							'_give_payment_meta'
 						)
 					);
-					$donation_meta = maybe_unserialize( $donation_meta );
-					$donation_meta = ! is_array( $donation_meta ) ? array() : $donation_meta;
-					Give_Cache::set_db_query( "_give_payment_meta_{$object_id}", $donation_meta );
+					$donation_meta = maybe_unserialize($donation_meta);
+					$donation_meta = ! is_array($donation_meta) ? array() : $donation_meta;
+					Give_Cache::set_db_query("_give_payment_meta_{$object_id}", $donation_meta);
 				}
 
 				// Get results.
-				if ( empty( $donation_meta ) ) {
+				if (empty($donation_meta)) {
 					$check = '';
-				} elseif ( in_array( $meta_key, array( '_give_payment_date', '_give_payment_currency' ) ) ) {
-					$payment_meta_key = str_replace( '_give_payment_', '', $meta_key );
+				} elseif (in_array($meta_key, array('_give_payment_date', '_give_payment_currency'))) {
+					$payment_meta_key = str_replace('_give_payment_', '', $meta_key);
 
-					if ( isset( $donation_meta[ $payment_meta_key ] ) ) {
-						$check = $donation_meta[ $payment_meta_key ];
+					if (isset($donation_meta[$payment_meta_key])) {
+						$check = $donation_meta[$payment_meta_key];
 					}
 				} else {
-					$payment_meta_key = str_replace( '_give_donor_billing_', '', $meta_key );
+					$payment_meta_key = str_replace('_give_donor_billing_', '', $meta_key);
 
-					switch ( $payment_meta_key ) {
+					switch ($payment_meta_key) {
 						case 'address1':
-							if ( isset( $donation_meta['user_info']['address']['line1'] ) ) {
+							if (isset($donation_meta['user_info']['address']['line1'])) {
 								$check = $donation_meta['user_info']['address']['line1'];
 							}
 							break;
 
 						case 'address2':
-							if ( isset( $donation_meta['user_info']['address']['line2'] ) ) {
+							if (isset($donation_meta['user_info']['address']['line2'])) {
 								$check = $donation_meta['user_info']['address']['line2'];
 							}
 							break;
 
 						case 'first_name':
-							if ( isset( $donation_meta['user_info']['first_name'] ) ) {
+							if (isset($donation_meta['user_info']['first_name'])) {
 								$check = $donation_meta['user_info']['first_name'];
 							}
 							break;
 
 						case 'last_name':
-							if ( isset( $donation_meta['user_info']['last_name'] ) ) {
+							if (isset($donation_meta['user_info']['last_name'])) {
 								$check = $donation_meta['user_info']['last_name'];
 							}
 							break;
 
 						default:
-							if ( isset( $donation_meta['user_info']['address'][ $payment_meta_key ] ) ) {
-								$check = $donation_meta['user_info']['address'][ $payment_meta_key ];
+							if (isset($donation_meta['user_info']['address'][$payment_meta_key])) {
+								$check = $donation_meta['user_info']['address'][$payment_meta_key];
 							}
 					}
 				}
@@ -505,19 +500,19 @@ 
 		}// End switch().
 
 		// Set cache.
-		Give_Cache::set_db_query( $cache_key, $check );
+		Give_Cache::set_db_query($cache_key, $check);
 	}
 
 	// Put result in an array on zero index.
-	if ( ! $single ) {
-		$check = array( $check );
+	if ( ! $single) {
+		$check = array($check);
 	}
 
 
 	return $check;
 }
 
-add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 
 
 /**
@@ -529,9 +524,9 @@ 
  *
  * @return void
  */
-function _give_20_bc_support_deprecated_meta_key_query( $query ) {
+function _give_20_bc_support_deprecated_meta_key_query($query) {
 	// Bailout.
-	if ( give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if (give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		return;
 	}
 
@@ -541,21 +536,21 @@ 
 		// '_give_payment_user_ip'     => '_give_payment_donor_ip',
 	);
 
-	$deprecated_meta_keys = array_flip( $new_meta_keys );
+	$deprecated_meta_keys = array_flip($new_meta_keys);
 
 	// Set meta keys.
 	$meta_keys = array();
 
 
 	// Bailout.
-	if ( ! empty( $query->query_vars['meta_key'] ) ) {
-		if ( in_array( $query->query_vars['meta_key'], $new_meta_keys ) ) {
+	if ( ! empty($query->query_vars['meta_key'])) {
+		if (in_array($query->query_vars['meta_key'], $new_meta_keys)) {
 			$meta_keys = $deprecated_meta_keys;
-		} elseif ( in_array( $query->query_vars['meta_key'], $deprecated_meta_keys ) ) {
+		} elseif (in_array($query->query_vars['meta_key'], $deprecated_meta_keys)) {
 			$meta_keys = $new_meta_keys;
 		}
 
-		if ( ! empty( $meta_keys ) ) {
+		if ( ! empty($meta_keys)) {
 			// Set meta_query
 			$query->set(
 				'meta_query',
@@ -566,35 +561,35 @@ 
 						'value' => $query->query_vars['meta_value'],
 					),
 					array(
-						'key'   => $meta_keys[ $query->query_vars['meta_key'] ],
+						'key'   => $meta_keys[$query->query_vars['meta_key']],
 						'value' => $query->query_vars['meta_value'],
 					),
 				)
 			);
 
 			// Unset single meta query.
-			unset( $query->query_vars['meta_key'] );
-			unset( $query->query_vars['meta_value'] );
+			unset($query->query_vars['meta_key']);
+			unset($query->query_vars['meta_value']);
 		}
 	} elseif (
-		! empty( $query->query_vars['meta_query'] ) &&
-		( 1 === count( $query->query_vars['meta_query'] ) )
+		! empty($query->query_vars['meta_query']) &&
+		(1 === count($query->query_vars['meta_query']))
 	) {
-		$meta_query = current( $query->query_vars['meta_query'] );
+		$meta_query = current($query->query_vars['meta_query']);
 
-		if ( empty( $meta_query[0]['key'] ) ) {
+		if (empty($meta_query[0]['key'])) {
 			return;
 		}
 
-		if ( in_array( $meta_query[0]['key'], $new_meta_keys ) ) {
+		if (in_array($meta_query[0]['key'], $new_meta_keys)) {
 			$meta_keys = $deprecated_meta_keys;
-		} elseif ( in_array( $meta_query[0]['key'], $deprecated_meta_keys ) ) {
+		} elseif (in_array($meta_query[0]['key'], $deprecated_meta_keys)) {
 			$meta_keys = $new_meta_keys;
 		} else {
 			return;
 		}
 
-		if ( ! empty( $meta_keys ) ) {
+		if ( ! empty($meta_keys)) {
 			// Set meta_query
 			$query->set(
 				'meta_query',
@@ -605,7 +600,7 @@ 
 						'value' => $query->query_vars['meta_query'][0]['value'],
 					),
 					array(
-						'key'   => $meta_keys[ $query->query_vars['meta_query'][0]['key'] ],
+						'key'   => $meta_keys[$query->query_vars['meta_query'][0]['key']],
 						'value' => $query->query_vars['meta_query'][0]['value'],
 					),
 				)
@@ -614,7 +609,7 @@ 
 	}
 }
 
-add_action( 'pre_get_posts', '_give_20_bc_support_deprecated_meta_key_query' );
+add_action('pre_get_posts', '_give_20_bc_support_deprecated_meta_key_query');
 
 
 /**
@@ -627,49 +622,49 @@ 
  * @param Give_Payment $payment
  * @param string       $key
  */
-function _give_20_bc_payment_save( $payment, $key ) {
+function _give_20_bc_payment_save($payment, $key) {
 	// Bailout.
-	if ( ! give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if ( ! give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		return;
 	}
 
-	switch ( $key ) {
+	switch ($key) {
 		case 'user_info':
-			if ( empty( $payment->user_info ) ) {
+			if (empty($payment->user_info)) {
 				// Bailout.
 				break;
-			} elseif ( is_string( $payment->user_info ) ) {
+			} elseif (is_string($payment->user_info)) {
 				// Check if value serialize.
-				$payment->user_info = maybe_unserialize( $payment->user_info );
+				$payment->user_info = maybe_unserialize($payment->user_info);
 			}
 
 
 			// Save first name.
-			if ( isset( $payment->user_info['first_name'] ) ) {
-				$payment->update_meta( '_give_donor_billing_first_name', $payment->user_info['first_name'] );
+			if (isset($payment->user_info['first_name'])) {
+				$payment->update_meta('_give_donor_billing_first_name', $payment->user_info['first_name']);
 			}
 
 
 			// Save last name.
-			if ( isset( $payment->user_info['last_name'] ) ) {
-				$payment->update_meta( '_give_donor_billing_last_name', $payment->user_info['last_name'] );
+			if (isset($payment->user_info['last_name'])) {
+				$payment->update_meta('_give_donor_billing_last_name', $payment->user_info['last_name']);
 			}
 
 
 			// Save address.
-			if ( ! empty( $payment->user_info['address'] ) ) {
-				foreach ( $payment->user_info['address'] as $address_name => $address ) {
-					switch ( $address_name ) {
+			if ( ! empty($payment->user_info['address'])) {
+				foreach ($payment->user_info['address'] as $address_name => $address) {
+					switch ($address_name) {
 						case 'line1':
-							$payment->update_meta( '_give_donor_billing_address1', $address );
+							$payment->update_meta('_give_donor_billing_address1', $address);
 							break;
 
 						case 'line2':
-							$payment->update_meta( '_give_donor_billing_address2', $address );
+							$payment->update_meta('_give_donor_billing_address2', $address);
 							break;
 
 						default:
-							$payment->update_meta( "_give_donor_billing_{$address_name}", $address );
+							$payment->update_meta("_give_donor_billing_{$address_name}", $address);
 					}
 				}
 			}
@@ -678,7 +673,7 @@ 
 	}
 }
 
-add_action( 'give_payment_save', '_give_20_bc_payment_save', 10, 2 );
+add_action('give_payment_save', '_give_20_bc_payment_save', 10, 2);
 
 
 /**
@@ -691,16 +686,16 @@ 
  *
  * @return mixed
  */
-function __give_20_bc_flush_cache( $check, $object_id ) {
+function __give_20_bc_flush_cache($check, $object_id) {
 	if (
-		! give_has_upgrade_completed( 'v20_move_metadata_into_new_table' ) &&
-		'give_payment' === get_post_type( $object_id )
+		! give_has_upgrade_completed('v20_move_metadata_into_new_table') &&
+		'give_payment' === get_post_type($object_id)
 	) {
-		Give_Cache::delete_group( $object_id, 'give-donations' );
+		Give_Cache::delete_group($object_id, 'give-donations');
 	}
 
 	return $check;
 }
 
-add_action( 'update_post_metadata', '__give_20_bc_flush_cache', 9999, 2 );
-add_action( 'add_post_metadata', '__give_20_bc_flush_cache', 9999, 2 );
+add_action('update_post_metadata', '__give_20_bc_flush_cache', 9999, 2);
+add_action('add_post_metadata', '__give_20_bc_flush_cache', 9999, 2);

includes/payments/class-payments-query.php

Doc Comments +3 added lines, -3 removed lines

@@ -107,7 +107,7 @@ 
 	 * @since  1.0
 	 * @access public
 	 *
-	 * @param $query_var
+	 * @param string $query_var
 	 * @param $value
 	 */
 	public function __set( $query_var, $value ) {
@@ -124,7 +124,7 @@ 
 	 * @since  1.0
 	 * @access public
 	 *
-	 * @param $query_var
+	 * @param string $query_var
 	 */
 	public function __unset( $query_var ) {
 		unset( $this->args[ $query_var ] );
@@ -468,7 +468,7 @@ 
 	 * @param string   $order
 	 * @param WP_Query $query
 	 *
-	 * @return mixed
+	 * @return string
 	 */
 	public function custom_orderby( $order, $query ) {
 

Spacing +144 added lines, -144 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -66,10 +66,10 @@ 
 	 *
 	 * @param  $args array The array of arguments that can be passed in and used for setting up this payment query.
 	 */
-	public function __construct( $args = array() ) {
+	public function __construct($args = array()) {
 		$defaults = array(
 			'output'          => 'payments',
-			'post_type'       => array( 'give_payment' ),
+			'post_type'       => array('give_payment'),
 			'start_date'      => false,
 			'end_date'        => false,
 			'number'          => 20,
@@ -96,7 +96,7 @@ 
 			'count'           => false,
 		);
 
-		$this->args = $this->_args = wp_parse_args( $args, $defaults );
+		$this->args = $this->_args = wp_parse_args($args, $defaults);
 
 		$this->init();
 	}
@@ -110,11 +110,11 @@ 
 	 * @param $query_var
 	 * @param $value
 	 */
-	public function __set( $query_var, $value ) {
-		if ( in_array( $query_var, array( 'meta_query', 'tax_query' ) ) ) {
-			$this->args[ $query_var ][] = $value;
+	public function __set($query_var, $value) {
+		if (in_array($query_var, array('meta_query', 'tax_query'))) {
+			$this->args[$query_var][] = $value;
 		} else {
-			$this->args[ $query_var ] = $value;
+			$this->args[$query_var] = $value;
 		}
 	}
 
@@ -126,8 +126,8 @@ 
 	 *
 	 * @param $query_var
 	 */
-	public function __unset( $query_var ) {
-		unset( $this->args[ $query_var ] );
+	public function __unset($query_var) {
+		unset($this->args[$query_var]);
 	}
 
 	/**
@@ -167,7 +167,7 @@ 
 		$this->give_forms();
 		$this->gateway_filter();
 
-		add_filter( 'posts_orderby', array( $this, 'custom_orderby' ), 10, 2 );
+		add_filter('posts_orderby', array($this, 'custom_orderby'), 10, 2);
 
 		/**
 		 * Fires after setup filters.
@@ -176,7 +176,7 @@ 
 		 *
 		 * @param Give_Payments_Query $this Payments query object.
 		 */
-		do_action( 'give_pre_get_payments', $this );
+		do_action('give_pre_get_payments', $this);
 	}
 
 	/**
@@ -186,7 +186,7 @@ 
 	 * @access private
 	 */
 	private function unset_filters() {
-		remove_filter( 'posts_orderby', array( $this, 'custom_orderby' ) );
+		remove_filter('posts_orderby', array($this, 'custom_orderby'));
 
 		/**
 		 * Fires after retrieving payments.
@@ -195,7 +195,7 @@ 
 		 *
 		 * @param Give_Payments_Query $this Payments query object.
 		 */
-		do_action( 'give_post_get_payments', $this );
+		do_action('give_post_get_payments', $this);
 	}
 
 
@@ -214,11 +214,11 @@ 
 	public function get_payments() {
 		global $post;
 
-		$cache_key      = Give_Cache::get_key( 'give_payment_query', $this->args, false );
-		$this->payments = Give_Cache::get_db_query( $cache_key );
+		$cache_key      = Give_Cache::get_key('give_payment_query', $this->args, false);
+		$this->payments = Give_Cache::get_db_query($cache_key);
 
 		// Return cached result.
-		if ( ! is_null( $this->payments ) ) {
+		if ( ! is_null($this->payments)) {
 			return $this->payments;
 		}
 
@@ -226,7 +226,7 @@ 
 		// Modify the query/query arguments before we retrieve payments.
 		$this->set_filters();
 
-		$query          = new WP_Query( $this->args );
+		$query          = new WP_Query($this->args);
 		$this->payments = array();
 
 		$custom_output = array(
@@ -234,32 +234,32 @@ 
 			'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() ) {
+		if ($query->have_posts()) {
 			$previous_post = $post;
 
-			while ( $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();
 
 			// Prevent nest loop from producing unexpected results.
-			if( $previous_post instanceof WP_Post ) {
+			if ($previous_post instanceof WP_Post) {
 				$post = $previous_post;
-				setup_postdata( $post );
+				setup_postdata($post);
 			}
 		}
 
-		Give_Cache::set_db_query( $cache_key, $this->payments );
+		Give_Cache::set_db_query($cache_key, $this->payments);
 
 		// Remove query filters after we retrieve payments.
 		$this->unset_filters();
@@ -278,31 +278,31 @@ 
 	public function get_payment_by_group() {
 		global $wpdb;
 
-		$allowed_groups = array( 'post_status' );
+		$allowed_groups = array('post_status');
 		$result         = array();
 
 
-		if ( in_array( $this->args['group_by'], $allowed_groups ) ) {
+		if (in_array($this->args['group_by'], $allowed_groups)) {
 			// Set only count in result.
-			if ( $this->args['count'] ) {
+			if ($this->args['count']) {
 
 				$this->set_filters();
 
-				$new_results = $wpdb->get_results( $this->get_sql(), ARRAY_N );
+				$new_results = $wpdb->get_results($this->get_sql(), ARRAY_N);
 
 				$this->unset_filters();
 
-				foreach ( $new_results as $results ) {
-					$result[ $results[0] ] = $results[1];
+				foreach ($new_results as $results) {
+					$result[$results[0]] = $results[1];
 				}
 
-				switch ( $this->args['group_by'] ) {
+				switch ($this->args['group_by']) {
 					case 'post_status':
 
 						/* @var Give_Payment $donation */
-						foreach ( give_get_payment_status_keys() as $status ) {
-							if ( ! isset( $result[ $status ] ) ) {
-								$result[ $status ] = 0;
+						foreach (give_get_payment_status_keys() as $status) {
+							if ( ! isset($result[$status])) {
+								$result[$status] = 0;
 							}
 						}
 
@@ -312,8 +312,8 @@ 
 				$donations = $this->get_payments();
 
 				/* @var $donation Give_Payment */
-				foreach ( $donations as $donation ) {
-					$result[ $donation->{$this->args['group_by']} ][] = $donation;
+				foreach ($donations as $donation) {
+					$result[$donation->{$this->args['group_by']}][] = $donation;
 				}
 			}
 		}
@@ -324,7 +324,7 @@ 
 		 *
 		 * @since 1.8.17
 		 */
-		return apply_filters( 'give_get_payment_by_group', $result, $this );
+		return apply_filters('give_get_payment_by_group', $result, $this);
 	}
 
 	/**
@@ -336,29 +336,29 @@ 
 	 * @return void
 	 */
 	public function date_filter_pre() {
-		if ( ! ( $this->args['start_date'] || $this->args['end_date'] ) ) {
+		if ( ! ($this->args['start_date'] || $this->args['end_date'])) {
 			return;
 		}
 
-		$this->setup_dates( $this->args['start_date'], $this->args['end_date'] );
-		$is_start_date = property_exists( __CLASS__, 'start_date' );
-		$is_end_date   = property_exists( __CLASS__, 'end_date' );
+		$this->setup_dates($this->args['start_date'], $this->args['end_date']);
+		$is_start_date = property_exists(__CLASS__, 'start_date');
+		$is_end_date   = property_exists(__CLASS__, 'end_date');
 
-		if ( $is_start_date || $is_end_date ) {
+		if ($is_start_date || $is_end_date) {
 			$date_query = array();
 
-			if ( $is_start_date && ! is_wp_error( $this->start_date ) ) {
-				$date_query['after'] = date( 'Y-m-d H:i:s', $this->start_date );
+			if ($is_start_date && ! is_wp_error($this->start_date)) {
+				$date_query['after'] = date('Y-m-d H:i:s', $this->start_date);
 			}
 
-			if ( $is_end_date && ! is_wp_error( $this->end_date ) ) {
-				$date_query['before'] = date( 'Y-m-d H:i:s', $this->end_date );
+			if ($is_end_date && ! is_wp_error($this->end_date)) {
+				$date_query['before'] = date('Y-m-d H:i:s', $this->end_date);
 			}
 
 			// Include Start Date and End Date while querying.
 			$date_query['inclusive'] = true;
 
-			$this->__set( 'date_query', $date_query );
+			$this->__set('date_query', $date_query);
 
 		}
 	}
@@ -372,12 +372,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');
 	}
 
 	/**
@@ -389,12 +389,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');
 	}
 
 	/**
@@ -407,17 +407,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');
 	}
 
 	/**
@@ -429,12 +429,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');
 	}
 
 	/**
@@ -446,23 +446,23 @@ 
 	 * @return void
 	 */
 	public function orderby() {
-		switch ( $this->args['orderby'] ) {
+		switch ($this->args['orderby']) {
 			case 'amount':
-				$this->__set( 'orderby', 'meta_value_num' );
-				$this->__set( 'meta_key', '_give_payment_total' );
+				$this->__set('orderby', 'meta_value_num');
+				$this->__set('meta_key', '_give_payment_total');
 				break;
 
 			case 'status':
-				$this->__set( 'orderby', 'post_status' );
+				$this->__set('orderby', 'post_status');
 				break;
 
 			case 'donation_form':
-				$this->__set( 'orderby', 'meta_value' );
-				$this->__set( 'meta_key', '_give_payment_form_title' );
+				$this->__set('orderby', 'meta_value');
+				$this->__set('meta_key', '_give_payment_form_title');
 				break;
 
 			default:
-				$this->__set( 'orderby', $this->args['orderby'] );
+				$this->__set('orderby', $this->args['orderby']);
 				break;
 		}
 	}
@@ -479,19 +479,19 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function custom_orderby( $order, $query ) {
+	public function custom_orderby($order, $query) {
 
-		if ( ! empty( $query->query['post_type'] ) ) {
-			$post_types = is_array( $query->query['post_type'] ) ? $query->query['post_type'] : array( $query->query['post_type'] );
+		if ( ! empty($query->query['post_type'])) {
+			$post_types = is_array($query->query['post_type']) ? $query->query['post_type'] : array($query->query['post_type']);
 
-			if ( ! in_array( 'give_payment', $post_types ) || ! isset( $query->query['orderby'] ) || is_array( $query->query['orderby'] ) ) {
+			if ( ! in_array('give_payment', $post_types) || ! isset($query->query['orderby']) || is_array($query->query['orderby'])) {
 				return $order;
 			}
 
 			global $wpdb;
-			switch ( $query->query['orderby'] ) {
+			switch ($query->query['orderby']) {
 				case 'post_status':
-					$order = $wpdb->posts . '.post_status ' . strtoupper( $query->query['order'] );
+					$order = $wpdb->posts.'.post_status '.strtoupper($query->query['order']);
 					break;
 			}
 		}
@@ -508,11 +508,11 @@ 
 	 * @return void
 	 */
 	public function user() {
-		if ( is_null( $this->args['user'] ) ) {
+		if (is_null($this->args['user'])) {
 			return;
 		}
 
-		if ( is_numeric( $this->args['user'] ) ) {
+		if (is_numeric($this->args['user'])) {
 			$user_key = '_give_payment_donor_id';
 		} else {
 			$user_key = '_give_payment_donor_email';
@@ -534,16 +534,16 @@ 
 	 * @return  void
 	 */
 	public function donor() {
-		if ( is_null( $this->args['donor'] ) || ! is_numeric( $this->args['donor'] ) ) {
+		if (is_null($this->args['donor']) || ! is_numeric($this->args['donor'])) {
 			return;
 		}
 
 		$donor_meta_type = Give()->donor_meta->meta_type;
 
-		$this->__set( 'meta_query', array(
+		$this->__set('meta_query', array(
 			'key'   => "_give_payment_{$donor_meta_type}_id",
 			'value' => (int) $this->args['donor'],
-		) );
+		));
 	}
 
 	/**
@@ -556,33 +556,33 @@ 
 	 */
 	public function search() {
 
-		if ( ! isset( $this->args['s'] ) ) {
+		if ( ! isset($this->args['s'])) {
 			return;
 		}
 
-		$search = trim( $this->args['s'] );
+		$search = trim($this->args['s']);
 
-		if ( empty( $search ) ) {
+		if (empty($search)) {
 			return;
 		}
 
-		$is_email = is_email( $search ) || strpos( $search, '@' ) !== false;
-		$is_user  = strpos( $search, strtolower( 'user:' ) ) !== false;
+		$is_email = is_email($search) || strpos($search, '@') !== false;
+		$is_user  = strpos($search, strtolower('user:')) !== false;
 
-		if ( ! empty( $this->args['search_in_notes'] ) ) {
+		if ( ! empty($this->args['search_in_notes'])) {
 
-			$notes = give_get_payment_notes( 0, $search );
+			$notes = give_get_payment_notes(0, $search);
 
-			if ( ! empty( $notes ) ) {
+			if ( ! empty($notes)) {
 
-				$payment_ids = wp_list_pluck( (array) $notes, 'comment_post_ID' );
+				$payment_ids = wp_list_pluck((array) $notes, 'comment_post_ID');
 
-				$this->__set( 'post__in', $payment_ids );
+				$this->__set('post__in', $payment_ids);
 			}
 
-			$this->__unset( 's' );
+			$this->__unset('s');
 
-		} elseif ( $is_email || strlen( $search ) == 32 ) {
+		} elseif ($is_email || strlen($search) == 32) {
 
 			$key         = $is_email ? '_give_payment_donor_email' : '_give_payment_purchase_key';
 			$search_meta = array(
@@ -591,40 +591,40 @@ 
 				'compare' => 'LIKE',
 			);
 
-			$this->__set( 'meta_query', $search_meta );
-			$this->__unset( 's' );
+			$this->__set('meta_query', $search_meta);
+			$this->__unset('s');
 
-		} elseif ( $is_user ) {
+		} elseif ($is_user) {
 
 			$search_meta = array(
 				'key'   => '_give_payment_donor_id',
-				'value' => trim( str_replace( 'user:', '', strtolower( $search ) ) ),
+				'value' => trim(str_replace('user:', '', strtolower($search))),
 			);
 
-			$this->__set( 'meta_query', $search_meta );
+			$this->__set('meta_query', $search_meta);
 
-			$this->__unset( 's' );
+			$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);
 
 		}
 
@@ -639,8 +639,8 @@ 
 	 * @return void
 	 */
 	public function mode() {
-		if ( empty( $this->args['mode'] ) || $this->args['mode'] == 'all' ) {
-			$this->__unset( 'mode' );
+		if (empty($this->args['mode']) || $this->args['mode'] == 'all') {
+			$this->__unset('mode');
 
 			return;
 		}
@@ -662,10 +662,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');
 	}
 
 	/**
@@ -678,13 +678,13 @@ 
 	 */
 	public function give_forms() {
 
-		if ( empty( $this->args['give_forms'] ) ) {
+		if (empty($this->args['give_forms'])) {
 			return;
 		}
 
 		$compare = '=';
 
-		if ( is_array( $this->args['give_forms'] ) ) {
+		if (is_array($this->args['give_forms'])) {
 			$compare = 'IN';
 		}
 
@@ -697,7 +697,7 @@ 
 			)
 		);
 
-		$this->__unset( 'give_forms' );
+		$this->__unset('give_forms');
 
 	}
 
@@ -711,13 +711,13 @@ 
 	 */
 	public function gateway_filter() {
 
-		if ( empty( $this->args['gateway'] ) ) {
+		if (empty($this->args['gateway'])) {
 			return;
 		}
 
 		$compare = '=';
 
-		if ( is_array( $this->args['gateway'] ) ) {
+		if (is_array($this->args['gateway'])) {
 			$compare = 'IN';
 		}
 
@@ -731,7 +731,7 @@ 
 			)
 		);
 
-		$this->__unset( 'gateway' );
+		$this->__unset('gateway');
 
 	}
 
@@ -751,9 +751,9 @@ 
 		global $wpdb;
 
 		$where = "WHERE {$wpdb->posts}.post_type = 'give_payment'";
-		$where .= " AND {$wpdb->posts}.post_status IN ('" . implode( "','", $this->args['post_status'] ) . "')";
+		$where .= " AND {$wpdb->posts}.post_status IN ('".implode("','", $this->args['post_status'])."')";
 
-		if( is_numeric( $this->args['post_parent'] ) ) {
+		if (is_numeric($this->args['post_parent'])) {
 			$where .= " AND {$wpdb->posts}.post_parent={$this->args['post_parent']}";
 		}
 
@@ -762,42 +762,42 @@ 
 		$group_by = '';
 
 		// Set group by.
-		if ( ! empty( $this->args['group_by'] ) ) {
+		if ( ! empty($this->args['group_by'])) {
 			$group_by = "GROUP BY {$wpdb->posts}.{$this->args['group_by']}";
 		}
 
 		// Set offset.
 		if (
-			empty( $this->args['nopaging'] ) &&
-			empty( $this->args['offset'] ) &&
-			( ! empty( $this->args['page'] ) && 0 < $this->args['page'] )
+			empty($this->args['nopaging']) &&
+			empty($this->args['offset']) &&
+			( ! empty($this->args['page']) && 0 < $this->args['page'])
 		) {
-			$this->args['offset'] = $this->args['posts_per_page'] * ( $this->args['page'] - 1 );
+			$this->args['offset'] = $this->args['posts_per_page'] * ($this->args['page'] - 1);
 		}
 
 		// Set fields.
 		$fields = "{$wpdb->posts}.*";
-		if ( ! empty( $this->args['fields'] ) && 'all' !== $this->args['fields'] ) {
-			if ( is_string( $this->args['fields'] ) ) {
+		if ( ! empty($this->args['fields']) && 'all' !== $this->args['fields']) {
+			if (is_string($this->args['fields'])) {
 				$fields = "{$wpdb->posts}.{$this->args['fields']}";
-			} elseif ( is_array( $this->args['fields'] ) ) {
-				$fields = "{$wpdb->posts}." . implode( " , {$wpdb->posts}.", $this->args['fields'] );
+			} elseif (is_array($this->args['fields'])) {
+				$fields = "{$wpdb->posts}.".implode(" , {$wpdb->posts}.", $this->args['fields']);
 			}
 		}
 
 		// Set count.
-		if ( ! empty( $this->args['count'] ) ) {
+		if ( ! empty($this->args['count'])) {
 			$fields = "COUNT({$wpdb->posts}.ID)";
 
-			if ( ! empty( $this->args['group_by'] ) ) {
+			if ( ! empty($this->args['group_by'])) {
 				$fields = "{$wpdb->posts}.{$this->args['group_by']}, {$fields}";
 			}
 		}
 
 		// Date query.
-		if ( ! empty( $this->args['date_query'] ) ) {
-			$date_query_obj = new WP_Date_Query( $this->args['date_query'] );
-			$where          .= str_replace(
+		if ( ! empty($this->args['date_query'])) {
+			$date_query_obj = new WP_Date_Query($this->args['date_query']);
+			$where .= str_replace(
 				array(
 					"\n",
 					'(   (',
@@ -813,22 +813,22 @@ 
 		}
 
 		// Meta query.
-		if ( ! empty( $this->args['meta_query'] ) ) {
-			$meta_query_obj = new WP_Meta_Query( $this->args['meta_query'] );
-			$where          = implode( ' ', $meta_query_obj->get_sql( 'post', $wpdb->posts, 'ID' ) ) . " {$where}";
-			$where          = Give()->payment_meta->__rename_meta_table_name( $where, 'posts_where' );
+		if ( ! empty($this->args['meta_query'])) {
+			$meta_query_obj = new WP_Meta_Query($this->args['meta_query']);
+			$where          = implode(' ', $meta_query_obj->get_sql('post', $wpdb->posts, 'ID'))." {$where}";
+			$where          = Give()->payment_meta->__rename_meta_table_name($where, 'posts_where');
 		}
 
 		// Set sql query.
 		$sql = $wpdb->prepare(
 			"SELECT {$fields} FROM {$wpdb->posts} LIMIT %d,%d;",
-			absint( $this->args['offset'] ),
-			( empty( $this->args['nopaging'] ) ? absint( $this->args['posts_per_page'] ) : 99999999999 )
+			absint($this->args['offset']),
+			(empty($this->args['nopaging']) ? absint($this->args['posts_per_page']) : 99999999999)
 		);
 
 		// $where, $orderby and order already prepared query they can generate notice if you re prepare them in above.
 		// WordPress consider LIKE condition as placeholder if start with s,f, or d.
-		$sql = str_replace( 'LIMIT', "{$where} {$group_by} {$orderby} {$this->args['order']} LIMIT", $sql );
+		$sql = str_replace('LIMIT', "{$where} {$group_by} {$orderby} {$this->args['order']} LIMIT", $sql);
 
 		return $sql;
 	}

includes/admin/tools/logs/class-gateway-error-logs-list-table.php

Spacing +52 added lines, -52 removed lines

@@ -9,13 +9,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';
 }
 
 /**
@@ -44,11 +44,11 @@ 
 	 */
 	public function __construct() {
 		// 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.
-			'ajax'     => false,// Does this table support ajax?.
-		) );
+		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?.
+		));
 	}
 
 	/**
@@ -62,19 +62,19 @@ 
 	 *
 	 * @return string Column Name.
 	 */
-	public function column_default( $item, $column_name ) {
+	public function column_default($item, $column_name) {
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 			case 'ID' :
 				return $item['ID_label'];
 			case 'payment_id' :
-				return empty( $item['payment_id'] ) ? esc_html__( 'n/a', 'give' ) : sprintf( "<a href=\"%s\" target=\"_blank\">{$item['payment_id']}</a>", get_edit_post_link( $item['payment_id'] ) );
+				return empty($item['payment_id']) ? esc_html__('n/a', 'give') : sprintf("<a href=\"%s\" target=\"_blank\">{$item['payment_id']}</a>", get_edit_post_link($item['payment_id']));
 			case 'gateway' :
-				return empty( $item['gateway'] ) ? esc_html__( 'n/a', 'give' ) : $item['gateway'];
+				return empty($item['gateway']) ? esc_html__('n/a', 'give') : $item['gateway'];
 			case 'error' :
-				return esc_html( $item['log_title'] );
+				return esc_html($item['log_title']);
 			default:
-				return $item[ $column_name ];
+				return $item[$column_name];
 		}
 	}
 
@@ -88,35 +88,35 @@ 
 	 *
 	 * @return void
 	 */
-	public function column_message( $item ) {
+	public function column_message($item) {
 		?>
 		<?php
-		echo Give()->tooltips->render_link( array(
-			'label'       => __( 'View Log Message', 'give' ),
+		echo Give()->tooltips->render_link(array(
+			'label'       => __('View Log Message', 'give'),
 			'tag_content' => '<span class="dashicons dashicons-visibility"></span>',
 			'link'        => "#TB_inline?width=640&amp;inlineId=log-message-{$item['ID']}",
 			'attributes'  => array(
 				'class' => 'thickbox give-error-log-details-link button button-small',
 			),
-		) );
+		));
 		?>
 		<div id="log-message-<?php echo $item['ID']; ?>" style="display:none;">
 			<?php
 
-			$serialized = strpos( $item['log_content'], '{"' );
+			$serialized = strpos($item['log_content'], '{"');
 
 			// Check to see if the log message contains serialized information
-			if ( $serialized !== false ) {
-				$length = strlen( $item['log_content'] ) - $serialized;
-				$intro  = substr( $item['log_content'], 0, - $length );
-				$data   = substr( $item['log_content'], $serialized, strlen( $item['log_content'] ) - 1 );
+			if ($serialized !== false) {
+				$length = strlen($item['log_content']) - $serialized;
+				$intro  = substr($item['log_content'], 0, - $length);
+				$data   = substr($item['log_content'], $serialized, strlen($item['log_content']) - 1);
 
-				echo wpautop( $intro );
-				echo wpautop( '<strong>' . esc_html__( 'Log data:', 'give' ) . '</strong>' );
-				echo '<div style="word-wrap: break-word;">' . wpautop( $data ) . '</div>';
+				echo wpautop($intro);
+				echo wpautop('<strong>'.esc_html__('Log data:', 'give').'</strong>');
+				echo '<div style="word-wrap: break-word;">'.wpautop($data).'</div>';
 			} else {
 				// No serialized data found
-				echo wpautop( $item['log_content'] );
+				echo wpautop($item['log_content']);
 			}
 			?>
 		</div>
@@ -132,12 +132,12 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'ID'         => esc_html__( 'Log ID', 'give' ),
-			'error'      => esc_html__( 'Error', 'give' ),
-			'gateway'    => esc_html__( 'Gateway', 'give' ),
-			'payment_id' => esc_html__( 'Donation ID', 'give' ),
-			'date'       => esc_html__( 'Date', 'give' ),
-			'message'    => esc_html__( 'Details', 'give' ),
+			'ID'         => esc_html__('Log ID', 'give'),
+			'error'      => esc_html__('Error', 'give'),
+			'gateway'    => esc_html__('Gateway', 'give'),
+			'payment_id' => esc_html__('Donation ID', 'give'),
+			'date'       => esc_html__('Date', 'give'),
+			'message'    => esc_html__('Details', 'give'),
 		);
 
 		return $columns;
@@ -151,7 +151,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -161,7 +161,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 		give_log_views();
 	}
 
@@ -176,7 +176,7 @@ 
 	public function get_logs() {
 		// Prevent the queries from getting cached.
 		// Without this there are occasional memory issues for some installs.
-		wp_suspend_cache_addition( true );
+		wp_suspend_cache_addition(true);
 
 		$logs_data = array();
 		$paged     = $this->get_paged();
@@ -186,17 +186,17 @@ 
 			'posts_per_page' => $this->per_page,
 		);
 
-		$logs = Give()->logs->get_connected_logs( $log_query );
+		$logs = Give()->logs->get_connected_logs($log_query);
 
-		if ( $logs ) {
-			foreach ( $logs as $log ) {
+		if ($logs) {
+			foreach ($logs as $log) {
 
 				$logs_data[] = array(
 					'ID'          => $log->ID,
-					'ID_label'    => '<span class=\'give-item-label give-item-label-gray\'>' . $log->ID . '</span>',
+					'ID_label'    => '<span class=\'give-item-label give-item-label-gray\'>'.$log->ID.'</span>',
 					'payment_id'  => $log->log_parent,
 					'error'       => 'error',
-					'gateway'     => give_get_payment_gateway( $log->log_parent ),
+					'gateway'     => give_get_payment_gateway($log->log_parent),
 					'date'        => $log->log_date,
 					'log_content' => $log->log_content,
 					'log_title'   => $log->log_title,
@@ -220,19 +220,19 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+	protected function display_tablenav($which) {
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav <?php echo esc_attr($which); ?>">
 
 			<div class="alignleft actions bulkactions">
-				<?php $this->bulk_actions( $which ); ?>
+				<?php $this->bulk_actions($which); ?>
 			</div>
 			<?php
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 
 			<br class="clear"/>
@@ -258,14 +258,14 @@ 
 		$columns               = $this->get_columns();
 		$hidden                = array(); // No hidden columns
 		$sortable              = $this->get_sortable_columns();
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 		$this->items           = $this->get_logs();
-		$total_items           = Give()->logs->get_log_count( 0, 'gateway_error' );
+		$total_items           = Give()->logs->get_log_count(0, 'gateway_error');
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 				'total_items' => $total_items,
 				'per_page'    => $this->per_page,
-				'total_pages' => ceil( $total_items / $this->per_page ),
+				'total_pages' => ceil($total_items / $this->per_page),
 			)
 		);
 	}

includes/admin/emails/class-new-offline-donation-email.php

Spacing +40 added lines, -40 removed lines

@@ -12,11 +12,11 @@ 
  */
 
 // Exit if access directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give_New_Offline_Donation_Email' ) ) :
+if ( ! class_exists('Give_New_Offline_Donation_Email')) :
 
 	/**
 	 * Give_New_Offline_Donation_Email
@@ -36,32 +36,32 @@ 
 		 */
 		public function init() {
 			// Initialize empty payment.
-			$this->payment = new Give_Payment( 0 );
+			$this->payment = new Give_Payment(0);
 
-			$this->load( array(
+			$this->load(array(
 				'id'                           => 'new-offline-donation',
-				'label'                        => __( 'New Offline Donation', 'give' ),
-				'description'                  => __( 'Sent to designated recipient(s) for a new (pending) offline donation.', 'give' ),
+				'label'                        => __('New Offline Donation', 'give'),
+				'description'                  => __('Sent to designated recipient(s) for a new (pending) offline donation.', 'give'),
 				'has_recipient_field'          => true,
-				'notification_status'          => give_is_gateway_active( 'offline' ) ? 'enabled' : 'disabled',
+				'notification_status'          => give_is_gateway_active('offline') ? 'enabled' : 'disabled',
 				'notification_status_editable' => false,
 				'preview_email_tags_values'    => array(
-					'payment_method' => esc_html__( 'Offline', 'give' ),
+					'payment_method' => esc_html__('Offline', 'give'),
 				),
 				'default_email_subject'        => $this->get_default_email_subject(),
 				'default_email_message'        => $this->get_default_email_message(),
 				'notices' => array(
 					'non-notification-status-editable' => sprintf(
 						'%1$s <a href="%2$s">%3$s &raquo;</a>',
-						__( 'This notification is automatically toggled based on whether the gateway is enabled or not.', 'give' ),
-						esc_url( admin_url('edit.php?post_type=give_forms&page=give-settings&tab=gateways&section=offline-donations') ),
-						__( 'Edit Setting', 'give' )
+						__('This notification is automatically toggled based on whether the gateway is enabled or not.', 'give'),
+						esc_url(admin_url('edit.php?post_type=give_forms&page=give-settings&tab=gateways&section=offline-donations')),
+						__('Edit Setting', 'give')
 					)
 				),
-			) );
+			));
 
-			add_action( 'give_insert_payment', array( $this, 'setup_email_notification' ) );
-			add_action( 'give_save_settings_give_settings', array( $this, 'set_notification_status' ), 10, 2 );
+			add_action('give_insert_payment', array($this, 'setup_email_notification'));
+			add_action('give_save_settings_give_settings', array($this, 'set_notification_status'), 10, 2);
 		}
 
 		/**
@@ -80,7 +80,7 @@ 
 			 */
 			$subject = apply_filters(
 				'give_offline_admin_donation_notification_subject',
-				__( 'New Pending Donation', 'give' )
+				__('New Pending Donation', 'give')
 			);
 
 			/**
@@ -105,18 +105,18 @@ 
 		 * @return string
 		 */
 		public function get_default_email_message() {
-			$message = __( 'Dear Admin,', 'give' ) . "\n\n";
-			$message .= __( 'An offline donation has been made on your website:', 'give' ) . ' ' . get_bloginfo( 'name' ) . ' ';
-			$message .= __( 'Hooray! The donation is in a pending status and is awaiting payment. Donation instructions have been emailed to the donor. Once you receive payment, be sure to mark the donation as complete using the link below.', 'give' ) . "\n\n";
+			$message = __('Dear Admin,', 'give')."\n\n";
+			$message .= __('An offline donation has been made on your website:', 'give').' '.get_bloginfo('name').' ';
+			$message .= __('Hooray! The donation is in a pending status and is awaiting payment. Donation instructions have been emailed to the donor. Once you receive payment, be sure to mark the donation as complete using the link below.', 'give')."\n\n";
 
-			$message .= '<strong>' . __( 'Donor:', 'give' ) . '</strong> {fullname}' . "\n";
-			$message .= '<strong>' . __( 'Amount:', 'give' ) . '</strong> {amount}' . "\n\n";
+			$message .= '<strong>'.__('Donor:', 'give').'</strong> {fullname}'."\n";
+			$message .= '<strong>'.__('Amount:', 'give').'</strong> {amount}'."\n\n";
 
 			$message .= sprintf(
 				'<a href="%1$s">%2$s</a>',
-				admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details&id=' . $this->payment->ID ),
-				__( 'Click Here to View and/or Update Donation Details', 'give' )
-			) . "\n\n";
+				admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details&id='.$this->payment->ID),
+				__('Click Here to View and/or Update Donation Details', 'give')
+			)."\n\n";
 
 			/**
 			 * Filter the donation receipt email message
@@ -154,10 +154,10 @@ 
 		 *
 		 * @return string
 		 */
-		public function get_email_message( $form_id = null ) {
+		public function get_email_message($form_id = null) {
 			$message = Give_Email_Notification_Util::get_value(
 				$this,
-				Give_Email_Setting_Field::get_prefix( $this, $form_id ) . 'email_message',
+				Give_Email_Setting_Field::get_prefix($this, $form_id).'email_message',
 				$form_id,
 				$this->config['default_email_message']
 			);
@@ -197,7 +197,7 @@ 
 		 *
 		 * @return array
 		 */
-		public function get_email_attachments( $form_id = null ) {
+		public function get_email_attachments($form_id = null) {
 			/**
 			 * Filter the attachments.
 			 * Note: This filter will deprecate soon.
@@ -248,11 +248,11 @@ 
 		 *
 		 * @param int $payment_id
 		 */
-		public function setup_email_notification( $payment_id ) {
-			$this->payment = new Give_Payment( $payment_id );
+		public function setup_email_notification($payment_id) {
+			$this->payment = new Give_Payment($payment_id);
 
 			// Exit if not donation was not with offline donation.
-			if ( 'offline' !== $this->payment->gateway ) {
+			if ('offline' !== $this->payment->gateway) {
 				return;
 			}
 
@@ -260,9 +260,9 @@ 
 			$this->setup_email_data();
 
 			// Send email.
-			$this->send_email_notification( array(
+			$this->send_email_notification(array(
 				'payment_id' => $this->payment->ID,
-			) );
+			));
 		}
 
 		/**
@@ -274,18 +274,18 @@ 
 		 * @param $update_options
 		 * @param $option_name
 		 */
-		public function set_notification_status( $update_options, $option_name ) {
+		public function set_notification_status($update_options, $option_name) {
 			// Get updated settings.
 			$update_options = give_get_settings();
 
-			$notification_status = isset( $update_options['gateways']['offline'] ) ? 'enabled' : 'disabled';
+			$notification_status = isset($update_options['gateways']['offline']) ? 'enabled' : 'disabled';
 
 			if (
-				empty( $update_options[ "{$this->config['id']}_notification" ] )
-				|| $notification_status !== $update_options[ "{$this->config['id']}_notification" ]
+				empty($update_options["{$this->config['id']}_notification"])
+				|| $notification_status !== $update_options["{$this->config['id']}_notification"]
 			) {
-				$update_options[ "{$this->config['id']}_notification" ] = $notification_status;
-				update_option( $option_name, $update_options );
+				$update_options["{$this->config['id']}_notification"] = $notification_status;
+				update_option($option_name, $update_options);
 			}
 		}
 
@@ -300,13 +300,13 @@ 
 		 *
 		 * @return array
 		 */
-		public function add_metabox_setting_field( $settings, $form_id ) {
+		public function add_metabox_setting_field($settings, $form_id) {
 
-			if ( in_array( 'offline', array_keys( give_get_enabled_payment_gateways($form_id) ) ) ) {
+			if (in_array('offline', array_keys(give_get_enabled_payment_gateways($form_id)))) {
 				$settings[] = array(
 					'id'     => $this->config['id'],
 					'title'  => $this->config['label'],
-					'fields' => $this->get_setting_fields( $form_id ),
+					'fields' => $this->get_setting_fields($form_id),
 				);
 			}
 

includes/admin/emails/filters.php

Spacing +12 added lines, -12 removed lines

@@ -19,28 +19,28 @@ 
  *
  * @return array
  */
-function give_email_notification_row_actions_callback( $row_actions, $email ) {
-	if( Give_Email_Notification_Util::is_email_preview( $email ) ) {
+function give_email_notification_row_actions_callback($row_actions, $email) {
+	if (Give_Email_Notification_Util::is_email_preview($email)) {
 		$preview_link = sprintf(
 			'<a href="%1$s" target="_blank">%2$s</a>',
 			wp_nonce_url(
 				add_query_arg(
-					array( 'give_action' => 'preview_email', 'email_type' => $email->config['id'] ),
+					array('give_action' => 'preview_email', 'email_type' => $email->config['id']),
 					home_url()
 				), 'give-preview-email'
 			),
-			__( 'Preview', 'give' )
+			__('Preview', 'give')
 		);
 
 		$send_preview_email_link = sprintf(
 			'<a href="%1$s">%2$s</a>',
 			wp_nonce_url(
-				add_query_arg( array(
+				add_query_arg(array(
 					'give_action'  => 'send_preview_email',
 					'email_type' => $email->config['id'],
 					'give-message' => 'sent-test-email',
-				) ), 'give-send-preview-email' ),
-			__( 'Send test email', 'give' )
+				)), 'give-send-preview-email' ),
+			__('Send test email', 'give')
 		);
 
 		$row_actions['email_preview'] = $preview_link;
@@ -49,7 +49,7 @@ 
 
 	return $row_actions;
 }
-add_filter( 'give_email_notification_row_actions', 'give_email_notification_row_actions_callback', 10, 2 );
+add_filter('give_email_notification_row_actions', 'give_email_notification_row_actions_callback', 10, 2);
 
 /**
  * This help to decode all email template tags.
@@ -61,12 +61,12 @@ 
  *
  * @return string
  */
-function give_decode_email_tags( $message, $email_obj ) {
-	if ( ! empty( $email_obj->tag_args ) ) {
-		$message = give_do_email_tags( $message, $email_obj->tag_args );
+function give_decode_email_tags($message, $email_obj) {
+	if ( ! empty($email_obj->tag_args)) {
+		$message = give_do_email_tags($message, $email_obj->tag_args);
 	}
 
 	return $message;
 }
 
-add_filter( 'give_email_message', 'give_decode_email_tags', 10, 2 );
\ No newline at end of file
+add_filter('give_email_message', 'give_decode_email_tags', 10, 2);
\ No newline at end of file

includes/admin/emails/class-email-access-email.php

Indentation +4 added lines, -4 removed lines

@@ -175,10 +175,10 @@
 		 */
 		public function get_default_email_message() {
 			$message = sprintf(
-				           __( 'Please click the link to access your donation history on <a target="_blank" href="%1$s">%1$s</a>. If you did not request this email, please contact <a href="mailto:%2$s">%2$s</a>.', 'give' ),
-				           get_bloginfo( 'url' ),
-				           get_bloginfo( 'admin_email' )
-			           ) . "\n\n";
+						   __( 'Please click the link to access your donation history on <a target="_blank" href="%1$s">%1$s</a>. If you did not request this email, please contact <a href="mailto:%2$s">%2$s</a>.', 'give' ),
+						   get_bloginfo( 'url' ),
+						   get_bloginfo( 'admin_email' )
+					   ) . "\n\n";
 			$message .= '{email_access_link}' . "\n\n";
 			$message .= "\n\n";
 			$message .= __( 'Sincerely,', 'give' ) . "\n";

Spacing +51 added lines, -51 removed lines

@@ -11,11 +11,11 @@ 
  */
 
 // Exit if access directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give_Email_Access_Email' ) ) :
+if ( ! class_exists('Give_Email_Access_Email')) :
 
 	/**
 	 * Give_Email_Access_Email
@@ -31,30 +31,30 @@ 
 		 * @since  2.0
 		 */
 		public function init() {
-			$this->load( array(
+			$this->load(array(
 				'id'                           => 'email-access',
-				'label'                        => __( 'Email access', 'give' ),
-				'description'                  => __( 'Sent when donors request access to their donation history using only their email as verification. (See Settings > General > Access Control)', 'give' ),
-				'notification_status'          => give_get_option( 'email_access', 'disabled' ),
+				'label'                        => __('Email access', 'give'),
+				'description'                  => __('Sent when donors request access to their donation history using only their email as verification. (See Settings > General > Access Control)', 'give'),
+				'notification_status'          => give_get_option('email_access', 'disabled'),
 				'form_metabox_setting'         => false,
 				'notification_status_editable' => false,
 				'email_tag_context'            => 'donor',
-				'recipient_group_name'         => __( 'Donor', 'give' ),
-				'default_email_subject'        => sprintf( __( 'Please confirm your email for %s', 'give' ), get_bloginfo( 'url' ) ),
+				'recipient_group_name'         => __('Donor', 'give'),
+				'default_email_subject'        => sprintf(__('Please confirm your email for %s', 'give'), get_bloginfo('url')),
 				'default_email_message'        => $this->get_default_email_message(),
 				'notices' => array(
 					'non-notification-status-editable' => sprintf(
 						'%1$s <a href="%2$s">%3$s &raquo;</a>',
-						__( 'This notification is automatically toggled based on whether the email access is enabled or not.', 'give' ),
-						esc_url( admin_url('edit.php?post_type=give_forms&page=give-settings&tab=general&section=access-control') ),
-						__( 'Edit Setting', 'give' )
+						__('This notification is automatically toggled based on whether the email access is enabled or not.', 'give'),
+						esc_url(admin_url('edit.php?post_type=give_forms&page=give-settings&tab=general&section=access-control')),
+						__('Edit Setting', 'give')
 					)
 				),
-			) );
+			));
 
-			add_filter( "give_{$this->config['id']}_email_notification", array( $this, 'setup_email_notification' ), 10, 2 );
-			add_action( 'give_save_settings_give_settings', array( $this, 'set_notification_status' ), 10, 2 );
-			add_filter( 'give_email_preview_header', array( $this, 'email_preview_header' ), 10, 2 );
+			add_filter("give_{$this->config['id']}_email_notification", array($this, 'setup_email_notification'), 10, 2);
+			add_action('give_save_settings_give_settings', array($this, 'set_notification_status'), 10, 2);
+			add_filter('give_email_preview_header', array($this, 'email_preview_header'), 10, 2);
 		}
 
 
@@ -68,11 +68,11 @@ 
 		 *
 		 * @return string
 		 */
-		public function get_email_subject( $form_id = null ) {
+		public function get_email_subject($form_id = null) {
 			$subject = wp_strip_all_tags(
 				Give_Email_Notification_Util::get_value(
 					$this,
-					Give_Email_Setting_Field::get_prefix( $this, $form_id ) . 'email_subject',
+					Give_Email_Setting_Field::get_prefix($this, $form_id).'email_subject',
 					$form_id,
 					$this->config['default_email_subject']
 				)
@@ -84,14 +84,14 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$subject = apply_filters( 'give_email_access_token_subject', $subject );
+			$subject = apply_filters('give_email_access_token_subject', $subject);
 
 			/**
 			 * Filters the donation notification subject.
 			 *
 			 * @since 2.0
 			 */
-			$subject = apply_filters( "give_{$this->config['id']}_get_email_subject", $subject, $this, $form_id );
+			$subject = apply_filters("give_{$this->config['id']}_get_email_subject", $subject, $this, $form_id);
 
 			return $subject;
 		}
@@ -107,10 +107,10 @@ 
 		 *
 		 * @return string
 		 */
-		public function get_email_message( $form_id = null ) {
+		public function get_email_message($form_id = null) {
 			$message = Give_Email_Notification_Util::get_value(
 				$this,
-				Give_Email_Setting_Field::get_prefix( $this, $form_id ) . 'email_message',
+				Give_Email_Setting_Field::get_prefix($this, $form_id).'email_message',
 				$form_id,
 				$this->config['default_email_message']
 			);
@@ -121,14 +121,14 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$message = apply_filters( 'give_email_access_token_message', $message );
+			$message = apply_filters('give_email_access_token_message', $message);
 
 			/**
 			 * Filter the email message
 			 *
 			 * @since 2.0
 			 */
-			$message = apply_filters( "give_{$this->config['id']}_get_default_email_message", $message, $this, $form_id );
+			$message = apply_filters("give_{$this->config['id']}_get_default_email_message", $message, $this, $form_id);
 
 			return $message;
 		}
@@ -143,7 +143,7 @@ 
 		 * @param int $form_id
 		 * @return array
 		 */
-		public function get_email_attachments( $form_id = null ) {
+		public function get_email_attachments($form_id = null) {
 			/**
 			 * Filters the donation notification email attachments.
 			 * By default, there is no attachment but plugins can hook in to provide one more multiple.
@@ -151,7 +151,7 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$attachments = apply_filters( 'give_admin_donation_notification_attachments', array() );
+			$attachments = apply_filters('give_admin_donation_notification_attachments', array());
 
 			/**
 			 * Filters the donation notification email attachments.
@@ -159,7 +159,7 @@ 
 			 *
 			 * @since 2.0
 			 */
-			$attachments = apply_filters( "give_{$this->config['id']}_get_email_attachments", $attachments, $this, $form_id );
+			$attachments = apply_filters("give_{$this->config['id']}_get_email_attachments", $attachments, $this, $form_id);
 
 			return $attachments;
 		}
@@ -175,14 +175,14 @@ 
 		 */
 		public function get_default_email_message() {
 			$message = sprintf(
-				           __( 'Please click the link to access your donation history on <a target="_blank" href="%1$s">%1$s</a>. If you did not request this email, please contact <a href="mailto:%2$s">%2$s</a>.', 'give' ),
-				           get_bloginfo( 'url' ),
-				           get_bloginfo( 'admin_email' )
-			           ) . "\n\n";
-			$message .= '{email_access_link}' . "\n\n";
+				           __('Please click the link to access your donation history on <a target="_blank" href="%1$s">%1$s</a>. If you did not request this email, please contact <a href="mailto:%2$s">%2$s</a>.', 'give'),
+				           get_bloginfo('url'),
+				           get_bloginfo('admin_email')
+			           )."\n\n";
+			$message .= '{email_access_link}'."\n\n";
 			$message .= "\n\n";
-			$message .= __( 'Sincerely,', 'give' ) . "\n";
-			$message .= get_bloginfo( 'name' ) . "\n";
+			$message .= __('Sincerely,', 'give')."\n";
+			$message .= get_bloginfo('name')."\n";
 
 			/**
 			 * Filter the new donation email message
@@ -191,7 +191,7 @@ 
 			 *
 			 * @param string $message
 			 */
-			return apply_filters( "give_{$this->config['id']}_get_default_email_message", $message, $this );
+			return apply_filters("give_{$this->config['id']}_get_default_email_message", $message, $this);
 		}
 
 
@@ -207,7 +207,7 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$from_name = apply_filters( 'give_donation_from_name', Give()->emails->get_from_name() );
+			$from_name = apply_filters('give_donation_from_name', Give()->emails->get_from_name());
 
 			/**
 			 * Filters the from email.
@@ -215,20 +215,20 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$from_email = apply_filters( 'give_donation_from_address', Give()->emails->get_from_address() );
+			$from_email = apply_filters('give_donation_from_address', Give()->emails->get_from_address());
 
-			Give()->emails->__set( 'from_name', $from_name );
-			Give()->emails->__set( 'from_email', $from_email );
-			Give()->emails->__set( 'heading', apply_filters( 'give_email_access_token_heading', __( 'Confirm Email', 'give' ) ) );
+			Give()->emails->__set('from_name', $from_name);
+			Give()->emails->__set('from_email', $from_email);
+			Give()->emails->__set('heading', apply_filters('give_email_access_token_heading', __('Confirm Email', 'give')));
 
 			/**
 			 * Filters the donation notification email headers.
 			 *
 			 * @since 1.0
 			 */
-			$headers = apply_filters( 'give_admin_donation_notification_headers', Give()->emails->get_headers() );
+			$headers = apply_filters('give_admin_donation_notification_headers', Give()->emails->get_headers());
 
-			Give()->emails->__set( 'headers', $headers );
+			Give()->emails->__set('headers', $headers);
 		}
 
 		/**
@@ -242,8 +242,8 @@ 
 		 *
 		 * @return bool
 		 */
-		public function setup_email_notification( $donor_id, $email ) {
-			$donor = Give()->donors->get_donor_by( 'email', $email );
+		public function setup_email_notification($donor_id, $email) {
+			$donor = Give()->donors->get_donor_by('email', $email);
 			$this->recipient_email = $email;
 
 			// Set email data.
@@ -268,17 +268,17 @@ 
 		 * @param $update_options
 		 * @param $option_name
 		 */
-		public function set_notification_status( $update_options, $option_name ) {
+		public function set_notification_status($update_options, $option_name) {
 			// Get updated settings.
 			$update_options = give_get_settings();
 
 			if (
-				! empty( $update_options['email_access'] )
-				&& ! empty( $update_options[ "{$this->config['id']}_notification" ] )
-				&& $update_options['email_access'] !== $update_options[ "{$this->config['id']}_notification" ]
+				! empty($update_options['email_access'])
+				&& ! empty($update_options["{$this->config['id']}_notification"])
+				&& $update_options['email_access'] !== $update_options["{$this->config['id']}_notification"]
 			) {
-				$update_options[ "{$this->config['id']}_notification" ] = $update_options['email_access'];
-				update_option( $option_name, $update_options );
+				$update_options["{$this->config['id']}_notification"] = $update_options['email_access'];
+				update_option($option_name, $update_options);
 			}
 		}
 
@@ -293,8 +293,8 @@ 
 		 * @param Give_Email_Access_Email $email
 		 * @return string
 		 */
-		public function email_preview_header( $email_preview_header, $email ) {
-			if( $this->config['id'] === $email->config['id'] ) {
+		public function email_preview_header($email_preview_header, $email) {
+			if ($this->config['id'] === $email->config['id']) {
 				$email_preview_header = '';
 			}