WordImpress / Give

includes/class-give-db-meta.php

Doc Comments +2 added lines, -2 removed lines

@@ -279,7 +279,7 @@ 
 	/**
 	 * Rename query clauses for new meta table
 	 *
-	 * @param $clause
+	 * @param string $clause
 	 * @param $filter
 	 *
 	 * @return mixed
@@ -513,7 +513,7 @@ 
 	 * @since 2.0.4
 	 * @access public
 	 *
-	 * @return string
+	 * @return boolean
 	 */
 	public function get_meta_type(){
 		return $this->meta_type;

Spacing +89 added lines, -89 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -77,40 +77,40 @@ 
 		parent::__construct();
 
 		// Bailout.
-		if ( empty( $this->supports ) || ! $this->is_custom_meta_table_active() ) {
+		if (empty($this->supports) || ! $this->is_custom_meta_table_active()) {
 			return;
 		}
 
-		if ( in_array( 'add_post_metadata', $this->supports ) ) {
-			add_filter( 'add_post_metadata', array( $this, '__add_meta' ), 0, 5 );
+		if (in_array('add_post_metadata', $this->supports)) {
+			add_filter('add_post_metadata', array($this, '__add_meta'), 0, 5);
 		}
 
-		if ( in_array( 'get_post_metadata', $this->supports ) ) {
-			add_filter( 'get_post_metadata', array( $this, '__get_meta' ), 0, 4 );
+		if (in_array('get_post_metadata', $this->supports)) {
+			add_filter('get_post_metadata', array($this, '__get_meta'), 0, 4);
 		}
 
-		if ( in_array( 'update_post_metadata', $this->supports ) ) {
-			add_filter( 'update_post_metadata', array( $this, '__update_meta' ), 0, 5 );
+		if (in_array('update_post_metadata', $this->supports)) {
+			add_filter('update_post_metadata', array($this, '__update_meta'), 0, 5);
 		}
 
-		if ( in_array( 'delete_post_metadata', $this->supports ) ) {
-			add_filter( 'delete_post_metadata', array( $this, '__delete_meta' ), 0, 5 );
+		if (in_array('delete_post_metadata', $this->supports)) {
+			add_filter('delete_post_metadata', array($this, '__delete_meta'), 0, 5);
 		}
 
-		if ( in_array( 'posts_where', $this->supports ) ) {
-			add_filter( 'posts_where', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_where', $this->supports)) {
+			add_filter('posts_where', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 
-		if ( in_array( 'posts_join', $this->supports ) ) {
-			add_filter( 'posts_join', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_join', $this->supports)) {
+			add_filter('posts_join', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 
-		if ( in_array( 'posts_groupby', $this->supports ) ) {
-			add_filter( 'posts_groupby', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_groupby', $this->supports)) {
+			add_filter('posts_groupby', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 
-		if ( in_array( 'posts_orderby', $this->supports ) ) {
-			add_filter( 'posts_orderby', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_orderby', $this->supports)) {
+			add_filter('posts_orderby', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 	}
 
@@ -128,16 +128,16 @@ 
 	 * @return  mixed                 Will be an array if $single is false. Will be value of meta data field if $single
 	 *                                is true.
 	 */
-	public function get_meta( $id = 0, $meta_key = '', $single = false ) {
-		$id = $this->sanitize_id( $id );
+	public function get_meta($id = 0, $meta_key = '', $single = false) {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		if ( $this->raw_result ) {
-			if ( ! ( $value = get_metadata( $this->meta_type, $id, $meta_key, false ) ) ) {
+		if ($this->raw_result) {
+			if ( ! ($value = get_metadata($this->meta_type, $id, $meta_key, false))) {
 				$value = '';
 			}
 
@@ -145,7 +145,7 @@ 
 			$this->raw_result = false;
 
 		} else {
-			$value = get_metadata( $this->meta_type, $id, $meta_key, $single );
+			$value = get_metadata($this->meta_type, $id, $meta_key, $single);
 		}
 
 		return $value;
@@ -167,18 +167,18 @@ 
 	 *
 	 * @return  int|bool                  False for failure. True for success.
 	 */
-	public function add_meta( $id = 0, $meta_key = '', $meta_value, $unique = false ) {
-		$id = $this->sanitize_id( $id );
+	public function add_meta($id = 0, $meta_key = '', $meta_value, $unique = false) {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		$meta_id = add_metadata( $this->meta_type, $id, $meta_key, $meta_value, $unique );
+		$meta_id = add_metadata($this->meta_type, $id, $meta_key, $meta_value, $unique);
 
-		if ( $meta_id ) {
-			$this->delete_cache( $id );
+		if ($meta_id) {
+			$this->delete_cache($id);
 		}
 
 		return $meta_id;
@@ -204,18 +204,18 @@ 
 	 *
 	 * @return  int|bool                  False on failure, true if success.
 	 */
-	public function update_meta( $id = 0, $meta_key = '', $meta_value, $prev_value = '' ) {
-		$id = $this->sanitize_id( $id );
+	public function update_meta($id = 0, $meta_key = '', $meta_value, $prev_value = '') {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		$meta_id = update_metadata( $this->meta_type, $id, $meta_key, $meta_value, $prev_value );
+		$meta_id = update_metadata($this->meta_type, $id, $meta_key, $meta_value, $prev_value);
 
-		if ( $meta_id ) {
-			$this->delete_cache( $id );
+		if ($meta_id) {
+			$this->delete_cache($id);
 		}
 
 		return $meta_id;
@@ -238,18 +238,18 @@ 
 	 *
 	 * @return  bool                  False for failure. True for success.
 	 */
-	public function delete_meta( $id = 0, $meta_key = '', $meta_value = '', $delete_all = '' ) {
-		$id = $this->sanitize_id( $id );
+	public function delete_meta($id = 0, $meta_key = '', $meta_value = '', $delete_all = '') {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		$is_meta_deleted = delete_metadata( $this->meta_type, $id, $meta_key, $meta_value, $delete_all );
+		$is_meta_deleted = delete_metadata($this->meta_type, $id, $meta_key, $meta_value, $delete_all);
 
-		if ( $is_meta_deleted ) {
-			$this->delete_cache( $id );
+		if ($is_meta_deleted) {
+			$this->delete_cache($id);
 		}
 
 		return $is_meta_deleted;
@@ -266,10 +266,10 @@ 
 	 *
 	 * @return string
 	 */
-	public function __rename_meta_table_name_in_query( $clause, $wp_query ) {
+	public function __rename_meta_table_name_in_query($clause, $wp_query) {
 		// Add new table to sql query.
-		if ( $this->is_post_type_query( $wp_query ) && ! empty( $wp_query->meta_query->queries ) ) {
-			$clause = $this->__rename_meta_table_name( $clause, current_filter() );
+		if ($this->is_post_type_query($wp_query) && ! empty($wp_query->meta_query->queries)) {
+			$clause = $this->__rename_meta_table_name($clause, current_filter());
 		}
 
 		return $clause;
@@ -284,39 +284,39 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function __rename_meta_table_name( $clause, $filter ){
+	public function __rename_meta_table_name($clause, $filter) {
 		global $wpdb;
 
-		$clause = str_replace( "{$wpdb->postmeta}.post_id", "{$this->table_name}.{$this->meta_type}_id", $clause );
-		$clause = str_replace( $wpdb->postmeta, $this->table_name, $clause );
+		$clause = str_replace("{$wpdb->postmeta}.post_id", "{$this->table_name}.{$this->meta_type}_id", $clause);
+		$clause = str_replace($wpdb->postmeta, $this->table_name, $clause);
 
-		switch( $filter ) {
+		switch ($filter) {
 			case 'posts_join':
-				$joins = array( 'INNER JOIN', 'LEFT JOIN' );
+				$joins = array('INNER JOIN', 'LEFT JOIN');
 
-				foreach ( $joins as $join ) {
-					if( false !== strpos( $clause, $join ) ) {
-						$clause = explode( $join, $clause );
+				foreach ($joins as $join) {
+					if (false !== strpos($clause, $join)) {
+						$clause = explode($join, $clause);
 
-						foreach ( $clause as $key => $clause_part ) {
-							if( empty( $clause_part ) ) {
+						foreach ($clause as $key => $clause_part) {
+							if (empty($clause_part)) {
 								continue;
 							}
 
-							preg_match( '/' . $wpdb->prefix . 'give_' . $this->meta_type . 'meta AS (.*) ON/', $clause_part, $alias_table_name );
+							preg_match('/'.$wpdb->prefix.'give_'.$this->meta_type.'meta AS (.*) ON/', $clause_part, $alias_table_name);
 
-							if( isset( $alias_table_name[1] ) ) {
-								$clause[$key] = str_replace( "{$alias_table_name[1]}.post_id", "{$alias_table_name[1]}.{$this->meta_type}_id", $clause_part );
+							if (isset($alias_table_name[1])) {
+								$clause[$key] = str_replace("{$alias_table_name[1]}.post_id", "{$alias_table_name[1]}.{$this->meta_type}_id", $clause_part);
 							}
 						}
 
-						$clause = implode( "{$join} ", $clause );
+						$clause = implode("{$join} ", $clause);
 					}
 				}
 				break;
 
 			case 'posts_where':
-				$clause = str_replace( array( 'mt2.post_id', 'mt1.post_id' ), array( "mt2.{$this->meta_type}_id", "mt1.{$this->meta_type}_id" ), $clause );
+				$clause = str_replace(array('mt2.post_id', 'mt1.post_id'), array("mt2.{$this->meta_type}_id", "mt1.{$this->meta_type}_id"), $clause);
 				break;
 		}
 
@@ -334,19 +334,19 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function is_post_type_query( $wp_query ) {
+	protected function is_post_type_query($wp_query) {
 		$status = false;
 
 		// Check if it is payment query.
-		if ( ! empty( $wp_query->query['post_type'] ) ) {
+		if ( ! empty($wp_query->query['post_type'])) {
 			if (
-				is_string( $wp_query->query['post_type'] ) &&
+				is_string($wp_query->query['post_type']) &&
 				$this->post_type === $wp_query->query['post_type']
 			) {
 				$status = true;
 			} elseif (
-				is_array( $wp_query->query['post_type'] ) &&
-				in_array( $this->post_type, $wp_query->query['post_type'] )
+				is_array($wp_query->query['post_type']) &&
+				in_array($this->post_type, $wp_query->query['post_type'])
 			) {
 				$status = true;
 			}
@@ -365,8 +365,8 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function is_valid_post_type( $ID ) {
-		return $ID && ( $this->post_type === get_post_type( $ID ) );
+	protected function is_valid_post_type($ID) {
+		return $ID && ($this->post_type === get_post_type($ID));
 	}
 
 	/**
@@ -392,8 +392,8 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_cache( $id, $meta_type = '' ) {
-		$meta_type = empty( $meta_type ) ? $this->meta_type : $meta_type;
+	private function delete_cache($id, $meta_type = '') {
+		$meta_type = empty($meta_type) ? $this->meta_type : $meta_type;
 
 		$group = array(
 			// 'form'    => 'give-forms',
@@ -403,8 +403,8 @@ 
 			// 'log'     => 'give-logs',
 		);
 
-		if ( array_key_exists( $meta_type, $group ) ) {
-			Give_Cache::delete_group( $id, $group[ $meta_type ] );
+		if (array_key_exists($meta_type, $group)) {
+			Give_Cache::delete_group($id, $group[$meta_type]);
 		}
 	}
 
@@ -419,8 +419,8 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function __call( $name, $arguments ) {
-		switch ( $name ) {
+	public function __call($name, $arguments) {
+		switch ($name) {
 			case '__add_meta':
 				$this->check = $arguments[0];
 				$id          = $arguments[1];
@@ -429,11 +429,11 @@ 
 				$unique      = $arguments[4];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
-				return $this->add_meta( $id, $meta_key, $meta_value, $unique );
+				return $this->add_meta($id, $meta_key, $meta_value, $unique);
 
 			case '__get_meta':
 				$this->check = $arguments[0];
@@ -442,13 +442,13 @@ 
 				$single      = $arguments[3];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
 				$this->raw_result = true;
 
-				return $this->get_meta( $id, $meta_key, $single );
+				return $this->get_meta($id, $meta_key, $single);
 
 			case '__update_meta':
 				$this->check = $arguments[0];
@@ -457,11 +457,11 @@ 
 				$meta_value  = $arguments[3];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
-				return $this->update_meta( $id, $meta_key, $meta_value );
+				return $this->update_meta($id, $meta_key, $meta_value);
 
 			case '__delete_meta':
 				$this->check = $arguments[0];
@@ -471,11 +471,11 @@ 
 				$delete_all  = $arguments[3];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
-				return $this->delete_meta( $id, $meta_key, $meta_value, $delete_all );
+				return $this->delete_meta($id, $meta_key, $meta_value, $delete_all);
 		}
 	}
 
@@ -500,10 +500,10 @@ 
 			KEY meta_key (meta_key({$this->min_index_length}))
 			) {$charset_collate};";
 
-		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
-		dbDelta( $sql );
+		require_once(ABSPATH.'wp-admin/includes/upgrade.php');
+		dbDelta($sql);
 
-		update_option( $this->table_name . '_db_version', $this->version );
+		update_option($this->table_name.'_db_version', $this->version);
 	}
 
 
@@ -515,7 +515,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_meta_type(){
+	public function get_meta_type() {
 		return $this->meta_type;
 	}
 
@@ -529,12 +529,12 @@ 
 	 *
 	 * @return  bool  False for failure. True for success.
 	 */
-	public function delete_all_meta( $id = 0 ) {
+	public function delete_all_meta($id = 0) {
 		global $wpdb;
-		$status = $wpdb->delete( $this->table_name, array( "{$this->meta_type}_id" => $id ), array( '%d' ) );
+		$status = $wpdb->delete($this->table_name, array("{$this->meta_type}_id" => $id), array('%d'));
 
-		if ( $status ) {
-			$this->delete_cache( $id, $this->meta_type );
+		if ($status) {
+			$this->delete_cache($id, $this->meta_type);
 		}
 
 		return $status;

includes/class-give-db.php

Spacing +61 added lines, -61 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -72,8 +72,8 @@ 
 	 * @access public
 	 */
 	public function __construct() {
-		if( is_multisite() ) {
-			add_action( 'switch_blog', array( $this, 'handle_switch_blog' ), 10, 2 );
+		if (is_multisite()) {
+			add_action('switch_blog', array($this, 'handle_switch_blog'), 10, 2);
 		}
 	}
 
@@ -111,16 +111,16 @@ 
 	 *
 	 * @return object
 	 */
-	public function get( $row_id ) {
+	public function get($row_id) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Bailout.
-		if ( empty( $row_id ) ) {
+		if (empty($row_id)) {
 			return null;
 		}
 
-		return $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id ) );
+		return $wpdb->get_row($wpdb->prepare("SELECT * FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -134,18 +134,18 @@ 
 	 *
 	 * @return object
 	 */
-	public function get_by( $column, $row_id ) {
+	public function get_by($column, $row_id) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Bailout.
-		if ( empty( $column ) || empty( $row_id ) ) {
+		if (empty($column) || empty($row_id)) {
 			return null;
 		}
 
-		$column = esc_sql( $column );
+		$column = esc_sql($column);
 
-		return $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $this->table_name WHERE $column = %s LIMIT 1;", $row_id ) );
+		return $wpdb->get_row($wpdb->prepare("SELECT * FROM $this->table_name WHERE $column = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -159,18 +159,18 @@ 
 	 *
 	 * @return string      Column value.
 	 */
-	public function get_column( $column, $row_id ) {
+	public function get_column($column, $row_id) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Bailout.
-		if ( empty( $column ) || empty( $row_id ) ) {
+		if (empty($column) || empty($row_id)) {
 			return null;
 		}
 
-		$column = esc_sql( $column );
+		$column = esc_sql($column);
 
-		return $wpdb->get_var( $wpdb->prepare( "SELECT $column FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id ) );
+		return $wpdb->get_var($wpdb->prepare("SELECT $column FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -185,19 +185,19 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_column_by( $column, $column_where, $column_value ) {
+	public function get_column_by($column, $column_where, $column_value) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Bailout.
-		if ( empty( $column ) || empty( $column_where ) || empty( $column_value ) ) {
+		if (empty($column) || empty($column_where) || empty($column_value)) {
 			return null;
 		}
 
-		$column_where = esc_sql( $column_where );
-		$column       = esc_sql( $column );
+		$column_where = esc_sql($column_where);
+		$column       = esc_sql($column);
 
-		return $wpdb->get_var( $wpdb->prepare( "SELECT $column FROM $this->table_name WHERE $column_where = %s LIMIT 1;", $column_value ) );
+		return $wpdb->get_var($wpdb->prepare("SELECT $column FROM $this->table_name WHERE $column_where = %s LIMIT 1;", $column_value));
 	}
 
 	/**
@@ -211,12 +211,12 @@ 
 	 *
 	 * @return int
 	 */
-	public function insert( $data, $type = '' ) {
+	public function insert($data, $type = '') {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Set default values.
-		$data = wp_parse_args( $data, $this->get_column_defaults() );
+		$data = wp_parse_args($data, $this->get_column_defaults());
 
 		/**
 		 * Fires before inserting data to the database.
@@ -225,7 +225,7 @@ 
 		 *
 		 * @param array $data
 		 */
-		do_action( "give_pre_insert_{$type}", $data );
+		do_action("give_pre_insert_{$type}", $data);
 
 		// Initialise column format array
 		$column_formats = $this->get_columns();
@@ -234,13 +234,13 @@ 
 		// $data = array_change_key_case( $data );
 
 		// White list columns
-		$data = array_intersect_key( $data, $column_formats );
+		$data = array_intersect_key($data, $column_formats);
 
 		// Reorder $column_formats to match the order of columns given in $data
-		$data_keys      = array_keys( $data );
-		$column_formats = array_merge( array_flip( $data_keys ), $column_formats );
+		$data_keys      = array_keys($data);
+		$column_formats = array_merge(array_flip($data_keys), $column_formats);
 
-		$wpdb->insert( $this->table_name, $data, $column_formats );
+		$wpdb->insert($this->table_name, $data, $column_formats);
 
 		/**
 		 * Fires after inserting data to the database.
@@ -250,7 +250,7 @@ 
 		 * @param int   $insert_id
 		 * @param array $data
 		 */
-		do_action( "give_post_insert_{$type}", $wpdb->insert_id, $data );
+		do_action("give_post_insert_{$type}", $wpdb->insert_id, $data);
 
 		return $wpdb->insert_id;
 	}
@@ -267,18 +267,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function update( $row_id, $data = array(), $where = '' ) {
+	public function update($row_id, $data = array(), $where = '') {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Row ID must be positive integer
-		$row_id = absint( $row_id );
+		$row_id = absint($row_id);
 
-		if ( empty( $row_id ) ) {
+		if (empty($row_id)) {
 			return false;
 		}
 
-		if ( empty( $where ) ) {
+		if (empty($where)) {
 			$where = $this->primary_key;
 		}
 
@@ -286,16 +286,16 @@ 
 		$column_formats = $this->get_columns();
 
 		// Force fields to lower case
-		$data = array_change_key_case( $data );
+		$data = array_change_key_case($data);
 
 		// White list columns
-		$data = array_intersect_key( $data, $column_formats );
+		$data = array_intersect_key($data, $column_formats);
 
 		// Reorder $column_formats to match the order of columns given in $data
-		$data_keys      = array_keys( $data );
-		$column_formats = array_merge( array_flip( $data_keys ), $column_formats );
+		$data_keys      = array_keys($data);
+		$column_formats = array_merge(array_flip($data_keys), $column_formats);
 
-		if ( false === $wpdb->update( $this->table_name, $data, array( $where => $row_id ), $column_formats ) ) {
+		if (false === $wpdb->update($this->table_name, $data, array($where => $row_id), $column_formats)) {
 			return false;
 		}
 
@@ -312,18 +312,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function delete( $row_id = 0 ) {
+	public function delete($row_id = 0) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Row ID must be positive integer
-		$row_id = absint( $row_id );
+		$row_id = absint($row_id);
 
-		if ( empty( $row_id ) ) {
+		if (empty($row_id)) {
 			return false;
 		}
 
-		if ( false === $wpdb->query( $wpdb->prepare( "DELETE FROM $this->table_name WHERE $this->primary_key = %d", $row_id ) ) ) {
+		if (false === $wpdb->query($wpdb->prepare("DELETE FROM $this->table_name WHERE $this->primary_key = %d", $row_id))) {
 			return false;
 		}
 
@@ -340,13 +340,13 @@ 
 	 *
 	 * @return bool          If the table name exists.
 	 */
-	public function table_exists( $table ) {
+	public function table_exists($table) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		$table = sanitize_text_field( $table );
+		$table = sanitize_text_field($table);
 
-		return $wpdb->get_var( $wpdb->prepare( "SHOW TABLES LIKE '%s'", $table ) ) === $table;
+		return $wpdb->get_var($wpdb->prepare("SHOW TABLES LIKE '%s'", $table)) === $table;
 	}
 
 	/**
@@ -360,16 +360,16 @@ 
 	 *
 	 * @return bool
 	 */
-	public function does_column_exist( $column_name ) {
+	public function does_column_exist($column_name) {
 
 		global $wpdb;
 
-		$column = $wpdb->get_results( $wpdb->prepare(
+		$column = $wpdb->get_results($wpdb->prepare(
 			"SELECT * FROM INFORMATION_SCHEMA.COLUMNS WHERE TABLE_SCHEMA = %s AND TABLE_NAME = %s AND COLUMN_NAME = %s ",
 			DB_NAME, $this->table_name, $column_name
-		) );
+		));
 
-		if ( ! empty( $column ) ) {
+		if ( ! empty($column)) {
 			return true;
 		}
 
@@ -385,7 +385,7 @@ 
 	 * @return bool Returns if the customers table was installed and upgrade routine run.
 	 */
 	public function installed() {
-		return $this->table_exists( $this->table_name );
+		return $this->table_exists($this->table_name);
 	}
 
 	/**
@@ -395,8 +395,8 @@ 
 	 * @access public
 	 */
 	public function register_table() {
-		$current_version = get_option( $this->table_name . '_db_version' );
-		if ( ! $current_version || version_compare( $current_version, $this->version, '<' ) ) {
+		$current_version = get_option($this->table_name.'_db_version');
+		if ( ! $current_version || version_compare($current_version, $this->version, '<')) {
 			$this->create_table();
 		}
 	}
@@ -421,23 +421,23 @@ 
 	 *
 	 * @return int|bool                The normalized log ID or false if it's found to not be valid.
 	 */
-	public function sanitize_id( $id ) {
-		if ( ! is_numeric( $id ) ) {
+	public function sanitize_id($id) {
+		if ( ! is_numeric($id)) {
 			return false;
 		}
 
 		$id = (int) $id;
 
 		// We were given a non positive number.
-		if ( absint( $id ) !== $id ) {
+		if (absint($id) !== $id) {
 			return false;
 		}
 
-		if ( empty( $id ) ) {
+		if (empty($id)) {
 			return false;
 		}
 
-		return absint( $id );
+		return absint($id);
 
 	}
 
@@ -451,23 +451,23 @@ 
 	 * @param $new_blog_id
 	 * @param $prev_blog_id
 	 */
-	public function handle_switch_blog( $new_blog_id, $prev_blog_id ) {
+	public function handle_switch_blog($new_blog_id, $prev_blog_id) {
 		global $wpdb;
 
 		// Bailout.
-		if ( $new_blog_id === $prev_blog_id ) {
+		if ($new_blog_id === $prev_blog_id) {
 			return;
 		}
 
 
 		$this->table_name = str_replace(
-			1 != $prev_blog_id ? $wpdb->get_blog_prefix( $prev_blog_id ) : $wpdb->base_prefix,
-			1 != $new_blog_id ? $wpdb->get_blog_prefix( $new_blog_id ) : $wpdb->base_prefix,
+			1 != $prev_blog_id ? $wpdb->get_blog_prefix($prev_blog_id) : $wpdb->base_prefix,
+			1 != $new_blog_id ? $wpdb->get_blog_prefix($new_blog_id) : $wpdb->base_prefix,
 			$this->table_name
 		);
 
-		if ( $this instanceof Give_DB_Meta ) {
-			$wpdb->{$this->get_meta_type() . 'meta'} = $this->table_name;
+		if ($this instanceof Give_DB_Meta) {
+			$wpdb->{$this->get_meta_type().'meta'} = $this->table_name;
 		}
 
 	}

includes/admin/class-addon-activation-banner.php

Doc Comments +1 added lines, -1 removed lines

@@ -505,7 +505,7 @@
 	 *
 	 * @param string $main_file Plugin Main File.
 	 *
-	 * @return bool|mixed|string
+	 * @return string
 	 */
 	public function get_plugin_folder_name( $main_file ) {
 		// Remove plugin file and get the Add-on's folder name only.

Spacing +104 added lines, -104 removed lines

@@ -7,7 +7,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -35,13 +35,13 @@ 
 	 *                               'testing'           => false, // (required) Never leave as "true" in production!!!
 	 *                               }
 	 */
-	function __construct( $_banner_details ) {
+	function __construct($_banner_details) {
 		global $give_addons, $pagenow;
 
 		// Append add-on information to the global variable.
 		$give_addons[] = $_banner_details;
 
-		if ( 'plugins.php' === $pagenow ) {
+		if ('plugins.php' === $pagenow) {
 
 			// Get the current user.
 			$current_user  = wp_get_current_user();
@@ -54,9 +54,9 @@ 
 			$this->add_addon_activate_meta();
 
 			// Check if notice callback is already hooked.
-			if ( ! $this->is_banner_notice_hooked() ) {
+			if ( ! $this->is_banner_notice_hooked()) {
 				// If multiple add-on are activated then show activation banner in tab view.
-				add_action( 'admin_notices', array( $this, 'addon_activation_banner_notices' ), 10 );
+				add_action('admin_notices', array($this, 'addon_activation_banner_notices'), 10);
 			}
 		}
 	}
@@ -69,8 +69,8 @@ 
 	 *
 	 * @return string
 	 */
-	public static function get_banner_user_meta_key( $addon_banner_key ) {
-		$addon_slug = sanitize_text_field( $addon_banner_key );
+	public static function get_banner_user_meta_key($addon_banner_key) {
+		$addon_slug = sanitize_text_field($addon_banner_key);
 
 		return "give_addon_{$addon_slug}_active_by_user";
 	}
@@ -85,24 +85,24 @@ 
 	 */
 	public function init() {
 		// Get the current page to add the notice to
-		add_action( 'current_screen', array( $this, 'give_addon_notice_ignore' ) );
+		add_action('current_screen', array($this, 'give_addon_notice_ignore'));
 
 		// Get the Give add-ons.
 		$give_addons = $this->get_plugin_file_names();
 
-		if ( ! empty( $give_addons ) ) {
+		if ( ! empty($give_addons)) {
 
 			// Go through each of the add-on and hook deactivate action.
-			foreach ( $give_addons as $addon_name => $give_addon ) {
+			foreach ($give_addons as $addon_name => $give_addon) {
 
 				// Testing?
-				if ( true === $give_addon['testing'] ) {
-					$nag_meta_key = 'give_addon_activation_ignore_' . $addon_name;
-					delete_user_meta( $this->user_id, $nag_meta_key );
+				if (true === $give_addon['testing']) {
+					$nag_meta_key = 'give_addon_activation_ignore_'.$addon_name;
+					delete_user_meta($this->user_id, $nag_meta_key);
 				}
 
 				// Add deactivate hook.
-				add_action( 'deactivate_' . $give_addon['plugin_main_file'], array( $this, 'remove_addon_activate_meta' ) );
+				add_action('deactivate_'.$give_addon['plugin_main_file'], array($this, 'remove_addon_activate_meta'));
 			}
 		}
 	}
@@ -118,29 +118,29 @@ 
 		global $give_addons;
 
 		// Get recently activated plugins.
-		$active_plugins = get_option( 'active_plugins' );
+		$active_plugins = get_option('active_plugins');
 
 		$file_names = array();
 
-		if ( empty( $give_addons ) ) {
+		if (empty($give_addons)) {
 			return $file_names;
 		}
 
 		// Go through each addon and get the plugin file url.
-		foreach ( $give_addons as $give_addon ) {
+		foreach ($give_addons as $give_addon) {
 			$file_name = '';
-			$file_path = explode( '/plugins/', $give_addon['file'] );
-			if ( $file_path ) {
-				$file_path = array_pop( $file_path );
-				$file_name = current( explode( '/', $file_path ) );
+			$file_path = explode('/plugins/', $give_addon['file']);
+			if ($file_path) {
+				$file_path = array_pop($file_path);
+				$file_name = current(explode('/', $file_path));
 			}
 
-			if ( ! empty( $file_name ) ) {
-				foreach ( $active_plugins as $plugin ) {
-					if ( false !== strpos( $plugin, $file_name ) ) {
-						$add_on_key                     = sanitize_title( $give_addon['name'] );
+			if ( ! empty($file_name)) {
+				foreach ($active_plugins as $plugin) {
+					if (false !== strpos($plugin, $file_name)) {
+						$add_on_key                     = sanitize_title($give_addon['name']);
 						$give_addon['plugin_main_file'] = $plugin; // Include plugin file.
-						$file_names[ $add_on_key ]      = $give_addon;
+						$file_names[$add_on_key]      = $give_addon;
 						break;
 					}
 				}
@@ -160,16 +160,16 @@ 
 		// Get all activated add-ons.
 		$give_addons = $this->get_plugin_file_names();
 
-		if ( ! empty( $give_addons ) ) {
+		if ( ! empty($give_addons)) {
 
 			// Go through rach add-ons and add meta data.
-			foreach ( $give_addons as $banner_addon_name => $addon ) {
+			foreach ($give_addons as $banner_addon_name => $addon) {
 
 				// User meta key.
-				$user_id = __give_get_active_by_user_meta( $banner_addon_name );
+				$user_id = __give_get_active_by_user_meta($banner_addon_name);
 
-				if ( ! $user_id ) {
-					update_option( self::get_banner_user_meta_key( $banner_addon_name ), $this->user_id, '' );
+				if ( ! $user_id) {
+					update_option(self::get_banner_user_meta_key($banner_addon_name), $this->user_id, '');
 				}
 			}
 		}
@@ -186,14 +186,14 @@ 
 		global $wp_filter;
 		$notice_already_hooked = false;
 
-		if ( isset( $wp_filter['admin_notices']->callbacks[10] ) ) {
+		if (isset($wp_filter['admin_notices']->callbacks[10])) {
 			// Get all of the hooks.
-			$admin_notice_callbacks = array_keys( $wp_filter['admin_notices']->callbacks[10] );
+			$admin_notice_callbacks = array_keys($wp_filter['admin_notices']->callbacks[10]);
 
-			if ( ! empty( $admin_notice_callbacks ) ) {
-				foreach ( $admin_notice_callbacks as $key ) {
+			if ( ! empty($admin_notice_callbacks)) {
+				foreach ($admin_notice_callbacks as $key) {
 					//If the key is found in your string, set $found to true
-					if ( false !== strpos( $key, 'addon_activation_banner_notices' ) ) {
+					if (false !== strpos($key, 'addon_activation_banner_notices')) {
 						$notice_already_hooked = true;
 					}
 				}
@@ -212,7 +212,7 @@ 
 		global $pagenow, $give_addons;
 
 		// Bailout.
-		if ( 'plugins.php' !== $pagenow ) {
+		if ('plugins.php' !== $pagenow) {
 			return false;
 		}
 
@@ -224,55 +224,55 @@ 
 		$latest_addon     = array();
 
 		// Get the plugin folder name, because many give-addon not sending proper plugin_file.
-		if ( ! empty( $recent_activated ) ) {
-			foreach ( $recent_activated as $recent_addon ) {
+		if ( ! empty($recent_activated)) {
+			foreach ($recent_activated as $recent_addon) {
 				// Get the add-on folder name.
-				$latest_addon[] = substr( $recent_addon, 0, strpos( $recent_addon, '/' ) );
+				$latest_addon[] = substr($recent_addon, 0, strpos($recent_addon, '/'));
 			}
 		}
 
 		// Go through each of the give add-on.
-		foreach ( $give_addons as $addon ) {
-			$addon_sanitized_name = sanitize_title( $addon['name'] );
+		foreach ($give_addons as $addon) {
+			$addon_sanitized_name = sanitize_title($addon['name']);
 
 			// Get the add-on dismiss status.
-			$add_on_state = get_user_meta( $this->user_id, "give_addon_activation_ignore_{$addon_sanitized_name}", true );
+			$add_on_state = get_user_meta($this->user_id, "give_addon_activation_ignore_{$addon_sanitized_name}", true);
 
 			// Get the option key.
-			$activate_by_user = (int) __give_get_active_by_user_meta( $addon_sanitized_name );
+			$activate_by_user = (int) __give_get_active_by_user_meta($addon_sanitized_name);
 
 			// Remove plugin file and get the Add-on's folder name only.
-			$file_path = $this->get_plugin_folder_name( $addon['file'] );
+			$file_path = $this->get_plugin_folder_name($addon['file']);
 
 			// If add-on were never dismissed.
-			if ( 'true' !== $add_on_state && $this->user_id === $activate_by_user ) {
-				if ( ! empty( $latest_addon ) && ( in_array( $file_path, $latest_addon, true ) || empty( $latest_addon ) ) ) {
+			if ('true' !== $add_on_state && $this->user_id === $activate_by_user) {
+				if ( ! empty($latest_addon) && (in_array($file_path, $latest_addon, true) || empty($latest_addon))) {
 					$addon_to_display[] = $addon;
 				}
 			}
 		}
 
-		if ( ! empty( $addon_to_display ) ) {
+		if ( ! empty($addon_to_display)) {
 			ob_start();
 
 			// Output inline styles here because there's no reason
 			// to enqueued them after the alert is dismissed.
 			$this->print_css_js();
 			?>
-			<div class="<?php echo ( 1 !== count( $addon_to_display ) ) ? 'give-alert-tab-wrapper' : ''; ?> updated give-addon-alert give-notice">
+			<div class="<?php echo (1 !== count($addon_to_display)) ? 'give-alert-tab-wrapper' : ''; ?> updated give-addon-alert give-notice">
 				<?php
 				// If multiple add-on are activated.
-				if ( 1 !== count( $addon_to_display ) ) {
+				if (1 !== count($addon_to_display)) {
 					?>
 					<div class="give-vertical-tab">
 						<div class="give-addon-tab-list">
 							<ul class="give-alert-addon-list">
 								<?php
 								$is_first = true;
-								foreach ( $addon_to_display as $banner ) {
+								foreach ($addon_to_display as $banner) {
 									?>
-									<li class="give-tab-list<?php echo ( true === $is_first ) ? ' active' : ''; ?>" id="give-addon-<?php echo esc_attr( basename( $banner['file'], '.php' ) ); ?>">
-										<a href="#"><?php echo esc_html( $banner['name'] ); ?></a>
+									<li class="give-tab-list<?php echo (true === $is_first) ? ' active' : ''; ?>" id="give-addon-<?php echo esc_attr(basename($banner['file'], '.php')); ?>">
+										<a href="#"><?php echo esc_html($banner['name']); ?></a>
 									</li>
 									<?php
 									$is_first = false;
@@ -282,11 +282,11 @@ 
 							</ul>
 						</div>
 						<div class="give-right-side-block">
-							<?php foreach ( $addon_to_display as $banner ) : ?>
-								<div class="give-tab-details <?php echo ( true === $is_first ) ? ' active' : ''; ?> " id="give-addon-<?php echo esc_attr( basename( $banner['file'], '.php' ) ); ?>">
+							<?php foreach ($addon_to_display as $banner) : ?>
+								<div class="give-tab-details <?php echo (true === $is_first) ? ' active' : ''; ?> " id="give-addon-<?php echo esc_attr(basename($banner['file'], '.php')); ?>">
 									<?php
 										// Render single add banner.
-										$this->render_single_addon_banner( $banner, false );
+										$this->render_single_addon_banner($banner, false);
 										$is_first = false;
 									?>
 								</div>
@@ -295,7 +295,7 @@ 
 					</div>
 					<?php
 				} else {
-					$this->render_single_addon_banner( $addon_to_display[0], true );
+					$this->render_single_addon_banner($addon_to_display[0], true);
 				}
 				?>
 			</div>
@@ -303,12 +303,12 @@ 
 			$notice_html = ob_get_clean();
 
 			// Register notice.
-			Give()->notices->register_notice( array(
+			Give()->notices->register_notice(array(
 				'id'               => 'give_add_on_activation_notice',
 				'type'             => 'updated',
 				'description_html' => $notice_html,
 				'show'             => true,
-			) );
+			));
 		}
 	}
 
@@ -320,7 +320,7 @@ 
 	 * @param array $banner_arr Banner options.
 	 * @param bool  $is_single  Is single.
 	 */
-	private function render_single_addon_banner( $banner_arr, $is_single ) {
+	private function render_single_addon_banner($banner_arr, $is_single) {
 		// Get all give add-on.
 		$give_addons = give_get_plugins();
 
@@ -328,80 +328,80 @@ 
 		$plugin_file = $banner_arr['file'];
 
 		// Get the plugin main file.
-		foreach ( $give_addons as $main_file => $addon ) {
+		foreach ($give_addons as $main_file => $addon) {
 			// Plugin should be activated.
-			if ( ! is_plugin_active( $main_file ) ) {
+			if ( ! is_plugin_active($main_file)) {
 				continue;
 			}
 
 			if (
-				isset( $banner_arr['name'] )
+				isset($banner_arr['name'])
 				&& 'add-on' === $addon['Type']
-				&& $this->get_plugin_folder_name( $main_file ) === $this->get_plugin_folder_name( $plugin_file )
+				&& $this->get_plugin_folder_name($main_file) === $this->get_plugin_folder_name($plugin_file)
 			) {
-				$plugin_file = WP_PLUGIN_DIR . '/' . $main_file;
+				$plugin_file = WP_PLUGIN_DIR.'/'.$main_file;
 				break;
 			}
 		}
 
 		// Create dismiss URL.
 		$dismiss_url = $is_single
-			? admin_url( 'plugins.php?give_addon_activation_ignore=1&give_addon=' . sanitize_title( $banner_arr['name'] ) )
-			: admin_url( 'plugins.php?give_addon_activation_ignore=1&give_addon=all' );
+			? admin_url('plugins.php?give_addon_activation_ignore=1&give_addon='.sanitize_title($banner_arr['name']))
+			: admin_url('plugins.php?give_addon_activation_ignore=1&give_addon=all');
 
 		// Get the add-on details.
-		$plugin_data = get_plugin_data( $plugin_file );
+		$plugin_data = get_plugin_data($plugin_file);
 		?>
-		<img src="<?php echo esc_url( GIVE_PLUGIN_URL . 'assets/dist/images/give-icon-full-circle.svg' ); ?>" class="give-logo" />
+		<img src="<?php echo esc_url(GIVE_PLUGIN_URL.'assets/dist/images/give-icon-full-circle.svg'); ?>" class="give-logo" />
 		<div class="give-alert-message">
 			<h3>
 				<?php
 				printf(
 					/* translators: %s: Add-on name */
 					'%s<span>%s</span>',
-					__( 'New Give Add-on Activated: ', 'give' ),
-					esc_html( $banner_arr['name'] )
+					__('New Give Add-on Activated: ', 'give'),
+					esc_html($banner_arr['name'])
 				);
 				?>
 			</h3>
-			<a href="<?php echo esc_url( $dismiss_url ); ?>" class="dismiss">
+			<a href="<?php echo esc_url($dismiss_url); ?>" class="dismiss">
 				<span class="dashicons dashicons-dismiss"></span>
 			</a>
 			<div class="alert-actions">
 				<?php
 				//Point them to your settings page.
-				if ( ! empty( $plugin_data['Description'] ) ) {
+				if ( ! empty($plugin_data['Description'])) {
 					?>
 					<span class="give-addon-description">
-					<em><?php echo esc_html( strip_tags( $plugin_data['Description'] ) ); ?></em></span><br />
+					<em><?php echo esc_html(strip_tags($plugin_data['Description'])); ?></em></span><br />
 					<?php
 				}
-				if ( isset( $banner_arr['settings_url'] ) ) {
+				if (isset($banner_arr['settings_url'])) {
 					printf(
 						'<a href="%s"><span class="dashicons dashicons-admin-settings"></span>%s</a>',
-						esc_url( $banner_arr['settings_url'] ),
-						esc_html__( 'Go to Settings', 'give' )
+						esc_url($banner_arr['settings_url']),
+						esc_html__('Go to Settings', 'give')
 					);
 				}
 				// Show them how to configure the Addon.
-				if ( isset( $banner_arr['documentation_url'] ) ) {
+				if (isset($banner_arr['documentation_url'])) {
 					printf(
 						'<a href="%s" target="_blank"><span class="dashicons dashicons-media-text"></span>%s</a>',
-						esc_url( $banner_arr['documentation_url'] ),
+						esc_url($banner_arr['documentation_url']),
 						sprintf(
 							/* translators: %s: Add-on name */
-							esc_html__( 'Documentation: %s Add-on', 'give' ),
-							esc_html( $banner_arr['name'] )
+							esc_html__('Documentation: %s Add-on', 'give'),
+							esc_html($banner_arr['name'])
 						)
 					);
 				}
 
 				//Let them signup for plugin updates
-				if ( isset( $banner_arr['support_url'] ) ) {
+				if (isset($banner_arr['support_url'])) {
 					printf(
 						'<a href="%s" target="_blank"><span class="dashicons dashicons-sos"></span>%s</a>',
-						esc_url( $banner_arr['support_url'] ),
-						esc_html__( 'Get Support', 'give' )
+						esc_url($banner_arr['support_url']),
+						esc_html__('Get Support', 'give')
 					);
 				}
 				?>
@@ -424,30 +424,30 @@ 
 		 * See here: http://codex.wordpress.org/Function_Reference/add_user_meta
 		 */
 		if (
-			isset( $_GET['give_addon'], $_GET['give_addon_activation_ignore'] )
+			isset($_GET['give_addon'], $_GET['give_addon_activation_ignore'])
 			&& '1' === $_GET['give_addon_activation_ignore']
 		) {
 			// Get the value of the 'give_addon' query string.
-			$addon_query_arg    = sanitize_text_field( wp_unslash( $_GET['give_addon'] ) );
+			$addon_query_arg    = sanitize_text_field(wp_unslash($_GET['give_addon']));
 			$deactivated_addons = array();
 
 			// If All add-on requested to dismiss.
-			if ( 'all' === $addon_query_arg ) {
+			if ('all' === $addon_query_arg) {
 				// Get all activated add-ons.
 				$give_addons = $this->get_plugin_file_names();
 
-				if ( ! empty( $give_addons ) ) {
-					$deactivated_addons = array_keys( $give_addons );
+				if ( ! empty($give_addons)) {
+					$deactivated_addons = array_keys($give_addons);
 				}
 			} else {
 				// Store the addon to deactivate.
 				$deactivated_addons[] = $addon_query_arg;
 			}
 
-			if ( ! empty( $deactivated_addons ) ) {
-				foreach ( $deactivated_addons as $addon ) {
+			if ( ! empty($deactivated_addons)) {
+				foreach ($deactivated_addons as $addon) {
 					// Record it user meta.
-					add_user_meta( $this->user_id, "give_addon_activation_ignore_{$addon}", 'true', true );
+					add_user_meta($this->user_id, "give_addon_activation_ignore_{$addon}", 'true', true);
 				}
 			}
 		}
@@ -462,26 +462,26 @@ 
 	 */
 	public function remove_addon_activate_meta() {
 		// Get the hook name and then grab the plugin file from it.
-		$plugin_file = str_replace( 'deactivate_', '', current_action() );
+		$plugin_file = str_replace('deactivate_', '', current_action());
 
 		// Get all activated add-ons.
 		$give_addons = $this->get_plugin_file_names();
 
-		if ( ! empty( $give_addons ) ) {
-			foreach ( $give_addons as $banner_addon_name => $addon ) {
-				if ( $plugin_file === $addon['plugin_main_file'] ) {
+		if ( ! empty($give_addons)) {
+			foreach ($give_addons as $banner_addon_name => $addon) {
+				if ($plugin_file === $addon['plugin_main_file']) {
 
 					// Get the user meta key.
-					$user_id = (int) __give_get_active_by_user_meta( $banner_addon_name );
+					$user_id = (int) __give_get_active_by_user_meta($banner_addon_name);
 
-					if ( $user_id ) {
+					if ($user_id) {
 						// Get user meta for this add-on.
 						$nag_meta_key = "give_addon_activation_ignore_{$banner_addon_name}";
 
 						// Delete plugin activation option key.
-						delete_option( self::get_banner_user_meta_key( $banner_addon_name ) );
+						delete_option(self::get_banner_user_meta_key($banner_addon_name));
 						// Delete user meta of plugin activation.
-						delete_user_meta( $user_id, $nag_meta_key );
+						delete_user_meta($user_id, $nag_meta_key);
 					}
 				}
 			}
@@ -508,11 +508,11 @@ 
 	 *
 	 * @return bool|mixed|string
 	 */
-	public function get_plugin_folder_name( $main_file ) {
+	public function get_plugin_folder_name($main_file) {
 		// Remove plugin file and get the Add-on's folder name only.
-		$file_path       = explode( '/plugins/', $main_file );
-		$addon_file_path = array_pop( $file_path );
-		$addon_file_path = substr( $addon_file_path, 0, strpos( $addon_file_path, '/' ) );
+		$file_path       = explode('/plugins/', $main_file);
+		$addon_file_path = array_pop($file_path);
+		$addon_file_path = substr($addon_file_path, 0, strpos($addon_file_path, '/'));
 
 		return $addon_file_path;
 	}

includes/admin/class-give-settings.php

Doc Comments +3 added lines, -3 removed lines

@@ -72,7 +72,7 @@ 
 	 *
 	 * @param $url
 	 *
-	 * @return mixed
+	 * @return string
 	 */
 	public function give_update_cmb_meta_box_url( $url ) {
 		// Path to Give's CMB
@@ -1009,7 +1009,7 @@ 
  * @since  1.0
  *
  * @param  array $field_arr
- * @param  array $saved_value
+ * @param  string|boolean $saved_value
  *
  * @return void
  */
@@ -1163,7 +1163,7 @@ 
  * @params $string text
  * @params $filter array
  *
- * @return text $string
+ * @return string $string
  */
 function give_slug_to_title( $string, $filters = array() ) {
 

Spacing +372 added lines, -372 removed lines

@@ -43,14 +43,14 @@ 
 	public function __construct() {
 
 		// Custom CMB2 Settings Fields
-		add_action( 'cmb2_render_give_title', 'give_title_callback', 10, 5 );
-		add_action( 'cmb2_render_give_description', 'give_description_callback', 10, 5 );
-		add_action( 'cmb2_render_enabled_gateways', 'give_enabled_gateways_callback', 10, 5 );
-		add_action( 'cmb2_render_default_gateway', 'give_default_gateway_callback', 10, 5 );
-		add_action( 'cmb2_render_email_preview_buttons', 'give_email_preview_buttons_callback', 10, 5 );
-		add_action( 'cmb2_render_system_info', 'give_system_info_callback', 10, 5 );
-		add_action( 'cmb2_render_api', 'give_api_callback', 10, 5 );
-		add_action( 'cmb2_render_license_key', 'give_license_key_callback', 10, 5 );
+		add_action('cmb2_render_give_title', 'give_title_callback', 10, 5);
+		add_action('cmb2_render_give_description', 'give_description_callback', 10, 5);
+		add_action('cmb2_render_enabled_gateways', 'give_enabled_gateways_callback', 10, 5);
+		add_action('cmb2_render_default_gateway', 'give_default_gateway_callback', 10, 5);
+		add_action('cmb2_render_email_preview_buttons', 'give_email_preview_buttons_callback', 10, 5);
+		add_action('cmb2_render_system_info', 'give_system_info_callback', 10, 5);
+		add_action('cmb2_render_api', 'give_api_callback', 10, 5);
+		add_action('cmb2_render_license_key', 'give_license_key_callback', 10, 5);
 	}
 
 
@@ -60,7 +60,7 @@ 
 	 * @since  1.0
 	 */
 	public function init() {
-		register_setting( $this->key, $this->key );
+		register_setting($this->key, $this->key);
 
 	}
 
@@ -74,9 +74,9 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function give_update_cmb_meta_box_url( $url ) {
+	public function give_update_cmb_meta_box_url($url) {
 		// Path to Give's CMB
-		return GIVE_PLUGIN_URL . '/includes/libraries/cmb2';
+		return GIVE_PLUGIN_URL.'/includes/libraries/cmb2';
 	}
 
 
@@ -88,27 +88,27 @@ 
 	 */
 	public function give_get_settings_tabs() {
 
-		$settings = $this->give_settings( null );
+		$settings = $this->give_settings(null);
 
 		$tabs             = array();
-		$tabs['general']  = __( 'General', 'give' );
-		$tabs['gateways'] = __( 'Payment Gateways', 'give' );
-		$tabs['display']  = __( 'Display Options', 'give' );
-		$tabs['emails']   = __( 'Emails', 'give' );
+		$tabs['general']  = __('General', 'give');
+		$tabs['gateways'] = __('Payment Gateways', 'give');
+		$tabs['display']  = __('Display Options', 'give');
+		$tabs['emails']   = __('Emails', 'give');
 
-		if ( ! empty( $settings['addons']['fields'] ) ) {
-			$tabs['addons'] = __( 'Add-ons', 'give' );
+		if ( ! empty($settings['addons']['fields'])) {
+			$tabs['addons'] = __('Add-ons', 'give');
 		}
 
-		if ( ! empty( $settings['licenses']['fields'] ) ) {
-			$tabs['licenses'] = __( 'Licenses', 'give' );
+		if ( ! empty($settings['licenses']['fields'])) {
+			$tabs['licenses'] = __('Licenses', 'give');
 		}
 
-		$tabs['advanced']    = __( 'Advanced', 'give' );
-		$tabs['api']         = __( 'API', 'give' );
-		$tabs['system_info'] = __( 'System Info', 'give' );
+		$tabs['advanced']    = __('Advanced', 'give');
+		$tabs['api']         = __('API', 'give');
+		$tabs['system_info'] = __('System Info', 'give');
 
-		return apply_filters( 'give_settings_tabs', $tabs );
+		return apply_filters('give_settings_tabs', $tabs);
 	}
 
 
@@ -119,7 +119,7 @@ 
 	 */
 	public function admin_page_display() {
 
-		$active_tab = isset( $_GET['tab'] ) && array_key_exists( $_GET['tab'], $this->give_get_settings_tabs() ) ? $_GET['tab'] : 'general';
+		$active_tab = isset($_GET['tab']) && array_key_exists($_GET['tab'], $this->give_get_settings_tabs()) ? $_GET['tab'] : 'general';
 
 		?>
 
@@ -129,22 +129,22 @@ 
 
 			<h2 class="nav-tab-wrapper">
 				<?php
-				foreach ( $this->give_get_settings_tabs() as $tab_id => $tab_name ) {
+				foreach ($this->give_get_settings_tabs() as $tab_id => $tab_name) {
 
-					$tab_url = esc_url( add_query_arg( array(
+					$tab_url = esc_url(add_query_arg(array(
 						'settings-updated' => false,
 						'tab'              => $tab_id,
-					) ) );
+					)));
 
 					$active = $active_tab == $tab_id ? ' nav-tab-active' : '';
 
-					echo '<a href="' . esc_url( $tab_url ) . '" class="nav-tab' . $active . '" id="tab-' . $tab_id . '">' . esc_html( $tab_name ) . '</a>';
+					echo '<a href="'.esc_url($tab_url).'" class="nav-tab'.$active.'" id="tab-'.$tab_id.'">'.esc_html($tab_name).'</a>';
 
 				}
 				?>
 			</h2>
 
-			<?php cmb2_metabox_form( $this->give_settings( $active_tab ), $this->key ); ?>
+			<?php cmb2_metabox_form($this->give_settings($active_tab), $this->key); ?>
 
 		</div><!-- .wrap -->
 
@@ -166,12 +166,12 @@ 
 	 *
 	 * @return string
 	 */
-	function give_modify_cmb2_form_output( $form_format, $object_id, $cmb ) {
+	function give_modify_cmb2_form_output($form_format, $object_id, $cmb) {
 
 		// only modify the give settings form
-		if ( 'give_settings' == $object_id ) {
+		if ('give_settings' == $object_id) {
 
-			return '<form class="cmb-form" method="post" id="%1$s" enctype="multipart/form-data" encoding="multipart/form-data"><input type="hidden" name="give_settings_saved" value="true"><input type="hidden" name="object_id" value="%2$s">%3$s<div class="give-submit-wrap"><input type="submit" name="submit-cmb" value="' . esc_attr__( 'Save Settings', 'give' ) . '" class="button-primary"></div></form>';
+			return '<form class="cmb-form" method="post" id="%1$s" enctype="multipart/form-data" encoding="multipart/form-data"><input type="hidden" name="give_settings_saved" value="true"><input type="hidden" name="object_id" value="%2$s">%3$s<div class="give-submit-wrap"><input type="submit" name="submit-cmb" value="'.esc_attr__('Save Settings', 'give').'" class="button-primary"></div></form>';
 
 		}
 
@@ -188,7 +188,7 @@ 
 	 *
 	 * @return array
 	 */
-	public function give_settings( $active_tab ) {
+	public function give_settings($active_tab) {
 
 		$give_settings = array(
 			/**
@@ -196,99 +196,99 @@ 
 			 */
 			'general'     => array(
 				'id'         => 'general_settings',
-				'give_title' => __( 'General Settings', 'give' ),
-				'show_on'    => array( 'key' => 'options-page', 'value' => array( $this->key ) ),
-				'fields'     => apply_filters( 'give_settings_general', array(
+				'give_title' => __('General Settings', 'give'),
+				'show_on'    => array('key' => 'options-page', 'value' => array($this->key)),
+				'fields'     => apply_filters('give_settings_general', array(
 						array(
-							'name' => __( 'General Settings', 'give' ),
+							'name' => __('General Settings', 'give'),
 							'desc' => '',
 							'type' => 'give_title',
 							'id'   => 'give_title_general_settings_1',
 						),
 						array(
-							'name'    => __( 'Success Page', 'give' ),
+							'name'    => __('Success Page', 'give'),
 							/* translators: %s: [give_receipt] */
-							'desc'    => sprintf( __( 'The page donors are sent to after completing their donations. The %s shortcode should be on this page.', 'give' ), '<code>[give_receipt]</code>' ),
+							'desc'    => sprintf(__('The page donors are sent to after completing their donations. The %s shortcode should be on this page.', 'give'), '<code>[give_receipt]</code>'),
 							'id'      => 'success_page',
 							'type'    => 'select',
-							'options' => give_cmb2_get_post_options( array(
+							'options' => give_cmb2_get_post_options(array(
 								'post_type'   => 'page',
 								'numberposts' => 999,
-							) ),
+							)),
 						),
 						array(
-							'name'    => __( 'Failed Donation Page', 'give' ),
-							'desc'    => __( 'The page donors are sent to if their donation is cancelled or fails.', 'give' ),
+							'name'    => __('Failed Donation Page', 'give'),
+							'desc'    => __('The page donors are sent to if their donation is cancelled or fails.', 'give'),
 							'id'      => 'failure_page',
 							'type'    => 'select',
-							'options' => give_cmb2_get_post_options( array(
+							'options' => give_cmb2_get_post_options(array(
 								'post_type'   => 'page',
 								'numberposts' => 999,
-							) ),
+							)),
 						),
 						array(
-							'name'    => __( 'Donation History Page', 'give' ),
+							'name'    => __('Donation History Page', 'give'),
 							/* translators: %s: [donation_history] */
-							'desc'    => sprintf( __( 'The page showing a complete donation history for the current user. The %s shortcode should be on this page.', 'give' ), '<code>[donation_history]</code>' ),
+							'desc'    => sprintf(__('The page showing a complete donation history for the current user. The %s shortcode should be on this page.', 'give'), '<code>[donation_history]</code>'),
 							'id'      => 'history_page',
 							'type'    => 'select',
-							'options' => give_cmb2_get_post_options( array(
+							'options' => give_cmb2_get_post_options(array(
 								'post_type'   => 'page',
 								'numberposts' => 999,
-							) ),
+							)),
 						),
 						array(
-							'name'    => __( 'Base Country', 'give' ),
-							'desc'    => __( 'The country your site operates from.', 'give' ),
+							'name'    => __('Base Country', 'give'),
+							'desc'    => __('The country your site operates from.', 'give'),
 							'id'      => 'base_country',
 							'type'    => 'select',
 							'options' => give_get_country_list(),
 						),
 						array(
-							'name' => __( 'Currency Settings', 'give' ),
+							'name' => __('Currency Settings', 'give'),
 							'desc' => '',
 							'type' => 'give_title',
 							'id'   => 'give_title_general_settings_2',
 						),
 						array(
-							'name'    => __( 'Currency', 'give' ),
-							'desc'    => __( 'The donation currency. Note that some payment gateways have currency restrictions.', 'give' ),
+							'name'    => __('Currency', 'give'),
+							'desc'    => __('The donation currency. Note that some payment gateways have currency restrictions.', 'give'),
 							'id'      => 'currency',
 							'type'    => 'select',
 							'options' => give_get_currencies(),
 							'default' => 'USD',
 						),
 						array(
-							'name'    => __( 'Currency Position', 'give' ),
-							'desc'    => __( 'The position of the currency symbol.', 'give' ),
+							'name'    => __('Currency Position', 'give'),
+							'desc'    => __('The position of the currency symbol.', 'give'),
 							'id'      => 'currency_position',
 							'type'    => 'select',
 							'options' => array(
 								/* translators: %s: currency symbol */
-								'before' => sprintf( __( 'Before - %s&#x200e;10', 'give' ), give_currency_symbol( give_get_currency() ) ),
+								'before' => sprintf(__('Before - %s&#x200e;10', 'give'), give_currency_symbol(give_get_currency())),
 								/* translators: %s: currency symbol */
-								'after'  => sprintf( __( 'After - 10%s&#x200f;', 'give' ), give_currency_symbol( give_get_currency() ) ),
+								'after'  => sprintf(__('After - 10%s&#x200f;', 'give'), give_currency_symbol(give_get_currency())),
 							),
 							'default' => 'before',
 						),
 						array(
-							'name'            => __( 'Thousands Separator', 'give' ),
-							'desc'            => __( 'The symbol (usually , or .) to separate thousands.', 'give' ),
+							'name'            => __('Thousands Separator', 'give'),
+							'desc'            => __('The symbol (usually , or .) to separate thousands.', 'give'),
 							'id'              => 'thousands_separator',
 							'type'            => 'text_small',
 							'sanitization_cb' => 'give_sanitize_thousand_separator',
 							'default'         => ',',
 						),
 						array(
-							'name'    => __( 'Decimal Separator', 'give' ),
-							'desc'    => __( 'The symbol (usually , or .) to separate decimal points.', 'give' ),
+							'name'    => __('Decimal Separator', 'give'),
+							'desc'    => __('The symbol (usually , or .) to separate decimal points.', 'give'),
 							'id'      => 'decimal_separator',
 							'type'    => 'text_small',
 							'default' => '.',
 						),
 						array(
-							'name'            => __( 'Number of Decimals', 'give' ),
-							'desc'            => __( 'The number of decimal points displayed in amounts.', 'give' ),
+							'name'            => __('Number of Decimals', 'give'),
+							'desc'            => __('The number of decimal points displayed in amounts.', 'give'),
 							'id'              => 'number_decimals',
 							'type'            => 'text_small',
 							'default'         => 2,
@@ -302,83 +302,83 @@ 
 			 */
 			'gateways'    => array(
 				'id'         => 'payment_gateways',
-				'give_title' => __( 'Payment Gateways', 'give' ),
-				'show_on'    => array( 'key' => 'options-page', 'value' => array( $this->key ) ),
-				'fields'     => apply_filters( 'give_settings_gateways', array(
+				'give_title' => __('Payment Gateways', 'give'),
+				'show_on'    => array('key' => 'options-page', 'value' => array($this->key)),
+				'fields'     => apply_filters('give_settings_gateways', array(
 						array(
-							'name' => __( 'Gateways Settings', 'give' ),
+							'name' => __('Gateways Settings', 'give'),
 							'desc' => '',
 							'id'   => 'give_title_gateway_settings_1',
 							'type' => 'give_title',
 						),
 						array(
-							'name' => __( 'Test Mode', 'give' ),
-							'desc' => __( 'While in test mode no live donations are processed. To fully use test mode, you must have a sandbox (test) account for the payment gateway you are testing.', 'give' ),
+							'name' => __('Test Mode', 'give'),
+							'desc' => __('While in test mode no live donations are processed. To fully use test mode, you must have a sandbox (test) account for the payment gateway you are testing.', 'give'),
 							'id'   => 'test_mode',
 							'type' => 'checkbox',
 						),
 						array(
-							'name' => __( 'Enabled Gateways', 'give' ),
-							'desc' => __( 'Enable your payment gateway. Can be ordered by dragging.', 'give' ),
+							'name' => __('Enabled Gateways', 'give'),
+							'desc' => __('Enable your payment gateway. Can be ordered by dragging.', 'give'),
 							'id'   => 'gateways',
 							'type' => 'enabled_gateways',
 						),
 						array(
-							'name' => __( 'Default Gateway', 'give' ),
-							'desc' => __( 'The gateway that will be selected by default.', 'give' ),
+							'name' => __('Default Gateway', 'give'),
+							'desc' => __('The gateway that will be selected by default.', 'give'),
 							'id'   => 'default_gateway',
 							'type' => 'default_gateway',
 						),
 						array(
-							'name' => __( 'PayPal Standard', 'give' ),
+							'name' => __('PayPal Standard', 'give'),
 							'desc' => '',
 							'type' => 'give_title',
 							'id'   => 'give_title_gateway_settings_2',
 						),
 						array(
-							'name' => __( 'PayPal Email', 'give' ),
-							'desc' => __( 'Enter your PayPal account\'s email.', 'give' ),
+							'name' => __('PayPal Email', 'give'),
+							'desc' => __('Enter your PayPal account\'s email.', 'give'),
 							'id'   => 'paypal_email',
 							'type' => 'text_email',
 						),
 						array(
-							'name' => __( 'PayPal Page Style', 'give' ),
-							'desc' => __( 'Enter the name of the page style to use, or leave blank to use the default.', 'give' ),
+							'name' => __('PayPal Page Style', 'give'),
+							'desc' => __('Enter the name of the page style to use, or leave blank to use the default.', 'give'),
 							'id'   => 'paypal_page_style',
 							'type' => 'text',
 						),
 						array(
-							'name'    => __( 'PayPal Transaction Type', 'give' ),
-							'desc'    => __( 'Nonprofits must verify their status to withdraw donations they receive via PayPal. PayPal users that are not verified nonprofits must demonstrate how their donations will be used, once they raise more than $10,000. By default, Give transactions are sent to PayPal as donations. You may change the transaction type using this option if you feel you may not meet PayPal\'s donation requirements.', 'give' ),
+							'name'    => __('PayPal Transaction Type', 'give'),
+							'desc'    => __('Nonprofits must verify their status to withdraw donations they receive via PayPal. PayPal users that are not verified nonprofits must demonstrate how their donations will be used, once they raise more than $10,000. By default, Give transactions are sent to PayPal as donations. You may change the transaction type using this option if you feel you may not meet PayPal\'s donation requirements.', 'give'),
 							'id'      => 'paypal_button_type',
 							'type'    => 'radio_inline',
 							'options' => array(
-								'donation' => __( 'Donation', 'give' ),
-								'standard' => __( 'Standard Transaction', 'give' ),
+								'donation' => __('Donation', 'give'),
+								'standard' => __('Standard Transaction', 'give'),
 							),
 							'default' => 'donation',
 						),
 						array(
-							'name' => __( 'Disable PayPal IPN Verification', 'give' ),
-							'desc' => __( 'If donations are not getting marked as complete, use a slightly less secure method of verifying donations.', 'give' ),
+							'name' => __('Disable PayPal IPN Verification', 'give'),
+							'desc' => __('If donations are not getting marked as complete, use a slightly less secure method of verifying donations.', 'give'),
 							'id'   => 'disable_paypal_verification',
 							'type' => 'checkbox',
 						),
 						array(
-							'name' => __( 'Offline Donations', 'give' ),
+							'name' => __('Offline Donations', 'give'),
 							'desc' => '',
 							'type' => 'give_title',
 							'id'   => 'give_title_gateway_settings_3',
 						),
 						array(
-							'name' => __( 'Collect Billing Details', 'give' ),
-							'desc' => __( 'Enable to request billing details for offline donations. Will appear above offline donation instructions. Can be enabled/disabled per form.', 'give' ),
+							'name' => __('Collect Billing Details', 'give'),
+							'desc' => __('Enable to request billing details for offline donations. Will appear above offline donation instructions. Can be enabled/disabled per form.', 'give'),
 							'id'   => 'give_offline_donation_enable_billing_fields',
 							'type' => 'checkbox',
 						),
 						array(
-							'name'    => __( 'Offline Donation Instructions', 'give' ),
-							'desc'    => __( 'The following content will appear for all forms when the user selects the offline donation payment option. Note: You may customize the content per form as needed.', 'give' ),
+							'name'    => __('Offline Donation Instructions', 'give'),
+							'desc'    => __('The following content will appear for all forms when the user selects the offline donation payment option. Note: You may customize the content per form as needed.', 'give'),
 							'id'      => 'global_offline_donation_content',
 							'default' => give_get_default_offline_donation_content(),
 							'type'    => 'wysiwyg',
@@ -387,15 +387,15 @@ 
 							),
 						),
 						array(
-							'name'    => __( 'Offline Donation Email Instructions Subject', 'give' ),
-							'desc'    => __( 'Enter the subject line for the donation receipt email.', 'give' ),
+							'name'    => __('Offline Donation Email Instructions Subject', 'give'),
+							'desc'    => __('Enter the subject line for the donation receipt email.', 'give'),
 							'id'      => 'offline_donation_subject',
-							'default' => esc_attr__( '{donation} - Offline Donation Instructions', 'give' ),
+							'default' => esc_attr__('{donation} - Offline Donation Instructions', 'give'),
 							'type'    => 'text',
 						),
 						array(
-							'name'    => __( 'Offline Donation Email Instructions', 'give' ),
-							'desc'    => __( 'Enter the instructions you want emailed to the donor after they have submitted the donation form. Most likely this would include important information like mailing address and who to make the check out to.', 'give' ),
+							'name'    => __('Offline Donation Email Instructions', 'give'),
+							'desc'    => __('Enter the instructions you want emailed to the donor after they have submitted the donation form. Most likely this would include important information like mailing address and who to make the check out to.', 'give'),
 							'id'      => 'global_offline_donation_email',
 							'default' => give_get_default_offline_donation_email_content(),
 							'type'    => 'wysiwyg',
@@ -409,104 +409,104 @@ 
 			/** Display Settings */
 			'display'     => array(
 				'id'         => 'display_settings',
-				'give_title' => __( 'Display Settings', 'give' ),
-				'show_on'    => array( 'key' => 'options-page', 'value' => array( $this->key ) ),
-				'fields'     => apply_filters( 'give_settings_display', array(
+				'give_title' => __('Display Settings', 'give'),
+				'show_on'    => array('key' => 'options-page', 'value' => array($this->key)),
+				'fields'     => apply_filters('give_settings_display', array(
 						array(
-							'name' => __( 'Display Settings', 'give' ),
+							'name' => __('Display Settings', 'give'),
 							'desc' => '',
 							'id'   => 'give_title_display_settings_1',
 							'type' => 'give_title',
 						),
 						array(
-							'name' => __( 'Disable CSS', 'give' ),
-							'desc' => __( 'Enable this option if you would like to disable all of Give\'s included CSS stylesheets.', 'give' ),
+							'name' => __('Disable CSS', 'give'),
+							'desc' => __('Enable this option if you would like to disable all of Give\'s included CSS stylesheets.', 'give'),
 							'id'   => 'disable_css',
 							'type' => 'checkbox',
 						),
 						array(
-							'name' => __( 'Enable Floating Labels', 'give' ),
+							'name' => __('Enable Floating Labels', 'give'),
 							/* translators: %s: http://docs.givewp.com/form-floating-labels */
-							'desc' => sprintf( wp_kses( __( 'Enable <a href="%s" target="_blank">floating labels</a> in Give\'s donation forms. Note that if the "Disable CSS" option is enabled, you will need to style the floating labels yourself.', 'give' ), array(
+							'desc' => sprintf(wp_kses(__('Enable <a href="%s" target="_blank">floating labels</a> in Give\'s donation forms. Note that if the "Disable CSS" option is enabled, you will need to style the floating labels yourself.', 'give'), array(
 								'a' => array(
 									'href'   => array(),
 									'target' => array(),
 								),
-							) ), esc_url( 'http://docs.givewp.com/form-floating-labels' ) ),
+							)), esc_url('http://docs.givewp.com/form-floating-labels')),
 							'id'   => 'floatlabels',
 							'type' => 'checkbox',
 						),
 						array(
-							'name' => __( 'Disable Welcome Screen', 'give' ),
+							'name' => __('Disable Welcome Screen', 'give'),
 							/* translators: %s: about page URL */
-							'desc' => sprintf( wp_kses( __( 'Enable this option if you would like to disable the <a href="%s" target="_blank">Give Welcome screen</a> every time Give is activated and/or updated.', 'give' ), array(
+							'desc' => sprintf(wp_kses(__('Enable this option if you would like to disable the <a href="%s" target="_blank">Give Welcome screen</a> every time Give is activated and/or updated.', 'give'), array(
 								'a' => array(
 									'href'   => array(),
 									'target' => array(),
 								),
-							) ), esc_url( admin_url( 'index.php?page=give-about' ) ) ),
+							)), esc_url(admin_url('index.php?page=give-about'))),
 							'id'   => 'disable_welcome',
 							'type' => 'checkbox',
 						),
 						array(
-							'name' => __( 'Post Types', 'give' ),
+							'name' => __('Post Types', 'give'),
 							'desc' => '',
 							'id'   => 'give_title_display_settings_2',
 							'type' => 'give_title',
 						),
 						array(
-							'name' => __( 'Disable Form Single Views', 'give' ),
-							'desc' => __( 'By default, all forms have single views enabled which create a specific URL on your website for that form. This option disables the singular and archive views from being publicly viewable. Note: you will need to embed forms using a shortcode or widget if enabled.', 'give' ),
+							'name' => __('Disable Form Single Views', 'give'),
+							'desc' => __('By default, all forms have single views enabled which create a specific URL on your website for that form. This option disables the singular and archive views from being publicly viewable. Note: you will need to embed forms using a shortcode or widget if enabled.', 'give'),
 							'id'   => 'disable_forms_singular',
 							'type' => 'checkbox',
 						),
 						array(
-							'name' => __( 'Disable Form Archives', 'give' ),
-							'desc' => __( 'Archives pages list all the forms you have created. This option will disable only the form\'s archive page(s). The single form\'s view will remain in place. Note: you will need to refresh your permalinks after this option has been enabled.', 'give' ),
+							'name' => __('Disable Form Archives', 'give'),
+							'desc' => __('Archives pages list all the forms you have created. This option will disable only the form\'s archive page(s). The single form\'s view will remain in place. Note: you will need to refresh your permalinks after this option has been enabled.', 'give'),
 							'id'   => 'disable_forms_archives',
 							'type' => 'checkbox',
 						),
 						array(
-							'name' => __( 'Disable Form Excerpts', 'give' ),
-							'desc' => __( 'The excerpt is an optional summary or description of a donation form; in short, a summary as to why the user should give.', 'give' ),
+							'name' => __('Disable Form Excerpts', 'give'),
+							'desc' => __('The excerpt is an optional summary or description of a donation form; in short, a summary as to why the user should give.', 'give'),
 							'id'   => 'disable_forms_excerpt',
 							'type' => 'checkbox',
 						),
 						array(
-							'name'    => __( 'Featured Image Size', 'give' ),
-							'desc'    => __( 'The Featured Image is an image that is chosen as the representative image for a donation form. Some themes may have custom featured image sizes. Please select the size you would like to display for your single donation form\'s featured image.', 'give' ),
+							'name'    => __('Featured Image Size', 'give'),
+							'desc'    => __('The Featured Image is an image that is chosen as the representative image for a donation form. Some themes may have custom featured image sizes. Please select the size you would like to display for your single donation form\'s featured image.', 'give'),
 							'id'      => 'featured_image_size',
 							'type'    => 'select',
 							'default' => 'large',
 							'options' => give_get_featured_image_sizes(),
 						),
 						array(
-							'name' => __( 'Disable Form Featured Image', 'give' ),
-							'desc' => __( 'If you do not wish to use the featured image functionality you can disable it using this option and it will not be displayed for single donation forms.', 'give' ),
+							'name' => __('Disable Form Featured Image', 'give'),
+							'desc' => __('If you do not wish to use the featured image functionality you can disable it using this option and it will not be displayed for single donation forms.', 'give'),
 							'id'   => 'disable_form_featured_img',
 							'type' => 'checkbox',
 						),
 						array(
-							'name' => __( 'Disable Single Form Sidebar', 'give' ),
-							'desc' => __( 'The sidebar allows you to add additional widget to the Give single form view. If you don\'t plan on using the sidebar you may disable it with this option.', 'give' ),
+							'name' => __('Disable Single Form Sidebar', 'give'),
+							'desc' => __('The sidebar allows you to add additional widget to the Give single form view. If you don\'t plan on using the sidebar you may disable it with this option.', 'give'),
 							'id'   => 'disable_form_sidebar',
 							'type' => 'checkbox',
 						),
 						array(
-							'name' => __( 'Taxonomies', 'give' ),
+							'name' => __('Taxonomies', 'give'),
 							'desc' => '',
 							'id'   => 'give_title_display_settings_3',
 							'type' => 'give_title',
 						),
 						array(
-							'name' => __( 'Enable Form Categories', 'give' ),
-							'desc' => __( 'Enables the "Category" taxonomy for all Give forms.', 'give' ),
+							'name' => __('Enable Form Categories', 'give'),
+							'desc' => __('Enables the "Category" taxonomy for all Give forms.', 'give'),
 							'id'   => 'categories',
 							'type' => 'checkbox',
 						),
 						array(
-							'name' => __( 'Enable Form Tags', 'give' ),
-							'desc' => __( 'Enables the "Tag" taxonomy for all Give forms.', 'give' ),
+							'name' => __('Enable Form Tags', 'give'),
+							'desc' => __('Enables the "Tag" taxonomy for all Give forms.', 'give'),
 							'id'   => 'tags',
 							'type' => 'checkbox',
 						),
@@ -519,101 +519,101 @@ 
 			 */
 			'emails'      => array(
 				'id'         => 'email_settings',
-				'give_title' => __( 'Email Settings', 'give' ),
-				'show_on'    => array( 'key' => 'options-page', 'value' => array( $this->key ) ),
-				'fields'     => apply_filters( 'give_settings_emails', array(
+				'give_title' => __('Email Settings', 'give'),
+				'show_on'    => array('key' => 'options-page', 'value' => array($this->key)),
+				'fields'     => apply_filters('give_settings_emails', array(
 						array(
-							'name' => __( 'Email Settings', 'give' ),
+							'name' => __('Email Settings', 'give'),
 							'desc' => '',
 							'id'   => 'give_title_email_settings_1',
 							'type' => 'give_title',
 						),
 						array(
 							'id'      => 'email_template',
-							'name'    => __( 'Email Template', 'give' ),
-							'desc'    => __( 'Choose a template. Click "Save Changes" then "Preview Donation Receipt" to see the new template.', 'give' ),
+							'name'    => __('Email Template', 'give'),
+							'desc'    => __('Choose a template. Click "Save Changes" then "Preview Donation Receipt" to see the new template.', 'give'),
 							'type'    => 'select',
 							'options' => give_get_email_templates(),
 						),
 						array(
 							'id'   => 'email_logo',
-							'name' => __( 'Logo', 'give' ),
-							'desc' => __( 'Upload or choose a logo to be displayed at the top of the donation receipt emails. Displayed on HTML emails only.', 'give' ),
+							'name' => __('Logo', 'give'),
+							'desc' => __('Upload or choose a logo to be displayed at the top of the donation receipt emails. Displayed on HTML emails only.', 'give'),
 							'type' => 'file',
 						),
 						array(
 							'id'      => 'from_name',
-							'name'    => __( 'From Name', 'give' ),
-							'desc'    => __( 'The name that appears in the "From" field in donation receipt emails.', 'give' ),
-							'default' => get_bloginfo( 'name' ),
+							'name'    => __('From Name', 'give'),
+							'desc'    => __('The name that appears in the "From" field in donation receipt emails.', 'give'),
+							'default' => get_bloginfo('name'),
 							'type'    => 'text',
 						),
 						array(
 							'id'      => 'from_email',
-							'name'    => __( 'From Email', 'give' ),
-							'desc'    => __( 'Email to send donation receipts from. This will act as the "from" and "reply-to" address.', 'give' ),
-							'default' => get_bloginfo( 'admin_email' ),
+							'name'    => __('From Email', 'give'),
+							'desc'    => __('Email to send donation receipts from. This will act as the "from" and "reply-to" address.', 'give'),
+							'default' => get_bloginfo('admin_email'),
 							'type'    => 'text',
 						),
 						array(
-							'name' => __( 'Donation Receipt', 'give' ),
+							'name' => __('Donation Receipt', 'give'),
 							'desc' => '',
 							'id'   => 'give_title_email_settings_2',
 							'type' => 'give_title',
 						),
 						array(
 							'id'      => 'donation_subject',
-							'name'    => __( 'Donation Email Subject', 'give' ),
-							'desc'    => __( 'Enter the subject line for the donation receipt email.', 'give' ),
-							'default' => esc_attr__( 'Donation Receipt', 'give' ),
+							'name'    => __('Donation Email Subject', 'give'),
+							'desc'    => __('Enter the subject line for the donation receipt email.', 'give'),
+							'default' => esc_attr__('Donation Receipt', 'give'),
 							'type'    => 'text',
 						),
 						array(
 							'id'      => 'donation_receipt',
-							'name'    => __( 'Donation Receipt', 'give' ),
+							'name'    => __('Donation Receipt', 'give'),
 							'desc'    => sprintf(
 							/* translators: %s: emails tags list */
-								__( 'Enter the email that is sent to users after completing a successful donation. HTML is accepted. Available template tags: %s', 'give' ),
-								'<br/>' . give_get_emails_tags_list()
+								__('Enter the email that is sent to users after completing a successful donation. HTML is accepted. Available template tags: %s', 'give'),
+								'<br/>'.give_get_emails_tags_list()
 							),
 							'type'    => 'wysiwyg',
 							'default' => give_get_default_donation_receipt_email(),
 						),
 						array(
-							'name' => __( 'New Donation Notification', 'give' ),
+							'name' => __('New Donation Notification', 'give'),
 							'desc' => '',
 							'id'   => 'give_title_email_settings_3',
 							'type' => 'give_title',
 						),
 						array(
 							'id'      => 'donation_notification_subject',
-							'name'    => __( 'Donation Notification Subject', 'give' ),
-							'desc'    => __( 'Enter the subject line for the donation notification email.', 'give' ),
+							'name'    => __('Donation Notification Subject', 'give'),
+							'desc'    => __('Enter the subject line for the donation notification email.', 'give'),
 							'type'    => 'text',
-							'default' => esc_attr__( 'New Donation - #{payment_id}', 'give' ),
+							'default' => esc_attr__('New Donation - #{payment_id}', 'give'),
 						),
 						array(
 							'id'      => 'donation_notification',
-							'name'    => __( 'Donation Notification', 'give' ),
+							'name'    => __('Donation Notification', 'give'),
 							'desc'    => sprintf(
 							/* translators: %s: emails tags list */
-								__( 'Enter the email that is sent to donation notification emails after completion of a donation. HTML is accepted. Available template tags: %s', 'give' ),
-								'<br/>' . give_get_emails_tags_list()
+								__('Enter the email that is sent to donation notification emails after completion of a donation. HTML is accepted. Available template tags: %s', 'give'),
+								'<br/>'.give_get_emails_tags_list()
 							),
 							'type'    => 'wysiwyg',
 							'default' => give_get_default_donation_notification_email(),
 						),
 						array(
 							'id'      => 'admin_notice_emails',
-							'name'    => __( 'Donation Notification Emails', 'give' ),
-							'desc'    => __( 'Enter the email address(es) that should receive a notification anytime a donation is made, please only enter <span class="give-underline">one email address per line</span> and <strong>not separated by commas</strong>.', 'give' ),
+							'name'    => __('Donation Notification Emails', 'give'),
+							'desc'    => __('Enter the email address(es) that should receive a notification anytime a donation is made, please only enter <span class="give-underline">one email address per line</span> and <strong>not separated by commas</strong>.', 'give'),
 							'type'    => 'textarea',
-							'default' => get_bloginfo( 'admin_email' ),
+							'default' => get_bloginfo('admin_email'),
 						),
 						array(
 							'id'   => 'disable_admin_notices',
-							'name' => __( 'Disable Admin Notifications', 'give' ),
-							'desc' => __( 'Check this box if you do not want to receive emails when new donations are made.', 'give' ),
+							'name' => __('Disable Admin Notifications', 'give'),
+							'desc' => __('Check this box if you do not want to receive emails when new donations are made.', 'give'),
 							'type' => 'checkbox',
 						),
 					)
@@ -622,99 +622,99 @@ 
 			/** Extension Settings */
 			'addons'      => array(
 				'id'         => 'addons',
-				'give_title' => __( 'Give Add-ons Settings', 'give' ),
-				'show_on'    => array( 'key' => 'options-page', 'value' => array( $this->key ) ),
-				'fields'     => apply_filters( 'give_settings_addons', array()
+				'give_title' => __('Give Add-ons Settings', 'give'),
+				'show_on'    => array('key' => 'options-page', 'value' => array($this->key)),
+				'fields'     => apply_filters('give_settings_addons', array()
 				),
 			),
 			/** Licenses Settings */
 			'licenses'    => array(
 				'id'         => 'licenses',
-				'give_title' => __( 'Give Licenses', 'give' ),
-				'show_on'    => array( 'key' => 'options-page', 'value' => array( $this->key ) ),
-				'fields'     => apply_filters( 'give_settings_licenses', array()
+				'give_title' => __('Give Licenses', 'give'),
+				'show_on'    => array('key' => 'options-page', 'value' => array($this->key)),
+				'fields'     => apply_filters('give_settings_licenses', array()
 				),
 			),
 			/** Advanced Options */
 			'advanced'    => array(
 				'id'         => 'advanced_options',
-				'give_title' => __( 'Advanced Options', 'give' ),
-				'show_on'    => array( 'key' => 'options-page', 'value' => array( $this->key ) ),
-				'fields'     => apply_filters( 'give_settings_advanced', array(
+				'give_title' => __('Advanced Options', 'give'),
+				'show_on'    => array('key' => 'options-page', 'value' => array($this->key)),
+				'fields'     => apply_filters('give_settings_advanced', array(
 						array(
-							'name' => __( 'Access Control', 'give' ),
+							'name' => __('Access Control', 'give'),
 							'desc' => '',
 							'id'   => 'give_title_session_control_1',
 							'type' => 'give_title',
 						),
 						array(
 							'id'      => 'session_lifetime',
-							'name'    => __( 'Session Lifetime', 'give' ),
-							'desc'    => __( 'The length of time a user\'s session is kept alive. Give starts a new session per user upon donation. Sessions allow donors to view their donation receipts without being logged in.', 'give' ),
+							'name'    => __('Session Lifetime', 'give'),
+							'desc'    => __('The length of time a user\'s session is kept alive. Give starts a new session per user upon donation. Sessions allow donors to view their donation receipts without being logged in.', 'give'),
 							'type'    => 'select',
 							'options' => array(
-								'86400'  => __( '24 Hours', 'give' ),
-								'172800' => __( '48 Hours', 'give' ),
-								'259200' => __( '72 Hours', 'give' ),
-								'604800' => __( '1 Week', 'give' ),
+								'86400'  => __('24 Hours', 'give'),
+								'172800' => __('48 Hours', 'give'),
+								'259200' => __('72 Hours', 'give'),
+								'604800' => __('1 Week', 'give'),
 							),
 						),
 						array(
-							'name' => __( 'Email Access', 'give' ),
-							'desc' => __( 'Would you like your donors to be able to access their donation history using only email? Donors whose sessions have expired and do not have an account may still access their donation history via a temporary email access link.', 'give' ),
+							'name' => __('Email Access', 'give'),
+							'desc' => __('Would you like your donors to be able to access their donation history using only email? Donors whose sessions have expired and do not have an account may still access their donation history via a temporary email access link.', 'give'),
 							'id'   => 'email_access',
 							'type' => 'checkbox',
 						),
 						array(
 							'id'      => 'recaptcha_key',
-							'name'    => __( 'reCAPTCHA Site Key', 'give' ),
+							'name'    => __('reCAPTCHA Site Key', 'give'),
 							/* translators: %s: https://www.google.com/recaptcha/ */
-							'desc'    => sprintf( __( 'If you would like to prevent spam on the email access form navigate to <a href="%s" target="_blank">the reCAPTCHA website</a> and sign up for an API key. The reCAPTCHA uses Google\'s user-friendly single click verification method.', 'give' ), esc_url( 'https://www.google.com/recaptcha/' ) ),
+							'desc'    => sprintf(__('If you would like to prevent spam on the email access form navigate to <a href="%s" target="_blank">the reCAPTCHA website</a> and sign up for an API key. The reCAPTCHA uses Google\'s user-friendly single click verification method.', 'give'), esc_url('https://www.google.com/recaptcha/')),
 							'default' => '',
 							'type'    => 'text',
 						),
 						array(
 							'id'      => 'recaptcha_secret',
-							'name'    => __( 'reCAPTCHA Secret Key', 'give' ),
-							'desc'    => __( 'Please paste the reCAPTCHA secret key here from your manage reCAPTCHA API Keys panel.', 'give' ),
+							'name'    => __('reCAPTCHA Secret Key', 'give'),
+							'desc'    => __('Please paste the reCAPTCHA secret key here from your manage reCAPTCHA API Keys panel.', 'give'),
 							'default' => '',
 							'type'    => 'text',
 						),
 						array(
-							'name' => __( 'Data Control', 'give' ),
+							'name' => __('Data Control', 'give'),
 							'desc' => '',
 							'id'   => 'give_title_data_control_2',
 							'type' => 'give_title',
 						),
 						array(
-							'name' => __( 'Remove All Data on Uninstall?', 'give' ),
-							'desc' => __( 'When the plugin is deleted, completely remove all Give data.', 'give' ),
+							'name' => __('Remove All Data on Uninstall?', 'give'),
+							'desc' => __('When the plugin is deleted, completely remove all Give data.', 'give'),
 							'id'   => 'uninstall_on_delete',
 							'type' => 'checkbox',
 						),
 						array(
-							'name' => __( 'Filter Control', 'give' ),
+							'name' => __('Filter Control', 'give'),
 							'desc' => '',
 							'id'   => 'give_title_filter_control',
 							'type' => 'give_title',
 						),
 						array(
 							/* translators: %s: the_content */
-							'name' => sprintf( __( 'Disable %s filter', 'give' ), '<code>the_content</code>' ),
+							'name' => sprintf(__('Disable %s filter', 'give'), '<code>the_content</code>'),
 							/* translators: 1: https://codex.wordpress.org/Plugin_API/Filter_Reference/the_content 2: the_content */
-							'desc' => sprintf( __( 'If you are seeing extra social buttons, related posts, or other unwanted elements appearing within your forms then you can disable WordPress\' content filter. <a href="%1$s" target="_blank">Learn more</a> about %2$s filter.', 'give' ), esc_url( 'https://codex.wordpress.org/Plugin_API/Filter_Reference/the_content' ), '<code>the_content</code>' ),
+							'desc' => sprintf(__('If you are seeing extra social buttons, related posts, or other unwanted elements appearing within your forms then you can disable WordPress\' content filter. <a href="%1$s" target="_blank">Learn more</a> about %2$s filter.', 'give'), esc_url('https://codex.wordpress.org/Plugin_API/Filter_Reference/the_content'), '<code>the_content</code>'),
 							'id'   => 'disable_the_content_filter',
 							'type' => 'checkbox',
 						),
 						array(
-							'name' => __( 'Script Loading', 'give' ),
+							'name' => __('Script Loading', 'give'),
 							'desc' => '',
 							'id'   => 'give_title_script_control',
 							'type' => 'give_title',
 						),
 						array(
-							'name' => __( 'Load Scripts in Footer?', 'give' ),
-							'desc' => __( 'Check this box if you would like Give to load all frontend JavaScript files in the footer.', 'give' ),
+							'name' => __('Load Scripts in Footer?', 'give'),
+							'desc' => __('Check this box if you would like Give to load all frontend JavaScript files in the footer.', 'give'),
 							'id'   => 'scripts_footer',
 							'type' => 'checkbox',
 						),
@@ -724,13 +724,13 @@ 
 			/** API Settings */
 			'api'         => array(
 				'id'         => 'api',
-				'give_title' => __( 'API', 'give' ),
-				'show_on'    => array( 'key' => 'options-page', 'value' => array( $this->key ) ),
+				'give_title' => __('API', 'give'),
+				'show_on'    => array('key' => 'options-page', 'value' => array($this->key)),
 				'show_names' => false, // Hide field names on the left
-				'fields'     => apply_filters( 'give_settings_system', array(
+				'fields'     => apply_filters('give_settings_system', array(
 						array(
 							'id'   => 'api',
-							'name' => __( 'API', 'give' ),
+							'name' => __('API', 'give'),
 							'type' => 'api',
 						),
 					)
@@ -739,13 +739,13 @@ 
 			/** Licenses Settings */
 			'system_info' => array(
 				'id'         => 'system_info',
-				'give_title' => __( 'System Info', 'give' ),
-				'show_on'    => array( 'key' => 'options-page', 'value' => array( $this->key ) ),
-				'fields'     => apply_filters( 'give_settings_system', array(
+				'give_title' => __('System Info', 'give'),
+				'show_on'    => array('key' => 'options-page', 'value' => array($this->key)),
+				'fields'     => apply_filters('give_settings_system', array(
 						array(
 							'id'   => 'system-info-textarea',
-							'name' => __( 'System Info', 'give' ),
-							'desc' => __( 'Please copy and paste this information in your ticket when contacting support.', 'give' ),
+							'name' => __('System Info', 'give'),
+							'desc' => __('Please copy and paste this information in your ticket when contacting support.', 'give'),
 							'type' => 'system_info',
 						),
 					)
@@ -753,15 +753,15 @@ 
 			),
 		);
 
-		$give_settings = apply_filters( 'give_registered_settings', $give_settings );
+		$give_settings = apply_filters('give_registered_settings', $give_settings);
 
 		// Return all settings array if no active tab
-		if ( empty( $active_tab ) || ! isset( $give_settings[ $active_tab ] ) ) {
+		if (empty($active_tab) || ! isset($give_settings[$active_tab])) {
 			return $give_settings;
 		}
 
 		// Add other tabs and settings fields as needed
-		return $give_settings[ $active_tab ];
+		return $give_settings[$active_tab];
 
 	}
 
@@ -770,11 +770,11 @@ 
 	 */
 	public function settings_notices() {
 
-		if ( ! isset( $_POST['give_settings_saved'] ) ) {
+		if ( ! isset($_POST['give_settings_saved'])) {
 			return;
 		}
 
-		add_settings_error( 'give-notices', 'global-settings-updated', __( 'Settings updated.', 'give' ), 'updated' );
+		add_settings_error('give-notices', 'global-settings-updated', __('Settings updated.', 'give'), 'updated');
 
 	}
 
@@ -789,14 +789,14 @@ 
 	 * @return mixed         Field value or exception is thrown.
 	 * @throws Exception     Throws an exception if the field is invalid.
 	 */
-	public function __get( $field ) {
+	public function __get($field) {
 
 		// Allowed fields to retrieve
-		if ( in_array( $field, array( 'key', 'fields', 'give_title', 'options_page' ), true ) ) {
+		if (in_array($field, array('key', 'fields', 'give_title', 'options_page'), true)) {
 			return $this->{$field};
 		}
 
-		throw new Exception( sprintf( __( 'Invalid property: %s', 'give' ), $field ) );
+		throw new Exception(sprintf(__('Invalid property: %s', 'give'), $field));
 	}
 
 
@@ -815,12 +815,12 @@ 
  *
  * @return mixed        Option value
  */
-function give_get_option( $key = '', $default = false ) {
+function give_get_option($key = '', $default = false) {
 	$give_options = give_get_settings();
-	$value        = ! empty( $give_options[ $key ] ) ? $give_options[ $key ] : $default;
-	$value        = apply_filters( 'give_get_option', $value, $key, $default );
+	$value        = ! empty($give_options[$key]) ? $give_options[$key] : $default;
+	$value        = apply_filters('give_get_option', $value, $key, $default);
 
-	return apply_filters( "give_get_option_{$key}", $value, $key, $default );
+	return apply_filters("give_get_option_{$key}", $value, $key, $default);
 }
 
 
@@ -838,33 +838,33 @@ 
  *
  * @return boolean True if updated, false if not.
  */
-function give_update_option( $key = '', $value = false ) {
+function give_update_option($key = '', $value = false) {
 
 	// If no key, exit
-	if ( empty( $key ) ) {
+	if (empty($key)) {
 		return false;
 	}
 
-	if ( empty( $value ) ) {
-		$remove_option = give_delete_option( $key );
+	if (empty($value)) {
+		$remove_option = give_delete_option($key);
 
 		return $remove_option;
 	}
 
 	// First let's grab the current settings
-	$options = get_option( 'give_settings' );
+	$options = get_option('give_settings');
 
 	// Let's let devs alter that value coming in
-	$value = apply_filters( 'give_update_option', $value, $key );
+	$value = apply_filters('give_update_option', $value, $key);
 
 	// Next let's try to update the value
-	$options[ $key ] = $value;
-	$did_update      = update_option( 'give_settings', $options );
+	$options[$key] = $value;
+	$did_update      = update_option('give_settings', $options);
 
 	// If it updated, let's update the global variable
-	if ( $did_update ) {
+	if ($did_update) {
 		global $give_options;
-		$give_options[ $key ] = $value;
+		$give_options[$key] = $value;
 	}
 
 	return $did_update;
@@ -883,25 +883,25 @@ 
  *
  * @return boolean True if updated, false if not.
  */
-function give_delete_option( $key = '' ) {
+function give_delete_option($key = '') {
 
 	// If no key, exit
-	if ( empty( $key ) ) {
+	if (empty($key)) {
 		return false;
 	}
 
 	// First let's grab the current settings
-	$options = get_option( 'give_settings' );
+	$options = get_option('give_settings');
 
 	// Next let's try to update the value
-	if ( isset( $options[ $key ] ) ) {
-		unset( $options[ $key ] );
+	if (isset($options[$key])) {
+		unset($options[$key]);
 	}
 
-	$did_update = update_option( 'give_settings', $options );
+	$did_update = update_option('give_settings', $options);
 
 	// If it updated, let's update the global variable
-	if ( $did_update ) {
+	if ($did_update) {
 		global $give_options;
 		$give_options = $options;
 	}
@@ -920,9 +920,9 @@ 
  */
 function give_get_settings() {
 
-	$settings = get_option( 'give_settings' );
+	$settings = get_option('give_settings');
 
-	return (array) apply_filters( 'give_get_settings', $settings );
+	return (array) apply_filters('give_get_settings', $settings);
 
 }
 
@@ -940,25 +940,25 @@ 
  *
  * @return array
  */
-function give_settings_array_insert( $array, $position, $insert ) {
-	if ( is_int( $position ) ) {
-		array_splice( $array, $position, 0, $insert );
+function give_settings_array_insert($array, $position, $insert) {
+	if (is_int($position)) {
+		array_splice($array, $position, 0, $insert);
 	} else {
 
-		foreach ( $array as $index => $subarray ) {
-			if ( isset( $subarray['id'] ) && $subarray['id'] == $position ) {
+		foreach ($array as $index => $subarray) {
+			if (isset($subarray['id']) && $subarray['id'] == $position) {
 				$pos = $index;
 			}
 		}
 
-		if ( ! isset( $pos ) ) {
+		if ( ! isset($pos)) {
 			return $array;
 		}
 
 		$array = array_merge(
-			array_slice( $array, 0, $pos ),
+			array_slice($array, 0, $pos),
 			$insert,
-			array_slice( $array, $pos )
+			array_slice($array, $pos)
 		);
 	}
 
@@ -978,23 +978,23 @@ 
  *
  * @return void
  */
-function give_enabled_gateways_callback( $field_arr, $saved_values = array() ) {
+function give_enabled_gateways_callback($field_arr, $saved_values = array()) {
 	$saved_values = __give_validate_active_gateways($saved_values);
 	$id       = $field_arr['id'];
-	$gateways = give_get_ordered_payment_gateways( give_get_payment_gateways() );
+	$gateways = give_get_ordered_payment_gateways(give_get_payment_gateways());
 
 	echo '<ul class="give-checklist-fields give-payment-gatways-list">';
 
-	foreach ( $gateways as $key => $option ) :
+	foreach ($gateways as $key => $option) :
 
-		if ( is_array( $saved_values ) && array_key_exists( $key, $saved_values ) ) {
+		if (is_array($saved_values) && array_key_exists($key, $saved_values)) {
 			$enabled = '1';
 		} else {
 			$enabled = null;
 		}
 
-		echo '<li><span class="give-drag-handle"><span class="dashicons dashicons-menu"></span></span><input name="' . $id . '[' . $key . ']" id="' . $id . '[' . $key . ']" type="checkbox" value="1" ' . checked( '1', $enabled, false ) . '/>&nbsp;';
-		echo '<label for="' . $id . '[' . $key . ']">' . $option['admin_label'] . '</label></li>';
+		echo '<li><span class="give-drag-handle"><span class="dashicons dashicons-menu"></span></span><input name="'.$id.'['.$key.']" id="'.$id.'['.$key.']" type="checkbox" value="1" '.checked('1', $enabled, false).'/>&nbsp;';
+		echo '<label for="'.$id.'['.$key.']">'.$option['admin_label'].'</label></li>';
 
 	endforeach;
 
@@ -1013,16 +1013,16 @@ 
  *
  * @return void
  */
-function give_default_gateway_callback( $field_arr, $saved_value ) {
+function give_default_gateway_callback($field_arr, $saved_value) {
 	$id          = $field_arr['id'];
 	$gateways    = give_get_enabled_payment_gateways();
-	$saved_value = give_get_default_gateway( null );
+	$saved_value = give_get_default_gateway(null);
 
-	echo '<select class="give-select" name="' . $id . '" id="' . $id . '">';
+	echo '<select class="give-select" name="'.$id.'" id="'.$id.'">';
 
-	foreach ( $gateways as $key => $option ) :
-		$selected = isset( $saved_value ) ? selected( $key, $saved_value, false ) : '';
-		echo '<option value="' . esc_attr( $key ) . '"' . $selected . '>' . esc_html( $option['admin_label'] ) . '</option>';
+	foreach ($gateways as $key => $option) :
+		$selected = isset($saved_value) ? selected($key, $saved_value, false) : '';
+		echo '<option value="'.esc_attr($key).'"'.$selected.'>'.esc_html($option['admin_label']).'</option>';
 	endforeach;
 
 	echo '</select>';
@@ -1040,13 +1040,13 @@ 
  *
  * @return void
  */
-function give_title_callback( $field_object, $escaped_value, $object_id, $object_type, $field_type_object ) {
+function give_title_callback($field_object, $escaped_value, $object_id, $object_type, $field_type_object) {
 
 	$id                = $field_type_object->field->args['id'];
 	$title             = $field_type_object->field->args['name'];
 	$field_description = $field_type_object->field->args['desc'];
 
-	echo '<hr>' . $field_description;
+	echo '<hr>'.$field_description;
 
 }
 
@@ -1061,7 +1061,7 @@ 
  *
  * @return void
  */
-function give_description_callback( $field_object, $escaped_value, $object_id, $object_type, $field_type_object ) {
+function give_description_callback($field_object, $escaped_value, $object_id, $object_type, $field_type_object) {
 
 	$id                = $field_type_object->field->args['id'];
 	$title             = $field_type_object->field->args['name'];
@@ -1080,25 +1080,25 @@ 
  * @see: https://github.com/WebDevStudios/CMB2/wiki/Adding-your-own-field-types
  * @return array An array of options that matches the CMB2 options array
  */
-function give_cmb2_get_post_options( $query_args, $force = false ) {
+function give_cmb2_get_post_options($query_args, $force = false) {
 
-	$post_options = array( '' => '' ); // Blank option
+	$post_options = array('' => ''); // Blank option
 
-	if ( ( ! isset( $_GET['page'] ) || 'give-settings' != $_GET['page'] ) && ! $force ) {
+	if (( ! isset($_GET['page']) || 'give-settings' != $_GET['page']) && ! $force) {
 		return $post_options;
 	}
 
-	$args = wp_parse_args( $query_args, array(
+	$args = wp_parse_args($query_args, array(
 		'post_type'   => 'page',
 		'numberposts' => 10,
-	) );
+	));
 
-	$posts = get_posts( $args );
+	$posts = get_posts($args);
 
-	if ( $posts ) {
-		foreach ( $posts as $post ) {
+	if ($posts) {
+		foreach ($posts as $post) {
 
-			$post_options[ $post->ID ] = $post->post_title;
+			$post_options[$post->ID] = $post->post_title;
 
 		}
 	}
@@ -1123,33 +1123,33 @@ 
 
 	$sizes            = array();
 	$get_sizes        = get_intermediate_image_sizes();
-	$core_image_sizes = array( 'thumbnail', 'medium', 'medium_large', 'large' );
+	$core_image_sizes = array('thumbnail', 'medium', 'medium_large', 'large');
 
 
 	// This will help us to filter special characters from a string
-	$filter_slug_items = array( '_', '-' );
+	$filter_slug_items = array('_', '-');
 
-	foreach ( $get_sizes as $_size ) {
+	foreach ($get_sizes as $_size) {
 
 		// Converting image size slug to title case
-		$sizes[ $_size ] = give_slug_to_title( $_size, $filter_slug_items );
+		$sizes[$_size] = give_slug_to_title($_size, $filter_slug_items);
 
-		if ( in_array( $_size, $core_image_sizes ) ) {
-			$sizes[ $_size ] .= ' (' . get_option( "{$_size}_size_w" ) . 'x' . get_option( "{$_size}_size_h" );
-		} elseif ( isset( $_wp_additional_image_sizes[ $_size ] ) ) {
-			$sizes[ $_size ] .= " ({$_wp_additional_image_sizes[ $_size ]['width']} x {$_wp_additional_image_sizes[ $_size ]['height']}";
+		if (in_array($_size, $core_image_sizes)) {
+			$sizes[$_size] .= ' ('.get_option("{$_size}_size_w").'x'.get_option("{$_size}_size_h");
+		} elseif (isset($_wp_additional_image_sizes[$_size])) {
+			$sizes[$_size] .= " ({$_wp_additional_image_sizes[$_size]['width']} x {$_wp_additional_image_sizes[$_size]['height']}";
 		}
 
 		// Based on the above image height check, label the respective resolution as responsive
-		if ( ( array_key_exists( $_size, $_wp_additional_image_sizes ) && ! $_wp_additional_image_sizes[ $_size ]['crop'] ) || ( in_array( $_size, $core_image_sizes ) && ! get_option( "{$_size}_crop" ) ) ) {
-			$sizes[ $_size ] .= ' - responsive';
+		if ((array_key_exists($_size, $_wp_additional_image_sizes) && ! $_wp_additional_image_sizes[$_size]['crop']) || (in_array($_size, $core_image_sizes) && ! get_option("{$_size}_crop"))) {
+			$sizes[$_size] .= ' - responsive';
 		}
 
-		$sizes[ $_size ] .= ')';
+		$sizes[$_size] .= ')';
 
 	}
 
-	return apply_filters( 'give_get_featured_image_sizes', $sizes );
+	return apply_filters('give_get_featured_image_sizes', $sizes);
 }
 
 
@@ -1165,14 +1165,14 @@ 
  *
  * @return text $string
  */
-function give_slug_to_title( $string, $filters = array() ) {
+function give_slug_to_title($string, $filters = array()) {
 
-	foreach ( $filters as $filter_item ) {
-		$string = str_replace( $filter_item, ' ', $string );
+	foreach ($filters as $filter_item) {
+		$string = str_replace($filter_item, ' ', $string);
 	}
 
 	// Return updated string after converting it to title case
-	return ucwords( $string );
+	return ucwords($string);
 
 }
 
@@ -1188,18 +1188,18 @@ 
  *
  * @return void
  */
-function give_license_key_callback( $field_object, $escaped_value, $object_id, $object_type, $field_type_object ) {
+function give_license_key_callback($field_object, $escaped_value, $object_id, $object_type, $field_type_object) {
 	/* @var CMB2_Types $field_type_object */
 
 	$id                 = $field_type_object->field->args['id'];
 	$field_description  = $field_type_object->field->args['desc'];
 	$license            = $field_type_object->field->args['options']['license'];
 	$license_key        = $escaped_value;
-	$is_license_key     = apply_filters( 'give_is_license_key', ( is_object( $license ) && ! empty( $license ) ) );
-	$is_valid_license   = apply_filters( 'give_is_valid_license', ( $is_license_key && property_exists( $license, 'license' ) && 'valid' === $license->license ) );
+	$is_license_key     = apply_filters('give_is_license_key', (is_object($license) && ! empty($license)));
+	$is_valid_license   = apply_filters('give_is_valid_license', ($is_license_key && property_exists($license, 'license') && 'valid' === $license->license));
 	$shortname          = $field_type_object->field->args['options']['shortname'];
 	$field_classes      = 'regular-text give-license-field';
-	$type               = empty( $escaped_value ) || ! $is_valid_license ? 'text' : 'password';
+	$type               = empty($escaped_value) || ! $is_valid_license ? 'text' : 'password';
 	$custom_html        = '';
 	$messages           = array();
 	$class              = '';
@@ -1211,83 +1211,83 @@ 
 
 	// By default query on edd api url will return license object which contain status and message property, this can break below functionality.
 	// To combat that check if status is set to error or not, if yes then set $is_license_key to false.
-	if ( $is_license_key && property_exists( $license, 'status' ) && 'error' === $license->status ) {
+	if ($is_license_key && property_exists($license, 'status') && 'error' === $license->status) {
 		$is_license_key = false;
 	}
 
 	// Check if current license is part of subscription or not.
-	$subscriptions = get_option( 'give_subscriptions' );
+	$subscriptions = get_option('give_subscriptions');
 
-	if ( $is_license_key && $subscriptions ) {
-		foreach ( $subscriptions as $subscription ) {
-			if ( in_array( $license_key, $subscription['licenses'] ) ) {
+	if ($is_license_key && $subscriptions) {
+		foreach ($subscriptions as $subscription) {
+			if (in_array($license_key, $subscription['licenses'])) {
 				$is_in_subscription = $subscription['id'];
 				break;
 			}
 		}
 	}
 
-	if ( $is_license_key ) {
+	if ($is_license_key) {
 
-		if ( empty( $license->success ) && property_exists( $license, 'error' ) ) {
+		if (empty($license->success) && property_exists($license, 'error')) {
 
 			// activate_license 'invalid' on anything other than valid, so if there was an error capture it
-			switch ( $license->error ) {
+			switch ($license->error) {
 				case 'expired' :
 					$class          = $license->error;
 					$messages[]     = sprintf(
-						__( 'Your license key expired on %1$s. Please <a href="%2$s" target="_blank" title="Renew your license key">renew your license key</a>.', 'give' ),
-						date_i18n( get_option( 'date_format' ), strtotime( $license->expires, current_time( 'timestamp' ) ) ),
-						$checkout_page_link . '?edd_license_key=' . $license_key . '&utm_campaign=admin&utm_source=licenses&utm_medium=expired'
+						__('Your license key expired on %1$s. Please <a href="%2$s" target="_blank" title="Renew your license key">renew your license key</a>.', 'give'),
+						date_i18n(get_option('date_format'), strtotime($license->expires, current_time('timestamp'))),
+						$checkout_page_link.'?edd_license_key='.$license_key.'&utm_campaign=admin&utm_source=licenses&utm_medium=expired'
 					);
-					$license_status = 'license-' . $class;
+					$license_status = 'license-'.$class;
 					break;
 
 				case 'missing' :
 					$class          = $license->error;
 					$messages[]     = sprintf(
-						__( 'Invalid license. Please <a href="%s" target="_blank" title="Visit account page">visit your account page</a> and verify it.', 'give' ),
-						$account_page_link . '?utm_campaign=admin&utm_source=licenses&utm_medium=missing'
+						__('Invalid license. Please <a href="%s" target="_blank" title="Visit account page">visit your account page</a> and verify it.', 'give'),
+						$account_page_link.'?utm_campaign=admin&utm_source=licenses&utm_medium=missing'
 					);
-					$license_status = 'license-' . $class;
+					$license_status = 'license-'.$class;
 					break;
 
 				case 'invalid' :
 					$class          = $license->error;
 					$messages[]     = sprintf(
-						__( 'Your %1$s is not active for this URL. Please <a href="%2$s" target="_blank" title="Visit account page">visit your account page</a> to manage your license key URLs.', 'give' ),
+						__('Your %1$s is not active for this URL. Please <a href="%2$s" target="_blank" title="Visit account page">visit your account page</a> to manage your license key URLs.', 'give'),
 						$addon_name,
-						$account_page_link . '?utm_campaign=admin&utm_source=licenses&utm_medium=invalid'
+						$account_page_link.'?utm_campaign=admin&utm_source=licenses&utm_medium=invalid'
 					);
-					$license_status = 'license-' . $class;
+					$license_status = 'license-'.$class;
 					break;
 
 				case 'site_inactive' :
 					$class          = $license->error;
 					$messages[]     = sprintf(
-						__( 'Your %1$s is not active for this URL. Please <a href="%2$s" target="_blank" title="Visit account page">visit your account page</a> to manage your license key URLs.', 'give' ),
+						__('Your %1$s is not active for this URL. Please <a href="%2$s" target="_blank" title="Visit account page">visit your account page</a> to manage your license key URLs.', 'give'),
 						$addon_name,
-						$account_page_link . '?utm_campaign=admin&utm_source=licenses&utm_medium=invalid'
+						$account_page_link.'?utm_campaign=admin&utm_source=licenses&utm_medium=invalid'
 					);
-					$license_status = 'license-' . $class;
+					$license_status = 'license-'.$class;
 					break;
 
 				case 'item_name_mismatch' :
 					$class          = $license->error;
-					$messages[]     = sprintf( __( 'This license %1$s does not belong to %2$s.', 'give' ), $license_key, $addon_name );
-					$license_status = 'license-' . $class;
+					$messages[]     = sprintf(__('This license %1$s does not belong to %2$s.', 'give'), $license_key, $addon_name);
+					$license_status = 'license-'.$class;
 					break;
 
 				case 'no_activations_left':
 					$class          = $license->error;
-					$messages[]     = sprintf( __( 'Your license key has reached it\'s activation limit. <a href="%s">View possible upgrades</a> now.', 'give' ), $account_page_link );
-					$license_status = 'license-' . $class;
+					$messages[]     = sprintf(__('Your license key has reached it\'s activation limit. <a href="%s">View possible upgrades</a> now.', 'give'), $account_page_link);
+					$license_status = 'license-'.$class;
 					break;
 
 				default:
 					$class          = $license->error;
 					$messages[]     = sprintf(
-						__( 'Your license is not activated. Please <a href="%3$s" target="_blank" title="Visit account page">visit your account page</a> to manage your license key URLs. %2$sError Code: %1$s.', 'give' ),
+						__('Your license is not activated. Please <a href="%3$s" target="_blank" title="Visit account page">visit your account page</a> to manage your license key URLs. %2$sError Code: %1$s.', 'give'),
 						$license->error,
 						'<br/>',
 						"{$account_page_link}?utm_campaign=admin&utm_source=licenses&utm_medium={$license->error}"
@@ -1296,74 +1296,74 @@ 
 					break;
 			}
 
-		} elseif ( $is_in_subscription ) {
+		} elseif ($is_in_subscription) {
 
-			$subscription_expires = strtotime( $subscriptions[ $is_in_subscription ]['expires'] );
-			$subscription_status  = __( 'renew', 'give' );
+			$subscription_expires = strtotime($subscriptions[$is_in_subscription]['expires']);
+			$subscription_status  = __('renew', 'give');
 
-			if ( ( 'active' !== $subscriptions[ $is_in_subscription ]['status'] ) ) {
-				$subscription_status = __( 'expire', 'give' );
+			if (('active' !== $subscriptions[$is_in_subscription]['status'])) {
+				$subscription_status = __('expire', 'give');
 			}
 
-			if ( $subscription_expires < current_time( 'timestamp', 1 ) ) {
-				$messages[]     = sprintf(
-					__( 'Your subscription (<a href="%1$s" target="_blank">#%2$d</a>) expired. Please <a href="%3$s" target="_blank" title="Renew your license key">renew your license key</a>', 'give' ),
-					urldecode( $subscriptions[ $is_in_subscription ]['invoice_url'] ),
-					$subscriptions[ $is_in_subscription ]['payment_id'],
-					$checkout_page_link . '?edd_license_key=' . $subscriptions[ $is_in_subscription ]['license_key'] . '&utm_campaign=admin&utm_source=licenses&utm_medium=expired'
+			if ($subscription_expires < current_time('timestamp', 1)) {
+				$messages[] = sprintf(
+					__('Your subscription (<a href="%1$s" target="_blank">#%2$d</a>) expired. Please <a href="%3$s" target="_blank" title="Renew your license key">renew your license key</a>', 'give'),
+					urldecode($subscriptions[$is_in_subscription]['invoice_url']),
+					$subscriptions[$is_in_subscription]['payment_id'],
+					$checkout_page_link.'?edd_license_key='.$subscriptions[$is_in_subscription]['license_key'].'&utm_campaign=admin&utm_source=licenses&utm_medium=expired'
 				);
 				$license_status = 'license-expired';
-			} elseif ( strtotime( '- 7 days', $subscription_expires ) < current_time( 'timestamp', 1 ) ) {
+			} elseif (strtotime('- 7 days', $subscription_expires) < current_time('timestamp', 1)) {
 				$messages[]     = sprintf(
-					__( 'Your subscription (<a href="%1$s" target="_blank">#%2$d</a>) will %3$s in %4$s.', 'give' ),
-					urldecode( $subscriptions[ $is_in_subscription ]['invoice_url'] ),
-					$subscriptions[ $is_in_subscription ]['payment_id'],
+					__('Your subscription (<a href="%1$s" target="_blank">#%2$d</a>) will %3$s in %4$s.', 'give'),
+					urldecode($subscriptions[$is_in_subscription]['invoice_url']),
+					$subscriptions[$is_in_subscription]['payment_id'],
 					$subscription_status,
-					human_time_diff( current_time( 'timestamp', 1 ), strtotime( $subscriptions[ $is_in_subscription ]['expires'] ) )
+					human_time_diff(current_time('timestamp', 1), strtotime($subscriptions[$is_in_subscription]['expires']))
 				);
 				$license_status = 'license-expires-soon';
 			} else {
 				$messages[]     = sprintf(
-					__( 'Your subscription (<a href="%1$s" target="_blank">#%2$d</a>) will %3$s on %4$s.', 'give' ),
-					urldecode( $subscriptions[ $is_in_subscription ]['invoice_url'] ),
-					$subscriptions[ $is_in_subscription ]['payment_id'],
+					__('Your subscription (<a href="%1$s" target="_blank">#%2$d</a>) will %3$s on %4$s.', 'give'),
+					urldecode($subscriptions[$is_in_subscription]['invoice_url']),
+					$subscriptions[$is_in_subscription]['payment_id'],
 					$subscription_status,
-					date_i18n( get_option( 'date_format' ), strtotime( $subscriptions[ $is_in_subscription ]['expires'], current_time( 'timestamp' ) ) )
+					date_i18n(get_option('date_format'), strtotime($subscriptions[$is_in_subscription]['expires'], current_time('timestamp')))
 				);
 				$license_status = 'license-expiration-date';
 			}
 
-		} elseif ( empty( $license->success ) ) {
+		} elseif (empty($license->success)) {
 			$class          = 'invalid';
 			$messages[]     = sprintf(
-				__( 'Your %1$s is not active for this URL. Please <a href="%2$s" target="_blank" title="Visit account page">visit your account page</a> to manage your license key URLs.', 'give' ),
+				__('Your %1$s is not active for this URL. Please <a href="%2$s" target="_blank" title="Visit account page">visit your account page</a> to manage your license key URLs.', 'give'),
 				$addon_name,
-				$account_page_link . '?utm_campaign=admin&utm_source=licenses&utm_medium=invalid'
+				$account_page_link.'?utm_campaign=admin&utm_source=licenses&utm_medium=invalid'
 			);
-			$license_status = 'license-' . $class;
+			$license_status = 'license-'.$class;
 
 		} else {
-			switch ( $license->license ) {
+			switch ($license->license) {
 				case 'valid' :
 				default:
 					$class      = 'valid';
-					$now        = current_time( 'timestamp' );
-					$expiration = strtotime( $license->expires, current_time( 'timestamp' ) );
+					$now        = current_time('timestamp');
+					$expiration = strtotime($license->expires, current_time('timestamp'));
 
-					if ( 'lifetime' === $license->expires ) {
-						$messages[]     = __( 'License key never expires.', 'give' );
+					if ('lifetime' === $license->expires) {
+						$messages[]     = __('License key never expires.', 'give');
 						$license_status = 'license-lifetime-notice';
-					} elseif ( $expiration > $now && $expiration - $now < ( DAY_IN_SECONDS * 30 ) ) {
+					} elseif ($expiration > $now && $expiration - $now < (DAY_IN_SECONDS * 30)) {
 						$messages[]     = sprintf(
-							__( 'Your license key expires soon! It expires on %1$s. <a href="%2$s" target="_blank" title="Renew license">Renew your license key</a>.', 'give' ),
-							date_i18n( get_option( 'date_format' ), strtotime( $license->expires, current_time( 'timestamp' ) ) ),
-							$checkout_page_link . '?edd_license_key=' . $license_key . '&utm_campaign=admin&utm_source=licenses&utm_medium=renew'
+							__('Your license key expires soon! It expires on %1$s. <a href="%2$s" target="_blank" title="Renew license">Renew your license key</a>.', 'give'),
+							date_i18n(get_option('date_format'), strtotime($license->expires, current_time('timestamp'))),
+							$checkout_page_link.'?edd_license_key='.$license_key.'&utm_campaign=admin&utm_source=licenses&utm_medium=renew'
 						);
 						$license_status = 'license-expires-soon';
 					} else {
 						$messages[]     = sprintf(
-							__( 'Your license key expires on %s.', 'give' ),
-							date_i18n( get_option( 'date_format' ), strtotime( $license->expires, current_time( 'timestamp' ) ) )
+							__('Your license key expires on %s.', 'give'),
+							date_i18n(get_option('date_format'), strtotime($license->expires, current_time('timestamp')))
 						);
 						$license_status = 'license-expiration-date';
 					}
@@ -1371,15 +1371,15 @@ 
 			}
 		}
 	} else {
-		$messages[]     = sprintf(
-			__( 'To receive updates, please enter your valid %s license key.', 'give' ),
+		$messages[] = sprintf(
+			__('To receive updates, please enter your valid %s license key.', 'give'),
 			$addon_name
 		);
 		$license_status = 'inactive';
 	}
 
 	// Add class for input field if license is active.
-	if ( $is_valid_license ) {
+	if ($is_valid_license) {
 		$field_classes .= ' give-license-active';
 	}
 
@@ -1387,33 +1387,33 @@ 
 	$input_field_html = "<input type=\"{$type}\" name=\"{$id}\" class=\"{$field_classes}\" value=\"{$license_key}\">";
 
 	// If license is active so show deactivate button.
-	if ( $is_valid_license ) {
+	if ($is_valid_license) {
 		// Get input field html.
 		$input_field_html = "<input type=\"{$type}\" name=\"{$id}\" class=\"{$field_classes}\" value=\"{$license_key}\" readonly=\"readonly\">";
 
-		$custom_html = '<input type="submit" class="button button-small give-license-deactivate" name="' . $id . '_deactivate" value="' . esc_attr__( 'Deactivate License', 'give' ) . '"/>';
+		$custom_html = '<input type="submit" class="button button-small give-license-deactivate" name="'.$id.'_deactivate" value="'.esc_attr__('Deactivate License', 'give').'"/>';
 
 	}
 
 	// Field description.
-	$custom_html .= '<label for="give_settings[' . $id . ']"> ' . $field_description . '</label>';
+	$custom_html .= '<label for="give_settings['.$id.']"> '.$field_description.'</label>';
 
 	// If no messages found then inform user that to get updated in future register yourself.
-	if ( empty( $messages ) ) {
-		$messages[] = apply_filters( "{$shortname}_default_addon_notice", __( 'To receive updates, please enter your valid license key.', 'give' ) );
+	if (empty($messages)) {
+		$messages[] = apply_filters("{$shortname}_default_addon_notice", __('To receive updates, please enter your valid license key.', 'give'));
 	}
 
-	foreach ( $messages as $message ) {
-		$custom_html .= '<div class="give-license-status-notice give-' . $license_status . '">';
-		$custom_html .= '<p>' . $message . '</p>';
+	foreach ($messages as $message) {
+		$custom_html .= '<div class="give-license-status-notice give-'.$license_status.'">';
+		$custom_html .= '<p>'.$message.'</p>';
 		$custom_html .= '</div>';
 	}
 
 	// Field html.
-	$custom_html = apply_filters( 'give_license_key_field_html', $input_field_html . $custom_html, $field_type_object );
+	$custom_html = apply_filters('give_license_key_field_html', $input_field_html.$custom_html, $field_type_object);
 
 	// Nonce.
-	wp_nonce_field( $id . '-nonce', $id . '-nonce' );
+	wp_nonce_field($id.'-nonce', $id.'-nonce');
 
 	// Print field html.
 	echo "<div class=\"give-license-key\"><label for=\"{$id}\">{$addon_name }</label></div><div class=\"give-license-block\">{$custom_html}</div>";
@@ -1428,7 +1428,7 @@ 
  */
 function give_api_callback() {
 
-	if ( ! current_user_can( 'manage_give_settings' ) ) {
+	if ( ! current_user_can('manage_give_settings')) {
 		return;
 	}
 
@@ -1437,9 +1437,9 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_tools_api_keys_before' );
+	do_action('give_tools_api_keys_before');
 
-	require_once GIVE_PLUGIN_DIR . 'includes/admin/class-api-keys-table.php';
+	require_once GIVE_PLUGIN_DIR.'includes/admin/class-api-keys-table.php';
 
 	$api_keys_table = new Give_API_Keys_Table();
 	$api_keys_table->prepare_items();
@@ -1448,9 +1448,9 @@ 
 	<span class="cmb2-metabox-description api-description">
 		<?php echo sprintf(
 		/* translators: 1: http://docs.givewp.com/api 2: http://docs.givewp.com/addon-zapier */
-			__( 'You can create API keys for individual users within their profile edit screen. API keys allow users to use the <a href="%1$s" target="_blank">Give REST API</a> to retrieve donation data in JSON or XML for external applications or devices, such as <a href="%2$s" target="_blank">Zapier</a>.', 'give' ),
-			esc_url( 'http://docs.givewp.com/api' ),
-			esc_url( 'http://docs.givewp.com/addon-zapier' )
+			__('You can create API keys for individual users within their profile edit screen. API keys allow users to use the <a href="%1$s" target="_blank">Give REST API</a> to retrieve donation data in JSON or XML for external applications or devices, such as <a href="%2$s" target="_blank">Zapier</a>.', 'give'),
+			esc_url('http://docs.givewp.com/api'),
+			esc_url('http://docs.givewp.com/addon-zapier')
 		); ?>
 	</span>
 	<?php
@@ -1460,10 +1460,10 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_tools_api_keys_after' );
+	do_action('give_tools_api_keys_after');
 }
 
-add_action( 'give_settings_tab_api_keys', 'give_api_callback' );
+add_action('give_settings_tab_api_keys', 'give_api_callback');
 
 /**
  * Hook Callback
@@ -1476,7 +1476,7 @@ 
  *
  * @return void
  */
-function give_hook_callback( $args ) {
+function give_hook_callback($args) {
 
 	$id = $args['id'];
 
@@ -1485,7 +1485,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( "give_{$id}" );
+	do_action("give_{$id}");
 
 }
 
@@ -1500,19 +1500,19 @@ 
  *
  * @return bool
  */
-function give_is_setting_enabled( $value, $compare_with = null ) {
-	if ( ! is_null( $compare_with ) ) {
+function give_is_setting_enabled($value, $compare_with = null) {
+	if ( ! is_null($compare_with)) {
 
-		if ( is_array( $compare_with ) ) {
+		if (is_array($compare_with)) {
 			// Output.
-			return in_array( $value, $compare_with );
+			return in_array($value, $compare_with);
 		}
 
 		// Output.
-		return ( $value === $compare_with );
+		return ($value === $compare_with);
 	}
 
 	// Backward compatibility: From version 1.8 most of setting is modified to enabled/disabled
 	// Output.
-	return ( in_array( $value, array( 'enabled', 'on', 'yes' ) ) ? true : false );
+	return (in_array($value, array('enabled', 'on', 'yes')) ? true : false);
 }

includes/admin/tools/export/give-export-donations-functions.php

Doc Comments +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
  *
  * @see http://wordpress.stackexchange.com/questions/58834/echo-all-meta-keys-of-a-custom-post-type
  *
- * @return string
+ * @return false|null
  */
 function give_export_donations_get_custom_fields() {
 

Spacing +72 added lines, -72 removed lines

@@ -17,18 +17,18 @@ 
 	$post_type = 'give_payment';
 	$responses = array();
 
-	$form_id = isset( $_POST['form_id'] ) ? intval( $_POST['form_id'] ) : '';
+	$form_id = isset($_POST['form_id']) ? intval($_POST['form_id']) : '';
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$args          = array(
-		'give_forms'     => array( $form_id ),
-		'posts_per_page' => - 1,
+	$args = array(
+		'give_forms'     => array($form_id),
+		'posts_per_page' => -1,
 		'fields'         => 'ids',
 	);
-	$donation_list = implode( ',', (array) give_get_payments( $args ) );
+	$donation_list = implode(',', (array) give_get_payments($args));
 
 	$query_and = sprintf(
 		"AND $wpdb->posts.ID IN (%s) 
@@ -43,12 +43,12 @@ 
         LEFT JOIN $wpdb->paymentmeta 
         ON $wpdb->posts.ID = $wpdb->paymentmeta.payment_id
         WHERE $wpdb->posts.post_type = '%s'
-    " . $query_and;
+    ".$query_and;
 
-	$meta_keys = $wpdb->get_col( $wpdb->prepare( $query, $post_type ) );
+	$meta_keys = $wpdb->get_col($wpdb->prepare($query, $post_type));
 
-	if ( ! empty( $meta_keys ) ) {
-		$responses['standard_fields'] = array_values( $meta_keys );
+	if ( ! empty($meta_keys)) {
+		$responses['standard_fields'] = array_values($meta_keys);
 	}
 
 	$query_and = sprintf(
@@ -64,9 +64,9 @@ 
         LEFT JOIN $wpdb->paymentmeta 
         ON $wpdb->posts.ID = $wpdb->paymentmeta.payment_id 
         WHERE $wpdb->posts.post_type = '%s'
-    " . $query_and;
+    ".$query_and;
 
-	$hidden_meta_keys   = $wpdb->get_col( $wpdb->prepare( $query, $post_type ) );
+	$hidden_meta_keys = $wpdb->get_col($wpdb->prepare($query, $post_type));
 
 	/**
 	 * Filter to modify hidden keys that are going to be ignore when displaying the hidden keys
@@ -78,7 +78,7 @@ 
 	 *
 	 * @return array $ignore_hidden_keys Hidden keys that are going to be ignore
 	 */
-	$ignore_hidden_keys = apply_filters( 'give_export_donations_ignore_hidden_keys', array(
+	$ignore_hidden_keys = apply_filters('give_export_donations_ignore_hidden_keys', array(
 		'_give_payment_meta',
 		'_give_payment_gateway',
 		'_give_payment_form_title',
@@ -111,14 +111,14 @@ 
 	);
 
 	// Unset ignored hidden keys.
-	foreach ( $ignore_hidden_keys as $key ) {
-		if ( ( $key = array_search( $key, $hidden_meta_keys ) ) !== false ) {
-			unset( $hidden_meta_keys[ $key ] );
+	foreach ($ignore_hidden_keys as $key) {
+		if (($key = array_search($key, $hidden_meta_keys)) !== false) {
+			unset($hidden_meta_keys[$key]);
 		}
 	}
 
-	if ( ! empty( $hidden_meta_keys ) ) {
-		$responses['hidden_fields'] = array_values( $hidden_meta_keys );
+	if ( ! empty($hidden_meta_keys)) {
+		$responses['hidden_fields'] = array_values($hidden_meta_keys);
 	}
 
 	/**
@@ -131,11 +131,11 @@ 
 	 *
 	 * @return array $responses
 	 */
-	wp_send_json( (array) apply_filters( 'give_export_donations_get_custom_fields', $responses, $form_id ) );
+	wp_send_json((array) apply_filters('give_export_donations_get_custom_fields', $responses, $form_id));
 
 }
 
-add_action( 'wp_ajax_give_export_donations_get_custom_fields', 'give_export_donations_get_custom_fields' );
+add_action('wp_ajax_give_export_donations_get_custom_fields', 'give_export_donations_get_custom_fields');
 
 /**
  * Register the payments batch exporter
@@ -143,10 +143,10 @@ 
  * @since  1.0
  */
 function give_register_export_donations_batch_export() {
-	add_action( 'give_batch_export_class_include', 'give_export_donations_include_export_class', 10, 1 );
+	add_action('give_batch_export_class_include', 'give_export_donations_include_export_class', 10, 1);
 }
 
-add_action( 'give_register_batch_exporter', 'give_register_export_donations_batch_export', 10 );
+add_action('give_register_batch_exporter', 'give_register_export_donations_batch_export', 10);
 
 
 /**
@@ -154,9 +154,9 @@ 
  *
  * @param $class Give_Export_Donations_CSV
  */
-function give_export_donations_include_export_class( $class ) {
-	if ( 'Give_Export_Donations_CSV' === $class ) {
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/give-export-donations-exporter.php';
+function give_export_donations_include_export_class($class) {
+	if ('Give_Export_Donations_CSV' === $class) {
+		require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/give-export-donations-exporter.php';
 	}
 }
 
@@ -168,8 +168,8 @@ 
  *
  * @return string
  */
-function give_export_donations_create_column_key( $string ) {
-	return sanitize_key( str_replace( ' ', '_', $string ) );
+function give_export_donations_create_column_key($string) {
+	return sanitize_key(str_replace(' ', '_', $string));
 }
 
 /**
@@ -181,19 +181,19 @@ 
  *
  * @return array
  */
-function give_export_donation_form_search_args( $args ) {
-	if ( empty( $_POST['fields'] ) ) {
+function give_export_donation_form_search_args($args) {
+	if (empty($_POST['fields'])) {
 		return $args;
 	}
 
-	$fields = isset( $_POST['fields'] ) ? $_POST['fields'] : null;
-	parse_str( $fields );
+	$fields = isset($_POST['fields']) ? $_POST['fields'] : null;
+	parse_str($fields);
 
-	if ( ! empty( $give_forms_categories ) && ! empty( $give_forms_tags ) ) {
+	if ( ! empty($give_forms_categories) && ! empty($give_forms_tags)) {
 		$args['tax_query']['relation'] = 'AND';
 	}
 
-	if ( ! empty( $give_forms_categories ) ) {
+	if ( ! empty($give_forms_categories)) {
 		$args['tax_query'][] = array(
 			'taxonomy' => 'give_forms_category',
 			'field'    => 'term_id',
@@ -202,7 +202,7 @@ 
 		);
 	}
 
-	if ( ! empty( $give_forms_tags ) ) {
+	if ( ! empty($give_forms_tags)) {
 		$args['tax_query'][] = array(
 			'taxonomy' => 'give_forms_tag',
 			'field'    => 'term_id',
@@ -214,7 +214,7 @@ 
 	return $args;
 }
 
-add_filter( 'give_ajax_form_search_args', 'give_export_donation_form_search_args' );
+add_filter('give_ajax_form_search_args', 'give_export_donation_form_search_args');
 
 /**
  * Add Donation standard fields in export donation page
@@ -225,7 +225,7 @@ 
 	?>
 	<tr>
 		<td scope="row" class="row-title">
-			<label><?php _e( 'Standard Columns:', 'give' ); ?></label>
+			<label><?php _e('Standard Columns:', 'give'); ?></label>
 		</td>
 		<td>
 			<div class="give-clearfix">
@@ -235,7 +235,7 @@ 
 
 							<li class="give-export-option-label give-export-option-donation-label">
 								<span>
-									<?php _e( 'Donation Payment Fields', 'give' ); ?>
+									<?php _e('Donation Payment Fields', 'give'); ?>
 								</span>
 							</li>
 
@@ -243,18 +243,18 @@ 
 								<label for="give-export-donation-id">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[donation_id]"
-									       id="give-export-donation-id"><?php _e( 'Donation ID', 'give' ); ?>
+									       id="give-export-donation-id"><?php _e('Donation ID', 'give'); ?>
 								</label>
 							</li>
 
 							<?php
-							if ( give_is_setting_enabled( give_get_option( 'sequential-ordering_status', 'disabled' ) ) ) {
+							if (give_is_setting_enabled(give_get_option('sequential-ordering_status', 'disabled'))) {
 								?>
 								<li>
 									<label for="give-export-seq-id">
 										<input type="checkbox" checked
 										       name="give_give_donations_export_option[seq_id]"
-										       id="give-export-seq-id"><?php _e( 'Donation Number', 'give' ); ?>
+										       id="give-export-seq-id"><?php _e('Donation Number', 'give'); ?>
 									</label>
 								</li>
 								<?php
@@ -265,7 +265,7 @@ 
 								<label for="give-export-donation-sum">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[donation_total]"
-									       id="give-export-donation-sum"><?php _e( 'Donation Total', 'give' ); ?>
+									       id="give-export-donation-sum"><?php _e('Donation Total', 'give'); ?>
 								</label>
 							</li>
 
@@ -273,7 +273,7 @@ 
 								<label for="give-export-donation-currency_code">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[currency_code]"
-									       id="give-export-donation-currency_code"><?php _e( 'Currency Code', 'give' ); ?>
+									       id="give-export-donation-currency_code"><?php _e('Currency Code', 'give'); ?>
 								</label>
 							</li>
 
@@ -281,7 +281,7 @@ 
 								<label for="give-export-donation-currency_symbol">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[currency_symbol]"
-									       id="give-export-donation-currency_symbol"><?php _e( 'Currency Symbol', 'give' ); ?>
+									       id="give-export-donation-currency_symbol"><?php _e('Currency Symbol', 'give'); ?>
 								</label>
 							</li>
 
@@ -289,7 +289,7 @@ 
 								<label for="give-export-donation-status">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[donation_status]"
-									       id="give-export-donation-status"><?php _e( 'Donation Status', 'give' ); ?>
+									       id="give-export-donation-status"><?php _e('Donation Status', 'give'); ?>
 								</label>
 							</li>
 
@@ -297,7 +297,7 @@ 
 								<label for="give-export-donation-date">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[donation_date]"
-									       id="give-export-donation-date"><?php _e( 'Donation Date', 'give' ); ?>
+									       id="give-export-donation-date"><?php _e('Donation Date', 'give'); ?>
 								</label>
 							</li>
 
@@ -305,7 +305,7 @@ 
 								<label for="give-export-donation-time">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[donation_time]"
-									       id="give-export-donation-time"><?php _e( 'Donation Time', 'give' ); ?>
+									       id="give-export-donation-time"><?php _e('Donation Time', 'give'); ?>
 								</label>
 							</li>
 
@@ -313,7 +313,7 @@ 
 								<label for="give-export-payment-gateway">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[payment_gateway]"
-									       id="give-export-payment-gateway"><?php _e( 'Payment Gateway', 'give' ); ?>
+									       id="give-export-payment-gateway"><?php _e('Payment Gateway', 'give'); ?>
 								</label>
 							</li>
 
@@ -323,7 +323,7 @@ 
 							 *
 							 * @since 2.1
 							 */
-							do_action( 'give_export_donation_standard_payment_fields' );
+							do_action('give_export_donation_standard_payment_fields');
 							?>
 						</ul>
 					</li>
@@ -333,7 +333,7 @@ 
 
 							<li class="give-export-option-label give-export-option-Form-label">
 								<span>
-									<?php _e( 'Donation Form Fields', 'give' ); ?>
+									<?php _e('Donation Form Fields', 'give'); ?>
 								</span>
 							</li>
 
@@ -342,7 +342,7 @@ 
 								<label for="give-export-donation-form-id">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[form_id]"
-									       id="give-export-donation-form-id"><?php _e( 'Donation Form ID', 'give' ); ?>
+									       id="give-export-donation-form-id"><?php _e('Donation Form ID', 'give'); ?>
 								</label>
 							</li>
 
@@ -350,7 +350,7 @@ 
 								<label for="give-export-donation-form-title">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[form_title]"
-									       id="give-export-donation-form-title"><?php _e( 'Donation Form Title', 'give' ); ?>
+									       id="give-export-donation-form-title"><?php _e('Donation Form Title', 'give'); ?>
 								</label>
 							</li>
 
@@ -358,7 +358,7 @@ 
 								<label for="give-export-donation-form-level-id">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[form_level_id]"
-									       id="give-export-donation-form-level-id"><?php _e( 'Donation Form Level ID', 'give' ); ?>
+									       id="give-export-donation-form-level-id"><?php _e('Donation Form Level ID', 'give'); ?>
 								</label>
 							</li>
 
@@ -366,7 +366,7 @@ 
 								<label for="give-export-donation-form-level-title">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[form_level_title]"
-									       id="give-export-donation-form-level-title"><?php _e( 'Donation Form Level Title', 'give' ); ?>
+									       id="give-export-donation-form-level-title"><?php _e('Donation Form Level Title', 'give'); ?>
 								</label>
 							</li>
 
@@ -376,7 +376,7 @@ 
 							 *
 							 * @since 2.1
 							 */
-							do_action( 'give_export_donation_standard_form_fields' );
+							do_action('give_export_donation_standard_form_fields');
 							?>
 						</ul>
 					</li>
@@ -386,7 +386,7 @@ 
 
 							<li class="give-export-option-label give-export-option-donor-label">
 								<span>
-									<?php _e( 'Donor Fields', 'give' ); ?>
+									<?php _e('Donor Fields', 'give'); ?>
 								</span>
 							</li>
 
@@ -394,7 +394,7 @@ 
 								<label for="give-export-first-name">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[first_name]"
-									       id="give-export-first-name"><?php _e( 'Donor\'s First Name', 'give' ); ?>
+									       id="give-export-first-name"><?php _e('Donor\'s First Name', 'give'); ?>
 								</label>
 							</li>
 
@@ -402,7 +402,7 @@ 
 								<label for="give-export-last-name">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[last_name]"
-									       id="give-export-last-name"><?php _e( 'Donor\'s Last Name', 'give' ); ?>
+									       id="give-export-last-name"><?php _e('Donor\'s Last Name', 'give'); ?>
 								</label>
 							</li>
 
@@ -410,7 +410,7 @@ 
 								<label for="give-export-email">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[email]"
-									       id="give-export-email"><?php _e( 'Donor\'s Email', 'give' ); ?>
+									       id="give-export-email"><?php _e('Donor\'s Email', 'give'); ?>
 								</label>
 							</li>
 
@@ -418,7 +418,7 @@ 
 								<label for="give-export-company">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[company]"
-									       id="give-export-company"><?php _e( 'Company Name', 'give' ); ?>
+									       id="give-export-company"><?php _e('Company Name', 'give'); ?>
 								</label>
 							</li>
 
@@ -426,7 +426,7 @@ 
 								<label for="give-export-address">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[address]"
-									       id="give-export-address"><?php _e( 'Donor\'s Billing Address', 'give' ); ?>
+									       id="give-export-address"><?php _e('Donor\'s Billing Address', 'give'); ?>
 								</label>
 							</li>
 
@@ -434,7 +434,7 @@ 
 								<label for="give-export-userid">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[userid]"
-									       id="give-export-userid"><?php _e( 'User ID', 'give' ); ?>
+									       id="give-export-userid"><?php _e('User ID', 'give'); ?>
 								</label>
 							</li>
 
@@ -442,7 +442,7 @@ 
 								<label for="give-export-donorid">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[donorid]"
-									       id="give-export-donorid"><?php _e( 'Donor ID', 'give' ); ?>
+									       id="give-export-donorid"><?php _e('Donor ID', 'give'); ?>
 								</label>
 							</li>
 
@@ -450,7 +450,7 @@ 
 								<label for="give-export-donor-ip">
 									<input type="checkbox" checked
 									       name="give_give_donations_export_option[donor_ip]"
-									       id="give-export-donor-ip"><?php _e( 'Donor IP Address', 'give' ); ?>
+									       id="give-export-donor-ip"><?php _e('Donor IP Address', 'give'); ?>
 								</label>
 							</li>
 
@@ -460,7 +460,7 @@ 
 							 *
 							 * @since 2.1
 							 */
-							do_action( 'give_export_donation_standard_donor_fields' );
+							do_action('give_export_donation_standard_donor_fields');
 							?>
 						</ul>
 					</li>
@@ -471,7 +471,7 @@ 
 					 *
 					 * @since 2.1.4
 					 */
-					do_action( 'give_export_donation_add_custom_column' );
+					do_action('give_export_donation_add_custom_column');
 					?>
 				</ul>
 			</div>
@@ -480,7 +480,7 @@ 
 	<?php
 }
 
-add_action( 'give_export_donation_fields', 'give_export_donation_standard_fields', 10 );
+add_action('give_export_donation_fields', 'give_export_donation_standard_fields', 10);
 
 /**
  * Add Donation Custom fields in export donation page
@@ -492,19 +492,19 @@ 
 	<tr
 		class="give-hidden give-export-donations-hide give-export-donations-standard-fields">
 		<td scope="row" class="row-title">
-			<label><?php _e( 'Custom Field Columns:', 'give' ); ?></label>
+			<label><?php _e('Custom Field Columns:', 'give'); ?></label>
 		</td>
 		<td class="give-field-wrap">
 			<div class="give-clearfix">
 				<ul class="give-export-option-ul"></ul>
-				<p class="give-field-description"><?php _e( 'The following fields may have been created by custom code, or another plugin.', 'give' ); ?></p>
+				<p class="give-field-description"><?php _e('The following fields may have been created by custom code, or another plugin.', 'give'); ?></p>
 			</div>
 		</td>
 	</tr>
 	<?php
 }
 
-add_action( 'give_export_donation_fields', 'give_export_donation_custom_fields', 30 );
+add_action('give_export_donation_fields', 'give_export_donation_custom_fields', 30);
 
 
 /**
@@ -517,17 +517,17 @@ 
 
 	<tr class="give-hidden give-export-donations-hide give-export-donations-hidden-fields">
 		<td scope="row" class="row-title">
-			<label><?php _e( 'Hidden Custom Field Columns:', 'give' ); ?></label>
+			<label><?php _e('Hidden Custom Field Columns:', 'give'); ?></label>
 		</td>
 		<td class="give-field-wrap">
 			<div class="give-clearfix">
 				<ul class="give-export-option-ul"></ul>
-				<p class="give-field-description"><?php _e( 'The following hidden custom fields contain data created by Give Core, a Give Add-on, another plugin, etc.<br/>Hidden fields are generally used for programming logic, but you may contain data you would like to export.', 'give' ); ?></p>
+				<p class="give-field-description"><?php _e('The following hidden custom fields contain data created by Give Core, a Give Add-on, another plugin, etc.<br/>Hidden fields are generally used for programming logic, but you may contain data you would like to export.', 'give'); ?></p>
 			</div>
 		</td>
 	</tr>
 	<?php
 }
 
-add_action( 'give_export_donation_fields', 'give_export_donation_hidden_fields', 40 );
+add_action('give_export_donation_fields', 'give_export_donation_hidden_fields', 40);
 

includes/api/class-give-api-v2.php

Doc Comments +1 added lines, -1 removed lines

@@ -148,7 +148,7 @@
 	 * @param WP_REST_Request $request
 	 *
 	 * @access public
-	 * @return array|mixed|object
+	 * @return string|null
 	 */
 	public function get_donation_grid( $request ) {
 		$parameters = $request->get_params();

Spacing +23 added lines, -23 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -61,7 +61,7 @@ 
 	 * @return Give_API_V2
 	 */
 	public static function get_instance() {
-		if ( null === static::$instance ) {
+		if (null === static::$instance) {
 			self::$instance = new static();
 
 			self::$instance->init();
@@ -79,9 +79,9 @@ 
 	 */
 	private function init() {
 		// Setup hooks.
-		add_action( 'rest_api_init', array( $this, 'register_routes' ) );
-		add_action( 'wp_enqueue_scripts', array( $this, 'localize_script' ), 999 );
-		add_action( 'admin_enqueue_scripts', array( $this, 'localize_script' ), 999 );
+		add_action('rest_api_init', array($this, 'register_routes'));
+		add_action('wp_enqueue_scripts', array($this, 'localize_script'), 999);
+		add_action('admin_enqueue_scripts', array($this, 'localize_script'), 999);
 	}
 
 
@@ -93,15 +93,15 @@ 
 	 * @access private
 	 */
 	public function register_routes() {
-		register_rest_route( $this->rest_base, '/form/(?P<id>[\d]+)', array(
+		register_rest_route($this->rest_base, '/form/(?P<id>[\d]+)', array(
 			'methods'  => 'GET',
-			'callback' => array( $this, 'get_forms_data' ),
-		) );
+			'callback' => array($this, 'get_forms_data'),
+		));
 
-		register_rest_route( $this->rest_base, '/form-grid', array(
+		register_rest_route($this->rest_base, '/form-grid', array(
 			'methods'  => 'GET',
-			'callback' => array( $this, 'get_donation_grid' ),
-		) );
+			'callback' => array($this, 'get_donation_grid'),
+		));
 	}
 
 	/**
@@ -112,14 +112,14 @@ 
 	 */
 	public function localize_script() {
 		$data = array(
-			'root' => esc_url_raw( Give_API_V2::get_rest_api() ),
+			'root' => esc_url_raw(Give_API_V2::get_rest_api()),
 			'rest_base' => $this->rest_base
 		);
 
-		if ( is_admin() ) {
-			wp_localize_script( 'give-admin-scripts', 'giveApiSettings', $data );
+		if (is_admin()) {
+			wp_localize_script('give-admin-scripts', 'giveApiSettings', $data);
 		} else {
-			wp_localize_script( 'give', 'giveApiSettings', $data );
+			wp_localize_script('give', 'giveApiSettings', $data);
 		}
 	}
 
@@ -131,15 +131,15 @@ 
 	 * @access public
 	 * @return array|mixed|object
 	 */
-	public function get_forms_data( $request ) {
+	public function get_forms_data($request) {
 		$parameters = $request->get_params();
 
 		// Bailout
-		if ( ! isset( $parameters['id'] ) || empty( $parameters['id'] ) ) {
-			return array( 'error' => 'no_parameter_given' );
+		if ( ! isset($parameters['id']) || empty($parameters['id'])) {
+			return array('error' => 'no_parameter_given');
 		}
 
-		return give_form_shortcode( $parameters );
+		return give_form_shortcode($parameters);
 	}
 
 	/**
@@ -150,10 +150,10 @@ 
 	 * @access public
 	 * @return array|mixed|object
 	 */
-	public function get_donation_grid( $request ) {
+	public function get_donation_grid($request) {
 		$parameters = $request->get_params();
 
-		return give_form_grid_shortcode( $parameters );
+		return give_form_grid_shortcode($parameters);
 	}
 
 	/**
@@ -168,8 +168,8 @@ 
 	 *
 	 * @return string Full URL to the endpoint.
 	 */
-	public static function get_rest_api( $blog_id = null, $path = '/', $scheme = 'rest' ) {
-		return trailingslashit( get_rest_url( $blog_id, $path, $scheme ) . self::$instance->rest_base );
+	public static function get_rest_api($blog_id = null, $path = '/', $scheme = 'rest') {
+		return trailingslashit(get_rest_url($blog_id, $path, $scheme).self::$instance->rest_base);
 	}
 }
 

includes/import-functions.php

Doc Comments +5 added lines, -5 removed lines

@@ -555,10 +555,10 @@ 
  *
  * @since 2.1
  *
- * @param $file_dir
- * @param $start
- * @param $end
- * @param $delimiter
+ * @param false|string $file_dir
+ * @param integer $start
+ * @param integer $end
+ * @param string $delimiter
  *
  * @return array
  */
@@ -584,7 +584,7 @@ 
  *
  * @since 2.1
  *
- * @param $file_id
+ * @param integer $file_id
  *
  * @return false|string file content
  */

Spacing +316 added lines, -316 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -20,7 +20,7 @@ 
  * @since 1.8.13
  */
 function give_import_donation_report() {
-	return get_option( 'give_import_donation_report', array() );
+	return get_option('give_import_donation_report', array());
 }
 
 
@@ -29,8 +29,8 @@ 
  *
  * @since 1.8.13
  */
-function give_import_donation_report_update( $value = array() ) {
-	update_option( 'give_import_donation_report', $value );
+function give_import_donation_report_update($value = array()) {
+	update_option('give_import_donation_report', $value);
 }
 
 /**
@@ -39,7 +39,7 @@ 
  * @since 1.8.13
  */
 function give_import_donation_report_reset() {
-	update_option( 'give_import_donation_report', array() );
+	update_option('give_import_donation_report', array());
 }
 
 /**
@@ -51,9 +51,9 @@ 
  *
  * @return array|bool|Give_Donate_Form|int|null|WP_Post
  */
-function give_import_get_form_data_from_csv( $data, $import_setting = array() ) {
+function give_import_get_form_data_from_csv($data, $import_setting = array()) {
 	$new_form = false;
-	$dry_run  = isset( $import_setting['dry_run'] ) ? $import_setting['dry_run'] : false;
+	$dry_run  = isset($import_setting['dry_run']) ? $import_setting['dry_run'] : false;
 
 	// Get the import report
 	$report = give_import_donation_report();
@@ -61,25 +61,25 @@ 
 	$form = false;
 	$meta = array();
 
-	if ( ! empty( $data['form_id'] ) ) {
-		$form = new Give_Donate_Form( $data['form_id'] );
+	if ( ! empty($data['form_id'])) {
+		$form = new Give_Donate_Form($data['form_id']);
 		// Add support to older php version.
 		$form_id = $form->get_ID();
-		if ( empty( $form_id ) ) {
+		if (empty($form_id)) {
 			$form = false;
 		} else {
-			$report['duplicate_form'] = ( ! empty( $report['duplicate_form'] ) ? ( absint( $report['duplicate_form'] ) + 1 ) : 1 );
+			$report['duplicate_form'] = ( ! empty($report['duplicate_form']) ? (absint($report['duplicate_form']) + 1) : 1);
 		}
 	}
 
-	if ( false === $form && ! empty( $data['form_title'] ) ) {
-		$form = get_page_by_title( $data['form_title'], OBJECT, 'give_forms' );
+	if (false === $form && ! empty($data['form_title'])) {
+		$form = get_page_by_title($data['form_title'], OBJECT, 'give_forms');
 
-		if ( ! empty( $form->ID ) ) {
+		if ( ! empty($form->ID)) {
 
-			$report['duplicate_form'] = ( ! empty( $report['duplicate_form'] ) ? ( absint( $report['duplicate_form'] ) + 1 ) : 1 );
+			$report['duplicate_form'] = ( ! empty($report['duplicate_form']) ? (absint($report['duplicate_form']) + 1) : 1);
 
-			$form = new Give_Donate_Form( $form->ID );
+			$form = new Give_Donate_Form($form->ID);
 		} else {
 			$form = new Give_Donate_Form();
 			$args = array(
@@ -87,43 +87,43 @@ 
 				'post_status' => 'publish',
 			);
 
-			if ( empty( $dry_run ) ) {
-				$form = $form->create( $args );
+			if (empty($dry_run)) {
+				$form = $form->create($args);
 			}
 
-			$report['create_form'] = ( ! empty( $report['create_form'] ) ? ( absint( $report['create_form'] ) + 1 ) : 1 );
+			$report['create_form'] = ( ! empty($report['create_form']) ? (absint($report['create_form']) + 1) : 1);
 			$new_form              = true;
 
 		}
 
-		$form = get_page_by_title( $data['form_title'], OBJECT, 'give_forms' );
-		if ( ! empty( $form->ID ) ) {
-			$form = new Give_Donate_Form( $form->ID );
+		$form = get_page_by_title($data['form_title'], OBJECT, 'give_forms');
+		if ( ! empty($form->ID)) {
+			$form = new Give_Donate_Form($form->ID);
 		}
 	}
 
-	if ( ! empty( $form ) && $form->get_ID() && ! empty( $data['form_level'] ) && empty( $dry_run ) ) {
+	if ( ! empty($form) && $form->get_ID() && ! empty($data['form_level']) && empty($dry_run)) {
 
 		$price_option = 'set';
-		$form_level   = strtolower( preg_replace( '/\s+/', '', $data['form_level'] ) );
+		$form_level   = strtolower(preg_replace('/\s+/', '', $data['form_level']));
 
-		if ( 'custom' !== $form_level ) {
+		if ('custom' !== $form_level) {
 			$prices     = (array) $form->get_prices();
 			$price_text = array();
-			foreach ( $prices as $key => $price ) {
-				if ( isset( $price['_give_id']['level_id'] ) ) {
-					$price_text[ $price['_give_id']['level_id'] ] = ( ! empty( $price['_give_text'] ) ? strtolower( preg_replace( '/\s+/', '', $price['_give_text'] ) ) : '' );
+			foreach ($prices as $key => $price) {
+				if (isset($price['_give_id']['level_id'])) {
+					$price_text[$price['_give_id']['level_id']] = ( ! empty($price['_give_text']) ? strtolower(preg_replace('/\s+/', '', $price['_give_text'])) : '');
 				}
 			}
 
-			if ( ! in_array( $form_level, $price_text ) ) {
+			if ( ! in_array($form_level, $price_text)) {
 
 				// For generating unquiet level id.
 				$count     = 1;
-				$new_level = count( $prices ) + $count;
-				while ( array_key_exists( $new_level, $price_text ) ) {
-					$count ++;
-					$new_level = count( $prices ) + $count;
+				$new_level = count($prices) + $count;
+				while (array_key_exists($new_level, $price_text)) {
+					$count++;
+					$new_level = count($prices) + $count;
 				}
 
 				$multi_level_donations = array(
@@ -131,42 +131,42 @@ 
 						'_give_id'     => array(
 							'level_id' => $new_level,
 						),
-						'_give_amount' => give_sanitize_amount_for_db( $data['amount'] ),
+						'_give_amount' => give_sanitize_amount_for_db($data['amount']),
 						'_give_text'   => $data['form_level'],
 					),
 				);
 
-				$price_text[ $new_level ] = strtolower( preg_replace( '/\s+/', '', $data['form_level'] ) );
+				$price_text[$new_level] = strtolower(preg_replace('/\s+/', '', $data['form_level']));
 
-				if ( ! empty( $prices ) && is_array( $prices ) && ! empty( $prices[0] ) ) {
-					$prices = wp_parse_args( $multi_level_donations, $prices );
+				if ( ! empty($prices) && is_array($prices) && ! empty($prices[0])) {
+					$prices = wp_parse_args($multi_level_donations, $prices);
 
 					// Sort $prices by amount in ascending order.
-					$prices = wp_list_sort( $prices, '_give_amount', 'ASC' );
+					$prices = wp_list_sort($prices, '_give_amount', 'ASC');
 				} else {
 					$prices = $multi_level_donations;
 				}
 
 				// Unset _give_default key from $prices.
-				foreach ( $prices as $key => $price ) {
-					if ( isset( $prices[ $key ]['_give_default'] ) ) {
-						unset( $prices[ $key ]['_give_default'] );
+				foreach ($prices as $key => $price) {
+					if (isset($prices[$key]['_give_default'])) {
+						unset($prices[$key]['_give_default']);
 					}
 				}
 
 				// Set the first $price of the $prices as default.
 				$prices[0]['_give_default'] = 'default';
 			}
-			$form->price_id = array_search( $form_level, $price_text );
+			$form->price_id = array_search($form_level, $price_text);
 
-			$donation_levels_amounts = wp_list_pluck( $prices, '_give_amount' );
-			$min_amount              = min( $donation_levels_amounts );
-			$max_amount              = max( $donation_levels_amounts );
+			$donation_levels_amounts = wp_list_pluck($prices, '_give_amount');
+			$min_amount              = min($donation_levels_amounts);
+			$max_amount              = max($donation_levels_amounts);
 
 			$meta = array(
 				'_give_levels_minimum_amount' => $min_amount,
 				'_give_levels_maximum_amount' => $max_amount,
-				'_give_donation_levels'       => array_values( $prices ),
+				'_give_donation_levels'       => array_values($prices),
 			);
 
 			$price_option = 'multi';
@@ -175,13 +175,13 @@ 
 		}
 
 		$defaults = array(
-			'_give_set_price' => give_sanitize_amount_for_db( $data['amount'] ),
+			'_give_set_price' => give_sanitize_amount_for_db($data['amount']),
 		);
 
 		// If new form is created.
-		if ( ! empty( $new_form ) ) {
+		if ( ! empty($new_form)) {
 			$new_form = array(
-				'_give_custom_amount_text'          => ( ! empty( $data['form_custom_amount_text'] ) ? $data['form_custom_amount_text'] : 'custom' ),
+				'_give_custom_amount_text'          => ( ! empty($data['form_custom_amount_text']) ? $data['form_custom_amount_text'] : 'custom'),
 				'_give_logged_in_only'              => 'enabled',
 				'_give_custom_amount'               => 'enabled',
 				'_give_payment_import'              => true,
@@ -194,18 +194,18 @@ 
 				'_give_show_register_form'          => 'both',
 				'_give_price_option'                => $price_option,
 			);
-			$defaults = wp_parse_args( $defaults, $new_form );
+			$defaults = wp_parse_args($defaults, $new_form);
 		}
 
-		$meta = wp_parse_args( $meta, $defaults );
+		$meta = wp_parse_args($meta, $defaults);
 
-		foreach ( $meta as $key => $value ) {
-			give_update_meta( $form->get_ID(), $key, $value );
+		foreach ($meta as $key => $value) {
+			give_update_meta($form->get_ID(), $key, $value);
 		}
 	}
 
 	// update the report
-	give_import_donation_report_update( $report );
+	give_import_donation_report_update($report);
 
 	return $form;
 }
@@ -219,36 +219,36 @@ 
  *
  * @return bool|false|WP_User
  */
-function give_import_get_user_from_csv( $data, $import_setting = array() ) {
+function give_import_get_user_from_csv($data, $import_setting = array()) {
 	$report               = give_import_donation_report();
-	$dry_run              = isset( $import_setting['dry_run'] ) ? $import_setting['dry_run'] : false;
+	$dry_run              = isset($import_setting['dry_run']) ? $import_setting['dry_run'] : false;
 	$dry_run_donor_create = false;
 	$donor_data           = array();
 	$donor_id             = false;
 
 	// check if donor id is not empty
-	if ( ! empty( $data['donor_id'] ) ) {
-		$donor_data = new Give_Donor( (int) $data['donor_id'] );
-		if ( ! empty( $donor_data->id ) ) {
-			$report['duplicate_donor'] = ( ! empty( $report['duplicate_donor'] ) ? ( absint( $report['duplicate_donor'] ) + 1 ) : 1 );
+	if ( ! empty($data['donor_id'])) {
+		$donor_data = new Give_Donor((int) $data['donor_id']);
+		if ( ! empty($donor_data->id)) {
+			$report['duplicate_donor'] = ( ! empty($report['duplicate_donor']) ? (absint($report['duplicate_donor']) + 1) : 1);
 		}
 	}
 
-	if ( empty( $donor_data->id ) && ! empty( $data['user_id'] ) ) {
+	if (empty($donor_data->id) && ! empty($data['user_id'])) {
 		$user_id    = (int) $data['user_id'];
-		$donor_data = new Give_Donor( $user_id, true );
+		$donor_data = new Give_Donor($user_id, true);
 
 
-		if ( empty( $donor_data->id ) ) {
-			$donor_data = get_user_by( 'id', $user_id );
+		if (empty($donor_data->id)) {
+			$donor_data = get_user_by('id', $user_id);
 
 			// if no wp user is found then no donor is create with that user id
-			if ( ! empty( $donor_data->ID ) ) {
+			if ( ! empty($donor_data->ID)) {
 
-				if ( empty( $dry_run ) ) {
-					$first_name = ( ! empty( $data['first_name'] ) ? $data['first_name'] : $donor_data->user_nicename );
-					$last_name  = ( ! empty( $data['last_name'] ) ? $data['last_name'] : ( ( $lastname = get_user_meta( $donor_data->ID, 'last_name', true ) ) ? $lastname : '' ) );
-					$name       = $first_name . ' ' . $last_name;
+				if (empty($dry_run)) {
+					$first_name = ( ! empty($data['first_name']) ? $data['first_name'] : $donor_data->user_nicename);
+					$last_name  = ( ! empty($data['last_name']) ? $data['last_name'] : (($lastname = get_user_meta($donor_data->ID, 'last_name', true)) ? $lastname : ''));
+					$name       = $first_name.' '.$last_name;
 					$user_email = $donor_data->user_email;
 					$donor_args = array(
 						'name'    => $name,
@@ -257,51 +257,51 @@ 
 					);
 
 					$donor_data = new Give_Donor();
-					$donor_data->create( $donor_args );
+					$donor_data->create($donor_args);
 
 					// Adding notes that donor is being imported from CSV.
 					$current_user = wp_get_current_user();
-					$donor_data->add_note( wp_sprintf( __( 'This donor was imported by %s', 'give' ), $current_user->user_email ) );
+					$donor_data->add_note(wp_sprintf(__('This donor was imported by %s', 'give'), $current_user->user_email));
 
 					// Add is used to ensure duplicate emails are not added
-					if ( $user_email != $data['email'] && ! empty( $data['email'] ) ) {
-						$donor_data->add_meta( 'additional_email', $data['email'] );
+					if ($user_email != $data['email'] && ! empty($data['email'])) {
+						$donor_data->add_meta('additional_email', $data['email']);
 					}
 				} else {
 					$dry_run_donor_create = true;
-					$donor_data           = array( 'id' => 1 );
+					$donor_data           = array('id' => 1);
 				}
 
-				$report['create_donor'] = ( ! empty( $report['create_donor'] ) ? ( absint( $report['create_donor'] ) + 1 ) : 1 );
-			} elseif ( $dry_run ) {
+				$report['create_donor'] = ( ! empty($report['create_donor']) ? (absint($report['create_donor']) + 1) : 1);
+			} elseif ($dry_run) {
 				$donor_data = array();
 			}
 		} else {
 			// Add is used to ensure duplicate emails are not added
-			if ( $donor_data->email != $data['email'] && empty( $dry_run ) ) {
-				$donor_data->add_meta( 'additional_email', ( ! empty( $data['email'] ) ? $data['email'] : $donor_data->email ) );
+			if ($donor_data->email != $data['email'] && empty($dry_run)) {
+				$donor_data->add_meta('additional_email', ( ! empty($data['email']) ? $data['email'] : $donor_data->email));
 			}
-			$report['duplicate_donor'] = ( ! empty( $report['duplicate_donor'] ) ? ( absint( $report['duplicate_donor'] ) + 1 ) : 1 );
+			$report['duplicate_donor'] = ( ! empty($report['duplicate_donor']) ? (absint($report['duplicate_donor']) + 1) : 1);
 		}
 	}
 
-	if ( empty( $donor_data->id ) && ! empty( $data['email'] ) && empty( $dry_run_donor_create ) ) {
+	if (empty($donor_data->id) && ! empty($data['email']) && empty($dry_run_donor_create)) {
 
-		$donor_data = new Give_Donor( $data['email'] );
-		if ( empty( $donor_data->id ) ) {
-			$donor_data = get_user_by( 'email', $data['email'] );
+		$donor_data = new Give_Donor($data['email']);
+		if (empty($donor_data->id)) {
+			$donor_data = get_user_by('email', $data['email']);
 
-			if ( empty( $donor_data->ID ) && isset( $import_setting['create_user'] ) && 1 === absint( $import_setting['create_user'] ) ) {
-				$data['first_name'] = ( ! empty( $data['first_name'] ) ? $data['first_name'] : $data['email'] );
-				$data['last_name']  = ( ! empty( $data['last_name'] ) ? $data['last_name'] : '' );
-				$give_role          = (array) give_get_option( 'donor_default_user_role', get_option( 'default_role', ( ( $give_donor = wp_roles()->is_role( 'give_donor' ) ) && ! empty( $give_donor ) ? 'give_donor' : 'subscriber' ) ) );
+			if (empty($donor_data->ID) && isset($import_setting['create_user']) && 1 === absint($import_setting['create_user'])) {
+				$data['first_name'] = ( ! empty($data['first_name']) ? $data['first_name'] : $data['email']);
+				$data['last_name']  = ( ! empty($data['last_name']) ? $data['last_name'] : '');
+				$give_role          = (array) give_get_option('donor_default_user_role', get_option('default_role', (($give_donor = wp_roles()->is_role('give_donor')) && ! empty($give_donor) ? 'give_donor' : 'subscriber')));
 				$donor_args         = array(
 					'user_login'      => $data['email'],
 					'user_email'      => $data['email'],
-					'user_registered' => date( 'Y-m-d H:i:s' ),
+					'user_registered' => date('Y-m-d H:i:s'),
 					'user_first'      => $data['first_name'],
 					'user_last'       => $data['last_name'],
-					'user_pass'       => wp_generate_password( 8, true ),
+					'user_pass'       => wp_generate_password(8, true),
 					'role'            => $give_role,
 				);
 
@@ -310,63 +310,63 @@ 
 				 *
 				 * @since 1.8.13
 				 */
-				$donor_args = (array) apply_filters( 'give_import_insert_user_args', $donor_args, $data, $import_setting );
+				$donor_args = (array) apply_filters('give_import_insert_user_args', $donor_args, $data, $import_setting);
 
-				if ( empty( $dry_run ) ) {
+				if (empty($dry_run)) {
 
 					// This action was added to remove the login when using the give register function.
-					add_filter( 'give_log_user_in_on_register', 'give_log_user_in_on_register_callback', 11 );
-					$donor_id = give_register_and_login_new_user( $donor_args );
-					remove_filter( 'give_log_user_in_on_register', 'give_log_user_in_on_register_callback', 11 );
+					add_filter('give_log_user_in_on_register', 'give_log_user_in_on_register_callback', 11);
+					$donor_id = give_register_and_login_new_user($donor_args);
+					remove_filter('give_log_user_in_on_register', 'give_log_user_in_on_register_callback', 11);
 
-					$donor_data = new Give_Donor( $donor_id, true );
-					$donor_data->update_meta( '_give_payment_import', true );
+					$donor_data = new Give_Donor($donor_id, true);
+					$donor_data->update_meta('_give_payment_import', true);
 
 				} else {
 					$dry_run_donor_create   = true;
-					$report['create_donor'] = ( ! empty( $report['create_donor'] ) ? ( absint( $report['create_donor'] ) + 1 ) : 1 );
+					$report['create_donor'] = ( ! empty($report['create_donor']) ? (absint($report['create_donor']) + 1) : 1);
 				}
 			} else {
-				$donor_id = ( ! empty( $donor_data->ID ) ? $donor_data->ID : false );
+				$donor_id = ( ! empty($donor_data->ID) ? $donor_data->ID : false);
 			}
 
-			if ( empty( $dry_run_donor_create ) && ( ! empty( $donor_id ) || ( isset( $import_setting['create_user'] ) && 0 === absint( $import_setting['create_user'] ) ) ) ) {
-				$donor_data = new Give_Donor( $donor_id, true );
+			if (empty($dry_run_donor_create) && ( ! empty($donor_id) || (isset($import_setting['create_user']) && 0 === absint($import_setting['create_user'])))) {
+				$donor_data = new Give_Donor($donor_id, true);
 
-				if ( empty( $donor_data->id ) ) {
+				if (empty($donor_data->id)) {
 
-					if ( ! empty( $data['form_id'] ) ) {
-						$form = new Give_Donate_Form( $data['form_id'] );
+					if ( ! empty($data['form_id'])) {
+						$form = new Give_Donate_Form($data['form_id']);
 					}
 
-					if ( empty( $dry_run ) ) {
-						$payment_title = ( isset( $data['form_title'] ) ? $data['form_title'] : ( isset( $form ) ? $form->get_name() : __( 'New Form', 'give' ) ) );
+					if (empty($dry_run)) {
+						$payment_title = (isset($data['form_title']) ? $data['form_title'] : (isset($form) ? $form->get_name() : __('New Form', 'give')));
 						$donor_args    = array(
-							'name'  => ! is_email( $payment_title ) ? $data['first_name'] . ' ' . $data['last_name'] : '',
+							'name'  => ! is_email($payment_title) ? $data['first_name'].' '.$data['last_name'] : '',
 							'email' => $data['email'],
 						);
-						if ( ! empty( $donor_id ) ) {
+						if ( ! empty($donor_id)) {
 							$donor_args['user_id'] = $donor_id;
 						}
-						$donor_data->create( $donor_args );
+						$donor_data->create($donor_args);
 
 						// Adding notes that donor is being imported from CSV.
 						$current_user = wp_get_current_user();
-						$donor_data->add_note( wp_sprintf( __( 'This donor was imported by %s', 'give' ), $current_user->user_email ) );
+						$donor_data->add_note(wp_sprintf(__('This donor was imported by %s', 'give'), $current_user->user_email));
 					} else {
 						$dry_run_donor_create = true;
 					}
-					$report['create_donor'] = ( ! empty( $report['create_donor'] ) ? ( absint( $report['create_donor'] ) + 1 ) : 1 );
+					$report['create_donor'] = ( ! empty($report['create_donor']) ? (absint($report['create_donor']) + 1) : 1);
 				} else {
-					$report['duplicate_donor'] = ( ! empty( $report['duplicate_donor'] ) ? ( absint( $report['duplicate_donor'] ) + 1 ) : 1 );
+					$report['duplicate_donor'] = ( ! empty($report['duplicate_donor']) ? (absint($report['duplicate_donor']) + 1) : 1);
 				}
 			}
 		} else {
-			$report['duplicate_donor'] = ( ! empty( $report['duplicate_donor'] ) ? ( absint( $report['duplicate_donor'] ) + 1 ) : 1 );
+			$report['duplicate_donor'] = ( ! empty($report['duplicate_donor']) ? (absint($report['duplicate_donor']) + 1) : 1);
 		}
 	}
 	// update the report
-	give_import_donation_report_update( $report );
+	give_import_donation_report_update($report);
 
 	return $donor_data;
 }
@@ -384,9 +384,9 @@ 
 	 *
 	 * @return array
 	 */
-	return (array) apply_filters( 'give_import_default_options', array(
-		'' => __( 'Do not import', 'give' ),
-	) );
+	return (array) apply_filters('give_import_default_options', array(
+		'' => __('Do not import', 'give'),
+	));
 }
 
 /**
@@ -402,74 +402,74 @@ 
 	 *
 	 * @return array
 	 */
-	return (array) apply_filters( 'give_import_donations_options', array(
-		'id'          => __( 'Donation ID', 'give' ),
+	return (array) apply_filters('give_import_donations_options', array(
+		'id'          => __('Donation ID', 'give'),
 		'amount'      => array(
-			__( 'Donation Amount', 'give' ),
-			__( 'Amount', 'give' )
+			__('Donation Amount', 'give'),
+			__('Amount', 'give')
 		),
 		'currency'      => array(
-			__( 'Donation Currencies', 'give' ),
-			__( 'Currencies', 'give' )
+			__('Donation Currencies', 'give'),
+			__('Currencies', 'give')
 		),
 		'post_date'   => array(
-			__( 'Donation Date', 'give' ),
-			__( 'Date', 'give' ),
+			__('Donation Date', 'give'),
+			__('Date', 'give'),
 		),
 		'first_name'  => array(
-			__( 'Donor First Name', 'give' ),
-			__( 'First Name', 'give' ),
-			__( 'Name', 'give' ),
+			__('Donor First Name', 'give'),
+			__('First Name', 'give'),
+			__('Name', 'give'),
 		),
 		'last_name'   => array(
-			__( 'Donor Last Name', 'give' ),
-			__( 'Last Name', 'give' ),
+			__('Donor Last Name', 'give'),
+			__('Last Name', 'give'),
 		),
 		'company_name'   => array(
-			__( 'Company Name', 'give' ),
-			__( 'Donor Company Name', 'give' ),
-			__( 'Donor Company', 'give' ),
-			__( 'Company', 'give' ),
+			__('Company Name', 'give'),
+			__('Donor Company Name', 'give'),
+			__('Donor Company', 'give'),
+			__('Company', 'give'),
 		),
 		'line1'       => array(
-			__( 'Address 1', 'give' ),
-			__( 'Address', 'give' ),
+			__('Address 1', 'give'),
+			__('Address', 'give'),
 		),
-		'line2'       => __( 'Address 2', 'give' ),
-		'city'        => __( 'City', 'give' ),
+		'line2'       => __('Address 2', 'give'),
+		'city'        => __('City', 'give'),
 		'state'       => array(
-			__( 'State', 'give' ),
-			__( 'Province', 'give' ),
-			__( 'County', 'give' ),
-			__( 'Region', 'give' ),
+			__('State', 'give'),
+			__('Province', 'give'),
+			__('County', 'give'),
+			__('Region', 'give'),
 		),
-		'country'     => __( 'Country', 'give' ),
+		'country'     => __('Country', 'give'),
 		'zip'         => array(
-			__( 'Zip Code', 'give' ),
-			__( 'Zip', 'give' ),
-			__( 'zipcode', 'give' ),
-			__( 'Postal Code', 'give' ),
-			__( 'Postal', 'give' ),
+			__('Zip Code', 'give'),
+			__('Zip', 'give'),
+			__('zipcode', 'give'),
+			__('Postal Code', 'give'),
+			__('Postal', 'give'),
 		),
 		'email'       => array(
-			__( 'Donor Email', 'give' ),
-			__( 'Email', 'give' )
+			__('Donor Email', 'give'),
+			__('Email', 'give')
 		),
 		'post_status' => array(
-			__( 'Donation Status', 'give' ),
-			__( 'Status', 'give' ),
+			__('Donation Status', 'give'),
+			__('Status', 'give'),
 		),
 		'gateway'     => array(
-			__( 'Payment Method', 'give' ),
-			__( 'Method', 'give' ),
+			__('Payment Method', 'give'),
+			__('Method', 'give'),
 		),
-		'notes'       => __( 'Notes', 'give' ),
+		'notes'       => __('Notes', 'give'),
 		'mode'        => array(
-			__( 'Test Mode', 'give' ),
-			__( 'Mode', 'give' ),
+			__('Test Mode', 'give'),
+			__('Mode', 'give'),
 		),
-		'post_meta'   => __( 'Import as Meta', 'give' ),
-	) );
+		'post_meta'   => __('Import as Meta', 'give'),
+	));
 }
 
 /**
@@ -485,10 +485,10 @@ 
 	 *
 	 * @return array
 	 */
-	return (array) apply_filters( 'give_import_donor_options', array(
-		'donor_id' => __( 'Donor ID', 'give' ),
-		'user_id'  => __( 'User ID', 'give' ),
-	) );
+	return (array) apply_filters('give_import_donor_options', array(
+		'donor_id' => __('Donor ID', 'give'),
+		'user_id'  => __('User ID', 'give'),
+	));
 }
 
 /**
@@ -504,23 +504,23 @@ 
 	 *
 	 * @return array
 	 */
-	return (array) apply_filters( 'give_import_donation_form_options', array(
+	return (array) apply_filters('give_import_donation_form_options', array(
 		'form_title'              => array(
-			__( 'Donation Form Title', 'give' ),
-			__( 'Donation Form', 'give' ),
-			__( 'Form Name', 'give' ),
-			__( 'Title', 'give' ),
+			__('Donation Form Title', 'give'),
+			__('Donation Form', 'give'),
+			__('Form Name', 'give'),
+			__('Title', 'give'),
 		),
 		'form_id'                 => array(
-			__( 'Donation Form ID', 'give' ),
-			__( 'Form ID', 'give' )
+			__('Donation Form ID', 'give'),
+			__('Form ID', 'give')
 		),
 		'form_level'              => array(
-			__( 'Donation Level', 'give' ),
-			__( 'Level', 'give' ),
+			__('Donation Level', 'give'),
+			__('Level', 'give'),
 		),
-		'form_custom_amount_text' => __( 'Custom Amount Text', 'give' ),
-	) );
+		'form_custom_amount_text' => __('Custom Amount Text', 'give'),
+	));
 }
 
 /**
@@ -533,7 +533,7 @@ 
  *
  * @return array
  */
-function give_get_donation_data_from_csv( $file_id, $start, $end, $delimiter = 'csv' ) {
+function give_get_donation_data_from_csv($file_id, $start, $end, $delimiter = 'csv') {
 	/**
 	 * Filter to modify delimiter of Import
 	 *
@@ -543,11 +543,11 @@ 
 	 *
 	 * @return string $delimiter
 	 */
-	$delimiter = (string) apply_filters( 'give_import_delimiter_set', $delimiter );
+	$delimiter = (string) apply_filters('give_import_delimiter_set', $delimiter);
 
-	$file_dir = give_get_file_data_by_file_id( $file_id );
+	$file_dir = give_get_file_data_by_file_id($file_id);
 
-	return give_get_raw_data_from_file( $file_dir, $start, $end, $delimiter );
+	return give_get_raw_data_from_file($file_dir, $start, $end, $delimiter);
 }
 
 /**
@@ -562,18 +562,18 @@ 
  *
  * @return array
  */
-function give_get_raw_data_from_file( $file_dir, $start, $end, $delimiter ) {
+function give_get_raw_data_from_file($file_dir, $start, $end, $delimiter) {
 	$raw_data = array();
 
 	$count    = 0;
-	if ( false !== ( $handle = fopen( $file_dir, 'r' ) ) ) {
-		while ( false !== ( $row = fgetcsv( $handle, 0, $delimiter ) ) ) {
-			if ( $count >= $start && $count <= $end ) {
+	if (false !== ($handle = fopen($file_dir, 'r'))) {
+		while (false !== ($row = fgetcsv($handle, 0, $delimiter))) {
+			if ($count >= $start && $count <= $end) {
 				$raw_data[] = $row;
 			}
-			$count ++;
+			$count++;
 		}
-		fclose( $handle );
+		fclose($handle);
 	}
 
 	return $raw_data;
@@ -588,8 +588,8 @@ 
  *
  * @return false|string file content
  */
-function give_get_file_data_by_file_id( $file_id ) {
-	return get_attached_file( $file_id );
+function give_get_file_data_by_file_id($file_id) {
+	return get_attached_file($file_id);
 }
 
 
@@ -602,7 +602,7 @@ 
  *
  * @return bool
  */
-function give_log_user_in_on_register_callback( $value ) {
+function give_log_user_in_on_register_callback($value) {
 	return false;
 }
 
@@ -618,87 +618,87 @@ 
  *
  * @return bool
  */
-function give_save_import_donation_to_db( $raw_key, $row_data, $main_key = array(), $import_setting = array() ) {
-	$data                          = array_combine( $raw_key, $row_data );
+function give_save_import_donation_to_db($raw_key, $row_data, $main_key = array(), $import_setting = array()) {
+	$data                          = array_combine($raw_key, $row_data);
 	$price_id                      = false;
 	$donor_id                      = 0;
 	$donor_data                    = array();
 	$form                          = array();
-	$import_setting['create_user'] = isset( $import_setting['create_user'] ) ? $import_setting['create_user'] : 1;
-	$dry_run                       = isset( $import_setting['dry_run'] ) ? $import_setting['dry_run'] : false;
+	$import_setting['create_user'] = isset($import_setting['create_user']) ? $import_setting['create_user'] : 1;
+	$dry_run                       = isset($import_setting['dry_run']) ? $import_setting['dry_run'] : false;
 	$_dry_run_is_duplicate         = false;
 	$dry_run_duplicate_form        = false;
 	$dry_run_duplicate_donor       = false;
-	$donation_key                  = empty( $import_setting['donation_key'] ) ? 1 : (int) $import_setting['donation_key'];
+	$donation_key                  = empty($import_setting['donation_key']) ? 1 : (int) $import_setting['donation_key'];
 	$payment_id = false;
 
-	$data = (array) apply_filters( 'give_save_import_donation_to_db', $data );
+	$data = (array) apply_filters('give_save_import_donation_to_db', $data);
 
-	$data['amount'] = give_maybe_sanitize_amount( $data['amount'] );
+	$data['amount'] = give_maybe_sanitize_amount($data['amount']);
 	$diff           = array();
 
-	if ( ! empty( $dry_run ) && 1 !== $donation_key ) {
-		$csv_raw_data = empty( $import_setting['csv_raw_data'] ) ? array() : $import_setting['csv_raw_data'];
-		$donors_list  = empty( $import_setting['donors_list'] ) ? array() : $import_setting['donors_list'];
+	if ( ! empty($dry_run) && 1 !== $donation_key) {
+		$csv_raw_data = empty($import_setting['csv_raw_data']) ? array() : $import_setting['csv_raw_data'];
+		$donors_list  = empty($import_setting['donors_list']) ? array() : $import_setting['donors_list'];
 		$key          = $donation_key - 1;
-		for ( $i = 0; $i < $key; $i ++ ) {
-			$csv_data           = array_combine( $raw_key, $csv_raw_data[ $i ] );
-			$csv_data['amount'] = give_maybe_sanitize_amount( $csv_data['amount'] );
+		for ($i = 0; $i < $key; $i++) {
+			$csv_data           = array_combine($raw_key, $csv_raw_data[$i]);
+			$csv_data['amount'] = give_maybe_sanitize_amount($csv_data['amount']);
 			// check for duplicate donations
-			$diff = array_diff( $csv_data, $data );
-			if ( empty( $diff ) ) {
+			$diff = array_diff($csv_data, $data);
+			if (empty($diff)) {
 				$_dry_run_is_duplicate   = true;
 				$dry_run_duplicate_form  = true;
 				$dry_run_duplicate_donor = true;
 			} else {
 				// check for duplicate donation form with form id
-				if ( ! empty( $csv_data['form_id'] ) && ! empty( $data['form_id'] ) && $csv_data['form_id'] === $data['form_id'] ) {
-					$form    = new Give_Donate_Form( $data['form_id'] );
+				if ( ! empty($csv_data['form_id']) && ! empty($data['form_id']) && $csv_data['form_id'] === $data['form_id']) {
+					$form    = new Give_Donate_Form($data['form_id']);
 					$form_id = $form->get_ID();
-					if ( ! empty( $form_id ) ) {
+					if ( ! empty($form_id)) {
 						$dry_run_duplicate_form = true;
 					}
 				}
 				// check for duplicate donation form with form title
-				if ( empty( $dry_run_duplicate_form ) && ! empty( $csv_data['form_title'] ) && ! empty( $data['form_title'] ) && $csv_data['form_title'] === $data['form_title'] ) {
+				if (empty($dry_run_duplicate_form) && ! empty($csv_data['form_title']) && ! empty($data['form_title']) && $csv_data['form_title'] === $data['form_title']) {
 					$dry_run_duplicate_form = true;
 				}
 
 
 				// check for duplicate donor by donor id
-				if ( ! empty( $csv_data['donor_id'] ) && ! empty( $data['donor_id'] ) && $csv_data['donor_id'] === $data['donor_id'] ) {
-					$donor = array_search( (int) $data['donor_id'], array_column( 'id', $donors_list ) );
-					if ( ! empty( $donor ) ) {
+				if ( ! empty($csv_data['donor_id']) && ! empty($data['donor_id']) && $csv_data['donor_id'] === $data['donor_id']) {
+					$donor = array_search((int) $data['donor_id'], array_column('id', $donors_list));
+					if ( ! empty($donor)) {
 						$dry_run_duplicate_donor = true;
 					}
 				}
 
 				// check for duplicate donor by user id
-				if ( empty( $dry_run_duplicate_donor ) && ! empty( $csv_data['user_id'] ) && ! empty( $data['user_id'] ) && $csv_data['user_id'] === $data['user_id'] ) {
-					$donor = array_search( (int) $data['user_id'], array_column( 'user_id', $donors_list ) );
-					if ( ! empty( $donor ) ) {
+				if (empty($dry_run_duplicate_donor) && ! empty($csv_data['user_id']) && ! empty($data['user_id']) && $csv_data['user_id'] === $data['user_id']) {
+					$donor = array_search((int) $data['user_id'], array_column('user_id', $donors_list));
+					if ( ! empty($donor)) {
 						$dry_run_duplicate_donor = true;
 					} else {
-						$donor = get_user_by( 'id', $csv_data['user_id'] );
-						if ( ! empty( $donor->ID ) ) {
+						$donor = get_user_by('id', $csv_data['user_id']);
+						if ( ! empty($donor->ID)) {
 							$dry_run_duplicate_donor = true;
 						}
 					}
 				}
 
 				// check for duplicate donor by donor id
-				if ( empty( $dry_run_duplicate_donor ) && ! empty( $csv_data['email'] ) && ! empty( $data['email'] ) && $csv_data['email'] === $data['email'] ) {
+				if (empty($dry_run_duplicate_donor) && ! empty($csv_data['email']) && ! empty($data['email']) && $csv_data['email'] === $data['email']) {
 					$dry_run_duplicate_donor = true;
 				}
 			}
 		}
 	}
 
-	if ( empty( $dry_run_duplicate_donor ) ) {
+	if (empty($dry_run_duplicate_donor)) {
 		// Here come the login function.
-		$donor_data = give_import_get_user_from_csv( $data, $import_setting );
-		if ( empty( $dry_run ) ) {
-			if ( ! empty( $donor_data->id ) ) {
+		$donor_data = give_import_get_user_from_csv($data, $import_setting);
+		if (empty($dry_run)) {
+			if ( ! empty($donor_data->id)) {
 				$donor_id = $donor_data->id;
 			} else {
 				return $payment_id;
@@ -707,40 +707,40 @@ 
 	} else {
 		// Get the report
 		$report                    = give_import_donation_report();
-		$report['duplicate_donor'] = ( ! empty( $report['duplicate_donor'] ) ? ( absint( $report['duplicate_donor'] ) + 1 ) : 1 );
+		$report['duplicate_donor'] = ( ! empty($report['duplicate_donor']) ? (absint($report['duplicate_donor']) + 1) : 1);
 		// update the report
-		give_import_donation_report_update( $report );
+		give_import_donation_report_update($report);
 	}
 
-	if ( empty( $dry_run_duplicate_form ) ) {
+	if (empty($dry_run_duplicate_form)) {
 		// get form data or register a form data.
-		$form = give_import_get_form_data_from_csv( $data, $import_setting );
-		if ( false == $form && empty( $dry_run ) ) {
+		$form = give_import_get_form_data_from_csv($data, $import_setting);
+		if (false == $form && empty($dry_run)) {
 			return $payment_id;
 		} else {
-			$price_id = ( ! empty( $form->price_id ) ) ? $form->price_id : false;
+			$price_id = ( ! empty($form->price_id)) ? $form->price_id : false;
 		}
 	} else {
 		// Get the report
 		$report                   = give_import_donation_report();
-		$report['duplicate_form'] = ( ! empty( $report['duplicate_form'] ) ? ( absint( $report['duplicate_form'] ) + 1 ) : 1 );
+		$report['duplicate_form'] = ( ! empty($report['duplicate_form']) ? (absint($report['duplicate_form']) + 1) : 1);
 		// update the report
-		give_import_donation_report_update( $report );
+		give_import_donation_report_update($report);
 	}
 
 	// Get the report
 	$report = give_import_donation_report();
 
 
-	$status  = give_import_donation_get_status( $data );
-	$country = ( ! empty( $data['country'] ) ? ( ( $country_code = array_search( $data['country'], give_get_country_list() ) ) ? $country_code : $data['country'] ) : '' );
-	$state   = ( ! empty( $data['state'] ) ? ( ( $state_code = array_search( $data['state'], give_get_states( $country ) ) ) ? $state_code : $data['state'] ) : '' );
+	$status  = give_import_donation_get_status($data);
+	$country = ( ! empty($data['country']) ? (($country_code = array_search($data['country'], give_get_country_list())) ? $country_code : $data['country']) : '');
+	$state   = ( ! empty($data['state']) ? (($state_code = array_search($data['state'], give_get_states($country))) ? $state_code : $data['state']) : '');
 
 	$address = array(
-		'line1'   => ( ! empty( $data['line1'] ) ? give_clean( $data['line1'] ) : '' ),
-		'line2'   => ( ! empty( $data['line2'] ) ? give_clean( $data['line2'] ) : '' ),
-		'city'    => ( ! empty( $data['city'] ) ? give_clean( $data['city'] ) : '' ),
-		'zip'     => ( ! empty( $data['zip'] ) ? give_clean( $data['zip'] ) : '' ),
+		'line1'   => ( ! empty($data['line1']) ? give_clean($data['line1']) : ''),
+		'line2'   => ( ! empty($data['line2']) ? give_clean($data['line2']) : ''),
+		'city'    => ( ! empty($data['city']) ? give_clean($data['city']) : ''),
+		'zip'     => ( ! empty($data['zip']) ? give_clean($data['zip']) : ''),
 		'state'   => $state,
 		'country' => $country,
 	);
@@ -750,22 +750,22 @@ 
 		'donor_id'        => $donor_id,
 		'price'           => $data['amount'],
 		'status'          => $status,
-		'currency'        => ! empty( $data['currency'] ) && array_key_exists( $data['currency'], give_get_currencies_list() ) ? $data['currency'] : give_get_currency(),
+		'currency'        => ! empty($data['currency']) && array_key_exists($data['currency'], give_get_currencies_list()) ? $data['currency'] : give_get_currency(),
 		'user_info'       => array(
 			'id'         => $donor_id,
-			'email'      => ( ! empty( $data['email'] ) ? $data['email'] : ( isset( $donor_data->email ) ? $donor_data->email : false ) ),
-			'first_name' => ( ! empty( $data['first_name'] ) ? $data['first_name'] : ( ! empty( $donor_id ) && ( $first_name = get_user_meta( $donor_id, 'first_name', true ) ) ? $first_name : $donor_data->name ) ),
-			'last_name'  => ( ! empty( $data['last_name'] ) ? $data['last_name'] : ( ! empty( $donor_id ) && ( $last_name = get_user_meta( $donor_id, 'last_name', true ) ) ? $last_name : $donor_data->name ) ),
+			'email'      => ( ! empty($data['email']) ? $data['email'] : (isset($donor_data->email) ? $donor_data->email : false)),
+			'first_name' => ( ! empty($data['first_name']) ? $data['first_name'] : ( ! empty($donor_id) && ($first_name = get_user_meta($donor_id, 'first_name', true)) ? $first_name : $donor_data->name)),
+			'last_name'  => ( ! empty($data['last_name']) ? $data['last_name'] : ( ! empty($donor_id) && ($last_name = get_user_meta($donor_id, 'last_name', true)) ? $last_name : $donor_data->name)),
 			'address'    => $address,
 		),
-		'gateway'         => ( ! empty( $data['gateway'] ) && 'offline' != strtolower( $data['gateway'] ) ? strtolower( $data['gateway'] ) : 'manual' ),
-		'give_form_title' => ( ! empty( $data['form_title'] ) ? $data['form_title'] : ( method_exists( $form, 'get_name' ) ? $form->get_name() : '' ) ),
-		'give_form_id'    => method_exists( $form, 'get_ID' ) ? $form->get_ID() : '',
+		'gateway'         => ( ! empty($data['gateway']) && 'offline' != strtolower($data['gateway']) ? strtolower($data['gateway']) : 'manual'),
+		'give_form_title' => ( ! empty($data['form_title']) ? $data['form_title'] : (method_exists($form, 'get_name') ? $form->get_name() : '')),
+		'give_form_id'    => method_exists($form, 'get_ID') ? $form->get_ID() : '',
 		'give_price_id'   => $price_id,
-		'purchase_key'    => strtolower( md5( uniqid() ) ),
+		'purchase_key'    => strtolower(md5(uniqid())),
 		'user_email'      => $data['email'],
-		'post_date'       => ( ! empty( $data['post_date'] ) ? mysql2date( 'Y-m-d H:i:s', $data['post_date'] ) : current_time( 'mysql' ) ),
-		'mode'            => ( ! empty( $data['mode'] ) ? ( 'true' == (string) $data['mode'] || 'TRUE' == (string) $data['mode'] ? 'test' : 'live' ) : ( isset( $import_setting['mode'] ) ? ( true == (bool) $import_setting['mode'] ? 'test' : 'live' ) : ( give_is_test_mode() ? 'test' : 'live' ) ) ),
+		'post_date'       => ( ! empty($data['post_date']) ? mysql2date('Y-m-d H:i:s', $data['post_date']) : current_time('mysql')),
+		'mode'            => ( ! empty($data['mode']) ? ('true' == (string) $data['mode'] || 'TRUE' == (string) $data['mode'] ? 'test' : 'live') : (isset($import_setting['mode']) ? (true == (bool) $import_setting['mode'] ? 'test' : 'live') : (give_is_test_mode() ? 'test' : 'live'))),
 	);
 
 	/**
@@ -780,74 +780,74 @@ 
 	 *
 	 * @return array $payment_data payment data
 	 */
-	$payment_data = apply_filters( 'give_import_before_import_payment', $payment_data, $data, $donor_data, $form );
+	$payment_data = apply_filters('give_import_before_import_payment', $payment_data, $data, $donor_data, $form);
 
 	// Get the report
 	$report = give_import_donation_report();
 
 	// Check for duplicate code.
-	$donation_duplicate = give_check_import_donation_duplicate( $payment_data, $data, $form, $donor_data );
-	if ( false !== $donation_duplicate || ! empty( $_dry_run_is_duplicate ) ) {
-		$report['donation_details'][ $import_setting['donation_key'] ]['duplicate'] = $donation_duplicate;
-		$report['duplicate_donation']                                               = ( ! empty( $report['duplicate_donation'] ) ? ( absint( $report['duplicate_donation'] ) + 1 ) : 1 );
+	$donation_duplicate = give_check_import_donation_duplicate($payment_data, $data, $form, $donor_data);
+	if (false !== $donation_duplicate || ! empty($_dry_run_is_duplicate)) {
+		$report['donation_details'][$import_setting['donation_key']]['duplicate'] = $donation_duplicate;
+		$report['duplicate_donation']                                               = ( ! empty($report['duplicate_donation']) ? (absint($report['duplicate_donation']) + 1) : 1);
 	} else {
 
-		if ( empty( $dry_run ) ) {
-			add_action( 'give_update_payment_status', 'give_donation_import_insert_default_payment_note', 1, 1 );
-			add_filter( 'give_insert_payment_args', 'give_donation_import_give_insert_payment_args', 11, 2 );
-			add_filter( 'give_update_donor_information', 'give_donation_import_update_donor_information', 11, 3 );
-			add_action( 'give_insert_payment', 'give_import_donation_insert_payment', 11, 2 );
+		if (empty($dry_run)) {
+			add_action('give_update_payment_status', 'give_donation_import_insert_default_payment_note', 1, 1);
+			add_filter('give_insert_payment_args', 'give_donation_import_give_insert_payment_args', 11, 2);
+			add_filter('give_update_donor_information', 'give_donation_import_update_donor_information', 11, 3);
+			add_action('give_insert_payment', 'give_import_donation_insert_payment', 11, 2);
 
 			// if it status is other then pending then first change the donation status to pending and after adding the payment meta update the donation status.
-			if ( 'pending' !== $status ) {
-				unset( $payment_data['status'] );
+			if ('pending' !== $status) {
+				unset($payment_data['status']);
 			}
 
-			$payment_id = give_insert_payment( $payment_data );
-			remove_action( 'give_update_payment_status', 'give_donation_import_insert_default_payment_note', 1 );
-			remove_filter( 'give_insert_payment_args', 'give_donation_import_give_insert_payment_args', 11 );
-			remove_filter( 'give_update_donor_information', 'give_donation_import_update_donor_information', 11 );
-			remove_action( 'give_insert_payment', 'give_import_donation_insert_payment', 11 );
+			$payment_id = give_insert_payment($payment_data);
+			remove_action('give_update_payment_status', 'give_donation_import_insert_default_payment_note', 1);
+			remove_filter('give_insert_payment_args', 'give_donation_import_give_insert_payment_args', 11);
+			remove_filter('give_update_donor_information', 'give_donation_import_update_donor_information', 11);
+			remove_action('give_insert_payment', 'give_import_donation_insert_payment', 11);
 
-			if ( $payment_id ) {
+			if ($payment_id) {
 
-				$payment = new Give_Payment( $payment_id );
+				$payment = new Give_Payment($payment_id);
 
-				$report['create_donation'] = ( ! empty( $report['create_donation'] ) ? ( absint( $report['create_donation'] ) + 1 ) : 1 );
+				$report['create_donation'] = ( ! empty($report['create_donation']) ? (absint($report['create_donation']) + 1) : 1);
 
-				$payment->update_meta( '_give_payment_import', true );
+				$payment->update_meta('_give_payment_import', true);
 
-				if ( ! empty( $import_setting['csv'] ) ) {
-					$payment->update_meta( '_give_payment_import_id', $import_setting['csv'] );
+				if ( ! empty($import_setting['csv'])) {
+					$payment->update_meta('_give_payment_import_id', $import_setting['csv']);
 				}
 
 				// Insert Company Name.
-				if ( ! empty( $data['company_name'] ) ) {
-					$payment->update_meta( '_give_donation_company', $data['company_name'] );
-					$donor_data->update_meta( '_give_donor_company', $data['company_name'] );
+				if ( ! empty($data['company_name'])) {
+					$payment->update_meta('_give_donation_company', $data['company_name']);
+					$donor_data->update_meta('_give_donor_company', $data['company_name']);
 				}
 
 				// Insert Notes.
-				if ( ! empty( $data['notes'] ) ) {
-					$payment->add_note( $data['notes'] );
+				if ( ! empty($data['notes'])) {
+					$payment->add_note($data['notes']);
 				}
 
-				$meta_exists = array_keys( $raw_key, 'post_meta' );
-				if ( ! empty( $main_key ) && ! empty( $meta_exists ) ) {
-					foreach ( $meta_exists as $meta_exist ) {
-						if ( ! empty( $main_key[ $meta_exist ] ) && ! empty( $row_data[ $meta_exist ] ) ) {
-							$payment->update_meta( $main_key[ $meta_exist ], $row_data[ $meta_exist ] );
+				$meta_exists = array_keys($raw_key, 'post_meta');
+				if ( ! empty($main_key) && ! empty($meta_exists)) {
+					foreach ($meta_exists as $meta_exist) {
+						if ( ! empty($main_key[$meta_exist]) && ! empty($row_data[$meta_exist])) {
+							$payment->update_meta($main_key[$meta_exist], $row_data[$meta_exist]);
 						}
 					}
 				}
 
 				// update the donation status if it's other then pending
-				if ( 'pending' !== $status ) {
-					$payment->update_status( $status );
+				if ('pending' !== $status) {
+					$payment->update_status($status);
 				}
 
 			} else {
-				$report['failed_donation'] = ( ! empty( $report['failed_donation'] ) ? ( absint( $report['failed_donation'] ) + 1 ) : 1 );
+				$report['failed_donation'] = ( ! empty($report['failed_donation']) ? (absint($report['failed_donation']) + 1) : 1);
 				$payment_id = false;
 			}
 
@@ -862,15 +862,15 @@ 
 			 * @param array $donor_data donor data
 			 * @param object $donor_data form object
 			 */
-			do_action( 'give_import_after_import_payment', $payment, $payment_data, $data, $donor_data, $form );
+			do_action('give_import_after_import_payment', $payment, $payment_data, $data, $donor_data, $form);
 		} else {
-			$report['create_donation'] = ( ! empty( $report['create_donation'] ) ? ( absint( $report['create_donation'] ) + 1 ) : 1 );
+			$report['create_donation'] = ( ! empty($report['create_donation']) ? (absint($report['create_donation']) + 1) : 1);
 			$payment_id = true;
 		}
 	}
 
 	// update the report
-	give_import_donation_report_update( $report );
+	give_import_donation_report_update($report);
 
 	return $payment_id;
 }
@@ -884,25 +884,25 @@ 
  *
  * @return string $status Donation status.
  */
-function give_import_donation_get_status( $data ) {
-	if ( empty( $data['post_status'] ) ) {
+function give_import_donation_get_status($data) {
+	if (empty($data['post_status'])) {
 		return 'publish';
 	}
 
 	$status = 'publish';
 
-	$donation_status = trim( $data['post_status'] );
-	$donation_status_key      = strtolower( preg_replace( '/\s+/', '', $donation_status ) );
+	$donation_status = trim($data['post_status']);
+	$donation_status_key = strtolower(preg_replace('/\s+/', '', $donation_status));
 
-	foreach ( give_get_payment_statuses() as $key => $value ) {
+	foreach (give_get_payment_statuses() as $key => $value) {
 		$match = false;
-		if ( $key === $donation_status_key ) {
+		if ($key === $donation_status_key) {
 			$match = true;
-		} else if ( stristr( $donation_status, $value ) ) {
+		} else if (stristr($donation_status, $value)) {
 			$match = true;
 		}
 
-		if ( ! empty( $match ) ) {
+		if ( ! empty($match)) {
 			$status = $key;
 			break;
 		}
@@ -922,12 +922,12 @@ 
  *
  * @return Give_Donor
  */
-function give_donation_import_update_donor_information( $donor, $payment_id, $payment_data ) {
+function give_donation_import_update_donor_information($donor, $payment_id, $payment_data) {
 	$old_donor = $donor;
-	if ( ! empty( $payment_data['donor_id'] ) ) {
-		$donor_id = absint( $payment_data['donor_id'] );
-		$donor    = new Give_Donor( $donor_id );
-		if ( ! empty( $donor->id ) ) {
+	if ( ! empty($payment_data['donor_id'])) {
+		$donor_id = absint($payment_data['donor_id']);
+		$donor    = new Give_Donor($donor_id);
+		if ( ! empty($donor->id)) {
 			return $donor;
 		}
 	}
@@ -940,12 +940,12 @@ 
  *
  * @since 1.8.13
  */
-function give_import_donation_insert_payment( $payment_id, $payment_data ) {
+function give_import_donation_insert_payment($payment_id, $payment_data) {
 	// Update Give Customers purchase_count
-	if ( ! empty( $payment_data['status'] ) && ( 'complete' === (string) $payment_data['status'] || 'publish' === (string) $payment_data['status'] ) ) {
-		$donor_id = (int) get_post_meta( $payment_id, '_give_payment_customer_id', true );
-		if ( ! empty( $donor_id ) ) {
-			$donor = new Give_Donor( $donor_id );
+	if ( ! empty($payment_data['status']) && ('complete' === (string) $payment_data['status'] || 'publish' === (string) $payment_data['status'])) {
+		$donor_id = (int) get_post_meta($payment_id, '_give_payment_customer_id', true);
+		if ( ! empty($donor_id)) {
+			$donor = new Give_Donor($donor_id);
 			$donor->increase_purchase_count();
 		}
 	}
@@ -956,8 +956,8 @@ 
  *
  * @since 1.8.13
  */
-function give_donation_import_give_insert_payment_args( $args, $payment_data ) {
-	if ( ! empty( $payment_data['user_info']['id'] ) ) {
+function give_donation_import_give_insert_payment_args($args, $payment_data) {
+	if ( ! empty($payment_data['user_info']['id'])) {
 		$args['post_author'] = (int) $payment_data['user_info']['id'];
 	}
 
@@ -969,11 +969,11 @@ 
  *
  * @since 1.8.13
  */
-function give_check_import_donation_duplicate( $payment_data, $data, $form, $donor_data ) {
+function give_check_import_donation_duplicate($payment_data, $data, $form, $donor_data) {
 	$return = false;
-	if ( ! empty( $data['post_date'] ) ) {
-		$post_date = mysql2date( 'Y-m-d-H-i-s', $payment_data['post_date'] );
-		$post_date = explode( '-', $post_date );
+	if ( ! empty($data['post_date'])) {
+		$post_date = mysql2date('Y-m-d-H-i-s', $payment_data['post_date']);
+		$post_date = explode('-', $post_date);
 		$args      = array(
 			'output'                 => 'post',
 			'cache_results'          => false,
@@ -994,7 +994,7 @@ 
 			'meta_query'             => array(
 				array(
 					'key'     => '_give_payment_total',
-					'value'   => preg_replace( '/[\$,]/', '', $payment_data['price'] ),
+					'value'   => preg_replace('/[\$,]/', '', $payment_data['price']),
 					'compare' => 'LIKE',
 				),
 				array(
@@ -1010,15 +1010,15 @@ 
 				),
 				array(
 					'key'     => '_give_payment_donor_id',
-					'value'   => isset( $donor_data->id ) ? $donor_data->id : '',
+					'value'   => isset($donor_data->id) ? $donor_data->id : '',
 					'compare' => '=',
 				),
 			),
 		);
 
-		$payments  = new Give_Payments_Query( $args );
+		$payments  = new Give_Payments_Query($args);
 		$donations = $payments->get_payments();
-		if ( ! empty( $donations ) ) {
+		if ( ! empty($donations)) {
 			$return = $donations;
 		}
 	}
@@ -1028,7 +1028,7 @@ 
 	 *
 	 * @since 1.8.18
 	 */
-	return apply_filters( 'give_check_import_donation_duplicate', $return, $payment_data, $data, $form, $donor_data );
+	return apply_filters('give_check_import_donation_duplicate', $return, $payment_data, $data, $form, $donor_data);
 }
 
 /**
@@ -1040,9 +1040,9 @@ 
  *
  * @return void
  */
-function give_donation_import_insert_default_payment_note( $payment_id ) {
+function give_donation_import_insert_default_payment_note($payment_id) {
 	$current_user = wp_get_current_user();
-	give_insert_payment_note( $payment_id, wp_sprintf( __( 'This donation was imported by %s', 'give' ), $current_user->user_email ) );
+	give_insert_payment_note($payment_id, wp_sprintf(__('This donation was imported by %s', 'give'), $current_user->user_email));
 }
 
 /**
@@ -1054,14 +1054,14 @@ 
  *
  * @return string URL
  */
-function give_import_page_url( $parameter = array() ) {
+function give_import_page_url($parameter = array()) {
 	$defalut_query_arg = array(
 		'post_type'     => 'give_forms',
 		'page'          => 'give-tools',
 		'tab'           => 'import',
 		'importer-type' => 'import_donations',
 	);
-	$import_query_arg  = wp_parse_args( $parameter, $defalut_query_arg );
+	$import_query_arg = wp_parse_args($parameter, $defalut_query_arg);
 
-	return add_query_arg( $import_query_arg, admin_url( 'edit.php' ) );
+	return add_query_arg($import_query_arg, admin_url('edit.php'));
 }
\ No newline at end of file

includes/payments/class-give-sequential-donation-number.php

Doc Comments +1 added lines, -1 removed lines

@@ -197,7 +197,7 @@
 	 * @since
 	 * @access public
 	 *
-	 * @param $serial_number
+	 * @param integer $serial_number
 	 *
 	 * @return string
 	 */

Spacing +47 added lines, -48 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 // Exit if access directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -40,7 +40,7 @@ 
 	 * @return Give_Sequential_Donation_Number
 	 */
 	public static function get_instance() {
-		if ( null === static::$instance ) {
+		if (null === static::$instance) {
 			self::$instance = new static();
 
 			self::$instance->init();
@@ -56,8 +56,8 @@ 
 	 * @since 2.1.0
 	 */
 	public function init() {
-		add_action( 'wp_insert_post', array( $this, '__save_donation_title' ), 10, 3 );
-		add_action( 'after_delete_post', array( $this, '__remove_serial_number' ), 10, 1 );
+		add_action('wp_insert_post', array($this, '__save_donation_title'), 10, 3);
+		add_action('after_delete_post', array($this, '__remove_serial_number'), 10, 1);
 	}
 
 	/**
@@ -73,27 +73,27 @@ 
 	 *
 	 * @return void
 	 */
-	public function __save_donation_title( $donation_id, $post, $existing_donation_updated ) {
+	public function __save_donation_title($donation_id, $post, $existing_donation_updated) {
 		// Bailout
 		if (
-			! give_is_setting_enabled( give_get_option( 'sequential-ordering_status', 'disabled' ) )
+			! give_is_setting_enabled(give_get_option('sequential-ordering_status', 'disabled'))
 			|| $existing_donation_updated
 			|| 'give_payment' !== $post->post_type
 		) {
 			return;
 		}
 
-		$serial_number = $this->__set_donation_number( $donation_id );
-		$serial_code   = $this->set_number_padding( $serial_number );
+		$serial_number = $this->__set_donation_number($donation_id);
+		$serial_code   = $this->set_number_padding($serial_number);
 
 		// Add prefix.
-		if ( $prefix = give_get_option( 'sequential-ordering_number_prefix', '' ) ) {
-			$serial_code = $prefix . $serial_code;
+		if ($prefix = give_get_option('sequential-ordering_number_prefix', '')) {
+			$serial_code = $prefix.$serial_code;
 		}
 
 		// Add suffix.
-		if ( $suffix = give_get_option( 'sequential-ordering_number_suffix', '' ) ) {
-			$serial_code = $serial_code . $suffix;
+		if ($suffix = give_get_option('sequential-ordering_number_suffix', '')) {
+			$serial_code = $serial_code.$suffix;
 		}
 
 		/**
@@ -103,7 +103,7 @@ 
 		 */
 		$serial_code = apply_filters(
 			'give_set_sequential_donation_title',
-			give_time_do_tags( $serial_code ),
+			give_time_do_tags($serial_code),
 			$donation_id,
 			$post,
 			$existing_donation_updated,
@@ -120,17 +120,17 @@ 
 				array(
 					'ID'         => $donation_id,
 					'post_name'  => "{$this->donation_title_prefix}-{$serial_number}",
-					'post_title' => trim( $serial_code )
+					'post_title' => trim($serial_code)
 				)
 			);
 
-			if ( is_wp_error( $wp_error ) ) {
-				throw new Exception( $wp_error->get_error_message() );
+			if (is_wp_error($wp_error)) {
+				throw new Exception($wp_error->get_error_message());
 			}
 
-			give_update_option( 'sequential-ordering_number', ( $serial_number + 1 ) );
-		} catch ( Exception $e ) {
-			error_log( "Give caught exception: {$e->getMessage()}" );
+			give_update_option('sequential-ordering_number', ($serial_number + 1));
+		} catch (Exception $e) {
+			error_log("Give caught exception: {$e->getMessage()}");
 		}
 	}
 
@@ -145,18 +145,18 @@ 
 	 *
 	 * @return int
 	 */
-	public function __set_donation_number( $donation_id ) {
+	public function __set_donation_number($donation_id) {
 		$table_data = array(
 			'payment_id' => $donation_id
 		);
 
 		// Customize sequential donation number starting point if needed.
 		if (
-			get_option( '_give_reset_sequential_number' ) &&
-			( $number = give_get_option( 'sequential-ordering_number', 0 ) )
+			get_option('_give_reset_sequential_number') &&
+			($number = give_get_option('sequential-ordering_number', 0))
 		) {
-			if ( Give()->sequential_donation_db->get_id_auto_increment_val() <= $number ) {
-				delete_option( '_give_reset_sequential_number' );
+			if (Give()->sequential_donation_db->get_id_auto_increment_val() <= $number) {
+				delete_option('_give_reset_sequential_number');
 			}
 
 			$table_data['id'] = $number;
@@ -170,7 +170,7 @@ 
 		 */
 		return apply_filters(
 			'give_set_sequential_donation_number',
-			Give()->sequential_donation_db->insert( $table_data ),
+			Give()->sequential_donation_db->insert($table_data),
 			$table_data
 		);
 	}
@@ -187,8 +187,8 @@ 
 	 *
 	 * @return bool
 	 */
-	public function __remove_serial_number( $donation_id ) {
-		return Give()->sequential_donation_db->delete( $this->get_serial_number( $donation_id ) );
+	public function __remove_serial_number($donation_id) {
+		return Give()->sequential_donation_db->delete($this->get_serial_number($donation_id));
 	}
 
 	/**
@@ -201,9 +201,9 @@ 
 	 *
 	 * @return string
 	 */
-	public function set_number_padding( $serial_number ) {
-		if ( $number_padding = give_get_option( 'sequential-ordering_number_padding', 0 ) ) {
-			$serial_number = str_pad( $serial_number, $number_padding, '0', STR_PAD_LEFT );
+	public function set_number_padding($serial_number) {
+		if ($number_padding = give_get_option('sequential-ordering_number_padding', 0)) {
+			$serial_number = str_pad($serial_number, $number_padding, '0', STR_PAD_LEFT);
 		}
 
 		return $serial_number;
@@ -220,12 +220,12 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_serial_code( $donation, $args = array() ) {
+	public function get_serial_code($donation, $args = array()) {
 		// Get id from object.
-		if( ! is_numeric( $donation ) ) {
-			if( $donation instanceof Give_Payment ) {
+		if ( ! is_numeric($donation)) {
+			if ($donation instanceof Give_Payment) {
 				$donation = $donation->ID;
-			} elseif ( $donation instanceof WP_Post ){
+			} elseif ($donation instanceof WP_Post) {
 				$donation = $donation->ID;
 			}
 		}
@@ -241,8 +241,8 @@ 
 
 		$serial_code = $args['default'] ? $donation : '';
 
-		if ( $donation_number = $this->get_serial_number( $donation ) ) {
-			$serial_code = get_the_title( $donation );
+		if ($donation_number = $this->get_serial_number($donation)) {
+			$serial_code = get_the_title($donation);
 		}
 
 		$serial_code = $args['with_hash'] ? "#{$serial_code}" : $serial_code;
@@ -257,7 +257,7 @@ 
 		 * @param array $args
 		 * @param string $donation_number
 		 */
-		return apply_filters( 'give_get_donation_serial_code', $serial_code, $donation, $args, $donation_number );
+		return apply_filters('give_get_donation_serial_code', $serial_code, $donation, $args, $donation_number);
 	}
 
 	/**
@@ -270,12 +270,12 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_serial_number( $donation_id_or_serial_code ) {
-		if ( is_numeric( $donation_id_or_serial_code ) ) {
-			return Give()->sequential_donation_db->get_column_by( 'id', 'payment_id', $donation_id_or_serial_code );
+	public function get_serial_number($donation_id_or_serial_code) {
+		if (is_numeric($donation_id_or_serial_code)) {
+			return Give()->sequential_donation_db->get_column_by('id', 'payment_id', $donation_id_or_serial_code);
 		}
 
-		return $this->get_serial_number( $this->get_donation_id( $donation_id_or_serial_code ) );
+		return $this->get_serial_number($this->get_donation_id($donation_id_or_serial_code));
 	}
 
 
@@ -289,10 +289,10 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_donation_id( $donation_number_or_serial_code ) {
+	public function get_donation_id($donation_number_or_serial_code) {
 		global $wpdb;
 
-		if ( is_numeric( $donation_number_or_serial_code ) ) {
+		if (is_numeric($donation_number_or_serial_code)) {
 			return Give()->sequential_donation_db->get_column_by(
 				'payment_id',
 				'id',
@@ -377,12 +377,11 @@ 
 		$donation_id = $this->get_max_donation_id();
 		$next_number = $this->get_max_number();
 
-		if ( ! $this->get_serial_number( $donation_id ) ) {
-			$next_number = $donation_id && ( $next_number < $donation_id ) ?
-				$donation_id :
-				$this->get_max_number();
+		if ( ! $this->get_serial_number($donation_id)) {
+			$next_number = $donation_id && ($next_number < $donation_id) ?
+				$donation_id : $this->get_max_number();
 		}
 
-		return ( $next_number + 1 );
+		return ($next_number + 1);
 	}
 }

includes/payments/class-payment-stats.php

Doc Comments +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
 	 * @param  $end_date   string|bool  The end date for which we'd like to filter our sale stats. If false, we'll use the default end date of `this_month`
 	 * @param  $status     string|array The sale status(es) to count. Only valid when retrieving global stats
 	 *
-	 * @return float|int                Total amount of donations based on the passed arguments.
+	 * @return string                Total amount of donations based on the passed arguments.
 	 */
 	public function get_sales( $form_id = 0, $start_date = false, $end_date = false, $status = 'publish' ) {
 
@@ -84,7 +84,7 @@ 
 	 * @param  $end_date    string|bool The end date for which we'd like to filter the donations stats. If false, method will use the default end date of `this_month`.
 	 * @param  $gateway_id  string|bool The gateway to get earnings for such as 'paypal' or 'stripe'.
 	 *
-	 * @return float|int                Total amount of donations based on the passed arguments.
+	 * @return string                Total amount of donations based on the passed arguments.
 	 */
 	public function get_earnings( $form_id = 0, $start_date = false, $end_date = false, $gateway_id = false ) {
 		global $wpdb;
@@ -207,7 +207,7 @@ 
 	 * @param  $end_date    string|bool The end date for which we'd like to filter our sale stats. If false, we'll use the default end date of `this_month`
 	 * @param  $gateway_id  string|bool The gateway to get earnings for such as 'paypal' or 'stripe'
 	 *
-	 * @return float|int                Total amount of donations based on the passed arguments.
+	 * @return string                Total amount of donations based on the passed arguments.
 	 */
 	public function get_earnings_cache_key( $form_id = 0, $start_date = false, $end_date = false, $gateway_id = false ) {
 

Spacing +46 added lines, -46 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -38,17 +38,17 @@ 
 	 *
 	 * @return float|int                Total amount of donations based on the passed arguments.
 	 */
-	public function get_sales( $form_id = 0, $start_date = false, $end_date = false, $status = 'publish' ) {
+	public function get_sales($form_id = 0, $start_date = false, $end_date = false, $status = 'publish') {
 
-		$this->setup_dates( $start_date, $end_date );
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
@@ -57,19 +57,19 @@ 
 			'start_date' => $this->start_date,
 			'end_date'   => $this->end_date,
 			'fields'     => 'ids',
-			'number'     => - 1,
+			'number'     => -1,
 			'output'     => ''
 		);
 
-		if ( ! empty( $form_id ) ) {
+		if ( ! empty($form_id)) {
 			$args['give_forms'] = $form_id;
 		}
 
 		/* @var Give_Payments_Query $payments */
-		$payments = new Give_Payments_Query( $args );
+		$payments = new Give_Payments_Query($args);
 		$payments = $payments->get_payments();
 
-		return count( $payments );
+		return count($payments);
 	}
 
 
@@ -86,17 +86,17 @@ 
 	 *
 	 * @return float|int                Total amount of donations based on the passed arguments.
 	 */
-	public function get_earnings( $form_id = 0, $start_date = false, $end_date = false, $gateway_id = false ) {
+	public function get_earnings($form_id = 0, $start_date = false, $end_date = false, $gateway_id = false) {
 		global $wpdb;
-		$this->setup_dates( $start_date, $end_date );
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
@@ -106,13 +106,13 @@ 
 			'start_date' => $this->start_date,
 			'end_date'   => $this->end_date,
 			'fields'     => 'ids',
-			'number'     => - 1,
+			'number'     => -1,
 			'output'     => '',
 		);
 
 
 		// Filter by Gateway ID meta_key
-		if ( $gateway_id ) {
+		if ($gateway_id) {
 			$args['meta_query'][] = array(
 				'key'   => '_give_payment_gateway',
 				'value' => $gateway_id,
@@ -120,40 +120,40 @@ 
 		}
 
 		// Filter by Gateway ID meta_key
-		if ( $form_id ) {
+		if ($form_id) {
 			$args['meta_query'][] = array(
 				'key'   => '_give_payment_form_id',
 				'value' => $form_id,
 			);
 		}
 
-		if ( ! empty( $args['meta_query'] ) && 1 < count( $args['meta_query'] ) ) {
+		if ( ! empty($args['meta_query']) && 1 < count($args['meta_query'])) {
 			$args['meta_query']['relation'] = 'AND';
 		}
 
-		$args = apply_filters( 'give_stats_earnings_args', $args );
-		$key  = Give_Cache::get_key( 'give_stats', $args );
+		$args = apply_filters('give_stats_earnings_args', $args);
+		$key  = Give_Cache::get_key('give_stats', $args);
 
 		// Set transient for faster stats.
-		$earnings = Give_Cache::get( $key );
+		$earnings = Give_Cache::get($key);
 
-		if ( false === $earnings ) {
+		if (false === $earnings) {
 
 			$this->timestamp = false;
-			$payments        = new Give_Payments_Query( $args );
+			$payments        = new Give_Payments_Query($args);
 			$payments        = $payments->get_payments();
 			$earnings        = 0;
 
-			if ( ! empty( $payments ) ) {
+			if ( ! empty($payments)) {
 				$query = "SELECT payment_id as id, meta_value as total
 					FROM {$wpdb->paymentmeta}
 					WHERE meta_key='_give_payment_total'
-					AND payment_id IN ('". implode( '\',\'', $payments ) ."')";
+					AND payment_id IN ('".implode('\',\'', $payments)."')";
 
 				$payments = $wpdb->get_results($query, ARRAY_A);
 
-				if( ! empty( $payments ) ) {
-					foreach ( $payments as $payment ) {
+				if ( ! empty($payments)) {
+					foreach ($payments as $payment) {
 						/**
 						 * Filter the donation amount
 						 * Note: this filter documented in payments/functions.php:give_donation_amount()
@@ -162,20 +162,20 @@ 
 						 */
 						$formatted_amount = apply_filters(
 							'give_donation_amount',
-							give_format_amount(  $payment['total'], array( 'donation_id' =>  $payment['id'] ) ),
+							give_format_amount($payment['total'], array('donation_id' =>  $payment['id'])),
 							$payment['total'],
 							$payment['id'],
-							array( 'type' => 'stats', 'currency'=> false, 'amount' => false )
+							array('type' => 'stats', 'currency'=> false, 'amount' => false)
 						);
 
-						$earnings += (float) give_maybe_sanitize_amount( $formatted_amount );
+						$earnings += (float) give_maybe_sanitize_amount($formatted_amount);
 					}
 				}
 
 			}
 
 			// Cache the results for one hour.
-			Give_Cache::set( $key, give_sanitize_amount_for_db( $earnings ), 60 * 60 );
+			Give_Cache::set($key, give_sanitize_amount_for_db($earnings), 60 * 60);
 		}
 
 		/**
@@ -189,10 +189,10 @@ 
 		 * @param  string|bool $end_date   Earning end date.
 		 * @param  string|bool $gateway_id Payment gateway id.
 		 */
-		$earnings = apply_filters( 'give_get_earnings', $earnings, $form_id, $start_date, $end_date, $gateway_id );
+		$earnings = apply_filters('give_get_earnings', $earnings, $form_id, $start_date, $end_date, $gateway_id);
 
 		//return earnings
-		return round( $earnings, give_get_price_decimals( $form_id ) );
+		return round($earnings, give_get_price_decimals($form_id));
 
 	}
 
@@ -209,17 +209,17 @@ 
 	 *
 	 * @return float|int                Total amount of donations based on the passed arguments.
 	 */
-	public function get_earnings_cache_key( $form_id = 0, $start_date = false, $end_date = false, $gateway_id = false ) {
+	public function get_earnings_cache_key($form_id = 0, $start_date = false, $end_date = false, $gateway_id = false) {
 
-		$this->setup_dates( $start_date, $end_date );
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
@@ -229,12 +229,12 @@ 
 			'start_date' => $this->start_date,
 			'end_date'   => $this->end_date,
 			'fields'     => 'ids',
-			'number'     => - 1,
+			'number'     => -1,
 		);
 
 
 		// Filter by Gateway ID meta_key
-		if ( $gateway_id ) {
+		if ($gateway_id) {
 			$args['meta_query'][] = array(
 				'key'   => '_give_payment_gateway',
 				'value' => $gateway_id,
@@ -242,19 +242,19 @@ 
 		}
 
 		// Filter by Gateway ID meta_key
-		if ( $form_id ) {
+		if ($form_id) {
 			$args['meta_query'][] = array(
 				'key'   => '_give_payment_form_id',
 				'value' => $form_id,
 			);
 		}
 
-		if ( ! empty( $args['meta_query'] ) && 1 < count( $args['meta_query'] ) ) {
+		if ( ! empty($args['meta_query']) && 1 < count($args['meta_query'])) {
 			$args['meta_query']['relation'] = 'AND';
 		}
 
-		$args = apply_filters( 'give_stats_earnings_args', $args );
-		$key  = Give_Cache::get_key( 'give_stats', $args );
+		$args = apply_filters('give_stats_earnings_args', $args);
+		$key  = Give_Cache::get_key('give_stats', $args);
 
 		//return earnings
 		return $key;
@@ -272,17 +272,17 @@ 
 	 *
 	 * @return array       Best selling forms
 	 */
-	public function get_best_selling( $number = 10 ) {
+	public function get_best_selling($number = 10) {
 		global $wpdb;
 
-		$meta_table = __give_v20_bc_table_details( 'form' );
+		$meta_table = __give_v20_bc_table_details('form');
 
-		$give_forms = $wpdb->get_results( $wpdb->prepare(
+		$give_forms = $wpdb->get_results($wpdb->prepare(
 			"SELECT {$meta_table['column']['id']} as form_id, max(meta_value) as sales
 				FROM {$meta_table['name']} WHERE meta_key='_give_form_sales' AND meta_value > 0
 				GROUP BY meta_value+0
 				DESC LIMIT %d;", $number
-		) );
+		));
 
 		return $give_forms;
 	}