WordImpress / Give

includes/class-give-db.php

Spacing +34 added lines, -34 removed lines

@@ -8,7 +8,7 @@ 
  * @license     http://opensource.org/licenses/gpl-2.0.php GNU Public License
  * @since       1.0
  */
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 } // Exit if accessed directly
 
@@ -77,10 +77,10 @@ 
 	 * @since   1.0
 	 * @return  object
 	 */
-	public function get( $row_id ) {
+	public function get($row_id) {
 		global $wpdb;
 
-		return $wpdb->get_row( "SELECT * FROM $this->table_name WHERE $this->primary_key = $row_id LIMIT 1;" );
+		return $wpdb->get_row("SELECT * FROM $this->table_name WHERE $this->primary_key = $row_id LIMIT 1;");
 	}
 
 	/**
@@ -90,10 +90,10 @@ 
 	 * @since   1.0
 	 * @return  object
 	 */
-	public function get_by( $column, $row_id ) {
+	public function get_by($column, $row_id) {
 		global $wpdb;
 
-		return $wpdb->get_row( "SELECT * FROM $this->table_name WHERE $column = '$row_id' LIMIT 1;" );
+		return $wpdb->get_row("SELECT * FROM $this->table_name WHERE $column = '$row_id' LIMIT 1;");
 	}
 
 	/**
@@ -103,10 +103,10 @@ 
 	 * @since   1.0
 	 * @return  string
 	 */
-	public function get_column( $column, $row_id ) {
+	public function get_column($column, $row_id) {
 		global $wpdb;
 
-		return $wpdb->get_var( "SELECT $column FROM $this->table_name WHERE $this->primary_key = $row_id LIMIT 1;" );
+		return $wpdb->get_var("SELECT $column FROM $this->table_name WHERE $this->primary_key = $row_id LIMIT 1;");
 	}
 
 	/**
@@ -116,10 +116,10 @@ 
 	 * @since   1.0
 	 * @return  string
 	 */
-	public function get_column_by( $column, $column_where, $column_value ) {
+	public function get_column_by($column, $column_where, $column_value) {
 		global $wpdb;
 
-		return $wpdb->get_var( "SELECT $column FROM $this->table_name WHERE $column_where = '$column_value' LIMIT 1;" );
+		return $wpdb->get_var("SELECT $column FROM $this->table_name WHERE $column_where = '$column_value' LIMIT 1;");
 	}
 
 	/**
@@ -129,30 +129,30 @@ 
 	 * @since   1.0
 	 * @return  int
 	 */
-	public function insert( $data, $type = '' ) {
+	public function insert($data, $type = '') {
 		global $wpdb;
 
 		// Set default values
-		$data = wp_parse_args( $data, $this->get_column_defaults() );
+		$data = wp_parse_args($data, $this->get_column_defaults());
 
-		do_action( 'give_pre_insert_' . $type, $data );
+		do_action('give_pre_insert_'.$type, $data);
 
 		// Initialise column format array
 		$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);
 
-		$wpdb->insert( $this->table_name, $data, $column_formats );
+		$wpdb->insert($this->table_name, $data, $column_formats);
 
-		do_action( 'give_post_insert_' . $type, $wpdb->insert_id, $data );
+		do_action('give_post_insert_'.$type, $wpdb->insert_id, $data);
 
 		return $wpdb->insert_id;
 	}
@@ -164,18 +164,18 @@ 
 	 * @since   1.0
 	 * @return  bool
 	 */
-	public function update( $row_id, $data = array(), $where = '' ) {
+	public function update($row_id, $data = array(), $where = '') {
 
 		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;
 		}
 
@@ -183,16 +183,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;
 		}
 
@@ -206,18 +206,18 @@ 
 	 * @since   1.0
 	 * @return  bool
 	 */
-	public function delete( $row_id = 0 ) {
+	public function delete($row_id = 0) {
 
 		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;
 		}
 
@@ -231,11 +231,11 @@ 
 	 * @param  string $table The table name
 	 * @return bool          If the table name exists
 	 */
-	public function table_exists( $table ) {
+	public function table_exists($table) {
 		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;
 	}
 
 }

includes/class-give-donate-form.php

Spacing +62 added lines, -62 removed lines

@@ -63,37 +63,37 @@ 
 	 *
 	 * @since 1.0
 	 */
-	public function __construct( $_id = false, $_args = array() ) {
+	public function __construct($_id = false, $_args = array()) {
 
-		if ( false === $_id ) {
+		if (false === $_id) {
 
 			$defaults = array(
 				'post_type'   => 'give_forms',
 				'post_status' => 'draft',
-				'post_title'  => __( 'New Give Form', 'give' )
+				'post_title'  => __('New Give Form', 'give')
 			);
 
-			$args = wp_parse_args( $_args, $defaults );
+			$args = wp_parse_args($_args, $defaults);
 
-			$_id = wp_insert_post( $args, true );
+			$_id = wp_insert_post($args, true);
 
 		}
 
-		$donate_form = WP_Post::get_instance( $_id );
+		$donate_form = WP_Post::get_instance($_id);
 
-		if ( ! is_object( $donate_form ) ) {
+		if ( ! is_object($donate_form)) {
 			return false;
 		}
 
-		if ( ! is_a( $donate_form, 'WP_Post' ) ) {
+		if ( ! is_a($donate_form, 'WP_Post')) {
 			return false;
 		}
 
-		if ( 'give_forms' !== $donate_form->post_type ) {
+		if ('give_forms' !== $donate_form->post_type) {
 			return false;
 		}
 
-		foreach ( $donate_form as $key => $value ) {
+		foreach ($donate_form as $key => $value) {
 
 			$this->$key = $value;
 
@@ -106,15 +106,15 @@ 
 	 *
 	 * @since 1.0
 	 */
-	public function __get( $key ) {
+	public function __get($key) {
 
-		if ( method_exists( $this, 'get_' . $key ) ) {
+		if (method_exists($this, 'get_'.$key)) {
 
-			return call_user_func( array( $this, 'get_' . $key ) );
+			return call_user_func(array($this, 'get_'.$key));
 
 		} else {
 
-			throw new Exception( 'Can\'t get property ' . $key );
+			throw new Exception('Can\'t get property '.$key);
 
 		}
 
@@ -140,13 +140,13 @@ 
 	 */
 	public function get_price() {
 
-		if ( ! isset( $this->price ) ) {
+		if ( ! isset($this->price)) {
 
-			$this->price = get_post_meta( $this->ID, '_give_set_price', true );
+			$this->price = get_post_meta($this->ID, '_give_set_price', true);
 
-			if ( $this->price ) {
+			if ($this->price) {
 
-				$this->price = give_sanitize_amount( $this->price );
+				$this->price = give_sanitize_amount($this->price);
 
 			} else {
 
@@ -156,7 +156,7 @@ 
 
 		}
 
-		return apply_filters( 'give_get_set_price', $this->price, $this->ID );
+		return apply_filters('give_get_set_price', $this->price, $this->ID);
 	}
 
 	/**
@@ -167,13 +167,13 @@ 
 	 */
 	public function get_prices() {
 
-		if ( ! isset( $this->prices ) ) {
+		if ( ! isset($this->prices)) {
 
-			$this->prices = get_post_meta( $this->ID, '_give_donation_levels', true );
+			$this->prices = get_post_meta($this->ID, '_give_donation_levels', true);
 
 		}
 
-		return apply_filters( 'give_get_donation_levels', $this->prices, $this->ID );
+		return apply_filters('give_get_donation_levels', $this->prices, $this->ID);
 
 	}
 
@@ -185,13 +185,13 @@ 
 	 */
 	public function get_goal() {
 
-		if ( ! isset( $this->goal ) ) {
+		if ( ! isset($this->goal)) {
 
-			$this->goal = get_post_meta( $this->ID, '_give_set_goal', true );
+			$this->goal = get_post_meta($this->ID, '_give_set_goal', true);
 
-			if ( $this->goal ) {
+			if ($this->goal) {
 
-				$this->goal = give_sanitize_amount( $this->goal );
+				$this->goal = give_sanitize_amount($this->goal);
 
 			} else {
 
@@ -201,7 +201,7 @@ 
 
 		}
 
-		return apply_filters( 'give_get_set_goal', $this->goal, $this->ID );
+		return apply_filters('give_get_set_goal', $this->goal, $this->ID);
 
 	}
 
@@ -213,9 +213,9 @@ 
 	 */
 	public function is_single_price_mode() {
 
-		$ret = get_post_meta( $this->ID, '_give_price_options_mode', true );
+		$ret = get_post_meta($this->ID, '_give_price_options_mode', true);
 
-		return (bool) apply_filters( 'give_single_price_option_mode', $ret, $this->ID );
+		return (bool) apply_filters('give_single_price_option_mode', $ret, $this->ID);
 
 	}
 
@@ -229,14 +229,14 @@ 
 	 */
 	public function has_variable_prices() {
 
-		$option = get_post_meta( $this->ID, '_give_price_option', true );
+		$option = get_post_meta($this->ID, '_give_price_option', true);
 		$ret    = 0;
 
-		if ( $option === 'multi' ) {
+		if ($option === 'multi') {
 			$ret = 1;
 		}
 
-		return (bool) apply_filters( 'give_has_variable_prices', $ret, $this->ID );
+		return (bool) apply_filters('give_has_variable_prices', $ret, $this->ID);
 
 	}
 
@@ -249,15 +249,15 @@ 
 	 */
 	public function get_sales() {
 
-		if ( ! isset( $this->sales ) ) {
+		if ( ! isset($this->sales)) {
 
-			if ( '' == get_post_meta( $this->ID, '_give_form_sales', true ) ) {
-				add_post_meta( $this->ID, '_give_form_sales', 0 );
+			if ('' == get_post_meta($this->ID, '_give_form_sales', true)) {
+				add_post_meta($this->ID, '_give_form_sales', 0);
 			} // End if
 
-			$this->sales = get_post_meta( $this->ID, '_give_form_sales', true );
+			$this->sales = get_post_meta($this->ID, '_give_form_sales', true);
 
-			if ( $this->sales < 0 ) {
+			if ($this->sales < 0) {
 				// Never let sales be less than zero
 				$this->sales = 0;
 			}
@@ -276,10 +276,10 @@ 
 	 */
 	public function increase_sales() {
 
-		$sales = give_get_form_sales_stats( $this->ID );
+		$sales = give_get_form_sales_stats($this->ID);
 		$sales = $sales + 1;
 
-		if ( update_post_meta( $this->ID, '_give_form_sales', $sales ) ) {
+		if (update_post_meta($this->ID, '_give_form_sales', $sales)) {
 			$this->sales = $sales;
 
 			return $sales;
@@ -296,13 +296,13 @@ 
 	 */
 	public function decrease_sales() {
 
-		$sales = give_get_form_sales_stats( $this->ID );
-		if ( $sales > 0 ) // Only decrease if not already zero
+		$sales = give_get_form_sales_stats($this->ID);
+		if ($sales > 0) // Only decrease if not already zero
 		{
 			$sales = $sales - 1;
 		}
 
-		if ( update_post_meta( $this->ID, '_give_form_sales', $sales ) ) {
+		if (update_post_meta($this->ID, '_give_form_sales', $sales)) {
 			$this->sales = $sales;
 
 			return $sales;
@@ -320,15 +320,15 @@ 
 	 */
 	public function get_earnings() {
 
-		if ( ! isset( $this->earnings ) ) {
+		if ( ! isset($this->earnings)) {
 
-			if ( '' == get_post_meta( $this->ID, '_give_form_earnings', true ) ) {
-				add_post_meta( $this->ID, '_give_form_earnings', 0 );
+			if ('' == get_post_meta($this->ID, '_give_form_earnings', true)) {
+				add_post_meta($this->ID, '_give_form_earnings', 0);
 			}
 
-			$this->earnings = get_post_meta( $this->ID, '_give_form_earnings', true );
+			$this->earnings = get_post_meta($this->ID, '_give_form_earnings', true);
 
-			if ( $this->earnings < 0 ) {
+			if ($this->earnings < 0) {
 				// Never let earnings be less than zero
 				$this->earnings = 0;
 			}
@@ -345,12 +345,12 @@ 
 	 * @since 1.0
 	 * @return float|false
 	 */
-	public function increase_earnings( $amount = 0 ) {
+	public function increase_earnings($amount = 0) {
 
-		$earnings = give_get_form_earnings_stats( $this->ID );
+		$earnings = give_get_form_earnings_stats($this->ID);
 		$earnings = $earnings + (float) $amount;
 
-		if ( update_post_meta( $this->ID, '_give_form_earnings', $earnings ) ) {
+		if (update_post_meta($this->ID, '_give_form_earnings', $earnings)) {
 			$this->earnings = $earnings;
 
 			return $earnings;
@@ -366,16 +366,16 @@ 
 	 * @since 1.0
 	 * @return float|false
 	 */
-	public function decrease_earnings( $amount ) {
+	public function decrease_earnings($amount) {
 
-		$earnings = give_get_form_earnings_stats( $this->ID );
+		$earnings = give_get_form_earnings_stats($this->ID);
 
-		if ( $earnings > 0 ) // Only decrease if greater than zero
+		if ($earnings > 0) // Only decrease if greater than zero
 		{
 			$earnings = $earnings - (float) $amount;
 		}
 
-		if ( update_post_meta( $this->ID, '_give_form_earnings', $earnings ) ) {
+		if (update_post_meta($this->ID, '_give_form_earnings', $earnings)) {
 			$this->earnings = $earnings;
 
 			return $earnings;
@@ -391,22 +391,22 @@ 
 	 * @since 1.0
 	 * @return bool
 	 */
-	public function is_free( $price_id = false ) {
+	public function is_free($price_id = false) {
 
 		$is_free          = false;
-		$variable_pricing = give_has_variable_prices( $this->ID );
+		$variable_pricing = give_has_variable_prices($this->ID);
 
-		if ( $variable_pricing && ! is_null( $price_id ) && $price_id !== false ) {
-			$price = give_get_price_option_amount( $this->ID, $price_id );
-		} elseif ( ! $variable_pricing ) {
-			$price = get_post_meta( $this->ID, '_give_set_price', true );
+		if ($variable_pricing && ! is_null($price_id) && $price_id !== false) {
+			$price = give_get_price_option_amount($this->ID, $price_id);
+		} elseif ( ! $variable_pricing) {
+			$price = get_post_meta($this->ID, '_give_set_price', true);
 		}
 
-		if ( isset( $price ) && (float) $price == 0 ) {
+		if (isset($price) && (float) $price == 0) {
 			$is_free = true;
 		}
 
-		return (bool) apply_filters( 'give_is_free_donation', $is_free, $this->ID, $price_id );
+		return (bool) apply_filters('give_is_free_donation', $is_free, $this->ID, $price_id);
 
 	}
 

Braces +6 added lines, -2 removed lines

@@ -297,10 +297,12 @@ 
 	public function decrease_sales() {
 
 		$sales = give_get_form_sales_stats( $this->ID );
-		if ( $sales > 0 ) // Only decrease if not already zero
+		if ( $sales > 0 ) {
+			// Only decrease if not already zero
 		{
 			$sales = $sales - 1;
 		}
+		}
 
 		if ( update_post_meta( $this->ID, '_give_form_sales', $sales ) ) {
 			$this->sales = $sales;
@@ -370,10 +372,12 @@ 
 
 		$earnings = give_get_form_earnings_stats( $this->ID );
 
-		if ( $earnings > 0 ) // Only decrease if greater than zero
+		if ( $earnings > 0 ) {
+			// Only decrease if greater than zero
 		{
 			$earnings = $earnings - (float) $amount;
 		}
+		}
 
 		if ( update_post_meta( $this->ID, '_give_form_earnings', $earnings ) ) {
 			$this->earnings = $earnings;

includes/class-give-html-elements.php

Spacing +124 added lines, -124 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -33,7 +33,7 @@ 
 	 *
 	 * @return string $output Give forms dropdown
 	 */
-	public function forms_dropdown( $args = array() ) {
+	public function forms_dropdown($args = array()) {
 
 		$defaults = array(
 			'name'        => 'forms',
@@ -43,43 +43,43 @@ 
 			'selected'    => 0,
 			'chosen'      => false,
 			'number'      => 30,
-			'placeholder' => sprintf( __( 'Select a %s', 'give' ), give_get_forms_label_singular() )
+			'placeholder' => sprintf(__('Select a %s', 'give'), give_get_forms_label_singular())
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$forms = get_posts( array(
+		$forms = get_posts(array(
 			'post_type'      => 'give_forms',
 			'orderby'        => 'title',
 			'order'          => 'ASC',
 			'posts_per_page' => $args['number']
-		) );
+		));
 
 		$options = array();
 
-		if ( $forms ) {
-			$options[0] = sprintf( __( 'Select a %s', 'give' ), give_get_forms_label_singular() );
-			foreach ( $forms as $form ) {
-				$options[ absint( $form->ID ) ] = esc_html( $form->post_title );
+		if ($forms) {
+			$options[0] = sprintf(__('Select a %s', 'give'), give_get_forms_label_singular());
+			foreach ($forms as $form) {
+				$options[absint($form->ID)] = esc_html($form->post_title);
 			}
 		} else {
-			$options[0] = __( 'No Give Forms Found', 'give' );
+			$options[0] = __('No Give Forms Found', 'give');
 		}
 
 		// This ensures that any selected forms are included in the drop down
-		if ( is_array( $args['selected'] ) ) {
-			foreach ( $args['selected'] as $item ) {
-				if ( ! in_array( $item, $options ) ) {
-					$options[ $item ] = get_the_title( $item );
+		if (is_array($args['selected'])) {
+			foreach ($args['selected'] as $item) {
+				if ( ! in_array($item, $options)) {
+					$options[$item] = get_the_title($item);
 				}
 			}
-		} elseif ( is_numeric( $args['selected'] ) && $args['selected'] !== 0 ) {
-			if ( ! in_array( $args['selected'], $options ) ) {
-				$options[ $args['selected'] ] = get_the_title( $args['selected'] );
+		} elseif (is_numeric($args['selected']) && $args['selected'] !== 0) {
+			if ( ! in_array($args['selected'], $options)) {
+				$options[$args['selected']] = get_the_title($args['selected']);
 			}
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $args['name'],
 			'selected'         => $args['selected'],
 			'id'               => $args['id'],
@@ -90,7 +90,7 @@ 
 			'placeholder'      => $args['placeholder'],
 			'show_option_all'  => false,
 			'show_option_none' => false
-		) );
+		));
 
 		return $output;
 	}
@@ -105,7 +105,7 @@ 
 	 *
 	 * @return string $output Donor dropdown
 	 */
-	public function donor_dropdown( $args = array() ) {
+	public function donor_dropdown($args = array()) {
 
 		$defaults = array(
 			'name'        => 'customers',
@@ -114,38 +114,38 @@ 
 			'multiple'    => false,
 			'selected'    => 0,
 			'chosen'      => true,
-			'placeholder' => __( 'Select a Donor', 'give' ),
+			'placeholder' => __('Select a Donor', 'give'),
 			'number'      => 30
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$customers = Give()->customers->get_customers( array(
+		$customers = Give()->customers->get_customers(array(
 			'number' => $args['number']
-		) );
+		));
 
 		$options = array();
 
-		if ( $customers ) {
-			$options[0] = __( 'No donor attached', 'give' );
-			foreach ( $customers as $customer ) {
-				$options[ absint( $customer->id ) ] = esc_html( $customer->name . ' (' . $customer->email . ')' );
+		if ($customers) {
+			$options[0] = __('No donor attached', 'give');
+			foreach ($customers as $customer) {
+				$options[absint($customer->id)] = esc_html($customer->name.' ('.$customer->email.')');
 			}
 		} else {
-			$options[0] = __( 'No donors found', 'give' );
+			$options[0] = __('No donors found', 'give');
 		}
 
-		if ( ! empty( $args['selected'] ) ) {
+		if ( ! empty($args['selected'])) {
 
 			// If a selected customer has been specified, we need to ensure it's in the initial list of customers displayed
 
-			if ( ! array_key_exists( $args['selected'], $options ) ) {
+			if ( ! array_key_exists($args['selected'], $options)) {
 
-				$customer = new Give_Customer( $args['selected'] );
+				$customer = new Give_Customer($args['selected']);
 
-				if ( $customer ) {
+				if ($customer) {
 
-					$options[ absint( $args['selected'] ) ] = esc_html( $customer->name . ' (' . $customer->email . ')' );
+					$options[absint($args['selected'])] = esc_html($customer->name.' ('.$customer->email.')');
 
 				}
 
@@ -153,17 +153,17 @@ 
 
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $args['name'],
 			'selected'         => $args['selected'],
 			'id'               => $args['id'],
-			'class'            => $args['class'] . ' give-customer-select',
+			'class'            => $args['class'].' give-customer-select',
 			'options'          => $options,
 			'multiple'         => $args['multiple'],
 			'chosen'           => $args['chosen'],
 			'show_option_all'  => false,
 			'show_option_none' => false
-		) );
+		));
 
 		return $output;
 	}
@@ -180,21 +180,21 @@ 
 	 *
 	 * @return string $output Category dropdown
 	 */
-	public function category_dropdown( $name = 'give_forms_categories', $selected = 0 ) {
-		$categories = get_terms( 'give_forms_category', apply_filters( 'give_forms_category_dropdown', array() ) );
+	public function category_dropdown($name = 'give_forms_categories', $selected = 0) {
+		$categories = get_terms('give_forms_category', apply_filters('give_forms_category_dropdown', array()));
 		$options    = array();
 
-		foreach ( $categories as $category ) {
-			$options[ absint( $category->term_id ) ] = esc_html( $category->name );
+		foreach ($categories as $category) {
+			$options[absint($category->term_id)] = esc_html($category->name);
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $name,
 			'selected'         => $selected,
 			'options'          => $options,
-			'show_option_all'  => __( 'All Categories', 'give' ),
+			'show_option_all'  => __('All Categories', 'give'),
 			'show_option_none' => false
-		) );
+		));
 
 		return $output;
 	}
@@ -212,25 +212,25 @@ 
 	 *
 	 * @return string $output Year dropdown
 	 */
-	public function year_dropdown( $name = 'year', $selected = 0, $years_before = 5, $years_after = 0 ) {
-		$current    = date( 'Y' );
-		$start_year = $current - absint( $years_before );
-		$end_year   = $current + absint( $years_after );
-		$selected   = empty( $selected ) ? date( 'Y' ) : $selected;
+	public function year_dropdown($name = 'year', $selected = 0, $years_before = 5, $years_after = 0) {
+		$current    = date('Y');
+		$start_year = $current - absint($years_before);
+		$end_year   = $current + absint($years_after);
+		$selected   = empty($selected) ? date('Y') : $selected;
 		$options    = array();
 
-		while ( $start_year <= $end_year ) {
-			$options[ absint( $start_year ) ] = $start_year;
-			$start_year ++;
+		while ($start_year <= $end_year) {
+			$options[absint($start_year)] = $start_year;
+			$start_year++;
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $name,
 			'selected'         => $selected,
 			'options'          => $options,
 			'show_option_all'  => false,
 			'show_option_none' => false
-		) );
+		));
 
 		return $output;
 	}
@@ -247,23 +247,23 @@ 
 	 *
 	 * @return string $output Month dropdown
 	 */
-	public function month_dropdown( $name = 'month', $selected = 0 ) {
+	public function month_dropdown($name = 'month', $selected = 0) {
 		$month    = 1;
 		$options  = array();
-		$selected = empty( $selected ) ? date( 'n' ) : $selected;
+		$selected = empty($selected) ? date('n') : $selected;
 
-		while ( $month <= 12 ) {
-			$options[ absint( $month ) ] = give_month_num_to_name( $month );
-			$month ++;
+		while ($month <= 12) {
+			$options[absint($month)] = give_month_num_to_name($month);
+			$month++;
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $name,
 			'selected'         => $selected,
 			'options'          => $options,
 			'show_option_all'  => false,
 			'show_option_none' => false
-		) );
+		));
 
 		return $output;
 	}
@@ -277,7 +277,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function select( $args = array() ) {
+	public function select($args = array()) {
 		$defaults = array(
 			'options'          => array(),
 			'name'             => null,
@@ -287,60 +287,60 @@ 
 			'chosen'           => false,
 			'placeholder'      => null,
 			'multiple'         => false,
-			'show_option_all'  => _x( 'All', 'all dropdown items', 'give' ),
-			'show_option_none' => _x( 'None', 'no dropdown items', 'give' )
+			'show_option_all'  => _x('All', 'all dropdown items', 'give'),
+			'show_option_none' => _x('None', 'no dropdown items', 'give')
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 
-		if ( $args['multiple'] ) {
+		if ($args['multiple']) {
 			$multiple = ' MULTIPLE';
 		} else {
 			$multiple = '';
 		}
 
-		if ( $args['chosen'] ) {
+		if ($args['chosen']) {
 			$args['class'] .= ' give-select-chosen';
 		}
 
-		if ( $args['placeholder'] ) {
+		if ($args['placeholder']) {
 			$placeholder = $args['placeholder'];
 		} else {
 			$placeholder = '';
 		}
 
-		$output = '<select name="' . esc_attr( $args['name'] ) . '" id="' . esc_attr( sanitize_key( str_replace( '-', '_', $args['id'] ) ) ) . '" class="give-select ' . esc_attr( $args['class'] ) . '"' . $multiple . ' data-placeholder="' . $placeholder . '">';
+		$output = '<select name="'.esc_attr($args['name']).'" id="'.esc_attr(sanitize_key(str_replace('-', '_', $args['id']))).'" class="give-select '.esc_attr($args['class']).'"'.$multiple.' data-placeholder="'.$placeholder.'">';
 
-		if ( $args['show_option_all'] ) {
-			if ( $args['multiple'] ) {
-				$selected = selected( true, in_array( 0, $args['selected'] ), false );
+		if ($args['show_option_all']) {
+			if ($args['multiple']) {
+				$selected = selected(true, in_array(0, $args['selected']), false);
 			} else {
-				$selected = selected( $args['selected'], 0, false );
+				$selected = selected($args['selected'], 0, false);
 			}
-			$output .= '<option value="all"' . $selected . '>' . esc_html( $args['show_option_all'] ) . '</option>';
+			$output .= '<option value="all"'.$selected.'>'.esc_html($args['show_option_all']).'</option>';
 		}
 
-		if ( ! empty( $args['options'] ) ) {
+		if ( ! empty($args['options'])) {
 
-			if ( $args['show_option_none'] ) {
-				if ( $args['multiple'] ) {
-					$selected = selected( true, in_array( - 1, $args['selected'] ), false );
+			if ($args['show_option_none']) {
+				if ($args['multiple']) {
+					$selected = selected(true, in_array( -1, $args['selected'] ), false);
 				} else {
-					$selected = selected( $args['selected'], - 1, false );
+					$selected = selected($args['selected'], - 1, false);
 				}
-				$output .= '<option value="-1"' . $selected . '>' . esc_html( $args['show_option_none'] ) . '</option>';
+				$output .= '<option value="-1"'.$selected.'>'.esc_html($args['show_option_none']).'</option>';
 			}
 
-			foreach ( $args['options'] as $key => $option ) {
+			foreach ($args['options'] as $key => $option) {
 
-				if ( $args['multiple'] && is_array( $args['selected'] ) ) {
-					$selected = selected( true, in_array( $key, $args['selected'] ), false );
+				if ($args['multiple'] && is_array($args['selected'])) {
+					$selected = selected(true, in_array($key, $args['selected']), false);
 				} else {
-					$selected = selected( $args['selected'], $key, false );
+					$selected = selected($args['selected'], $key, false);
 				}
 
-				$output .= '<option value="' . esc_attr( $key ) . '"' . $selected . '>' . esc_html( $option ) . '</option>';
+				$output .= '<option value="'.esc_attr($key).'"'.$selected.'>'.esc_html($option).'</option>';
 			}
 		}
 
@@ -358,7 +358,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function checkbox( $args = array() ) {
+	public function checkbox($args = array()) {
 		$defaults = array(
 			'name'    => null,
 			'current' => null,
@@ -369,16 +369,16 @@ 
 			)
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		$options = '';
-		if ( ! empty( $args['options']['disabled'] ) ) {
+		if ( ! empty($args['options']['disabled'])) {
 			$options .= ' disabled="disabled"';
-		} elseif ( ! empty( $args['options']['readonly'] ) ) {
+		} elseif ( ! empty($args['options']['readonly'])) {
 			$options .= ' readonly';
 		}
 
-		$output = '<input type="checkbox"' . $options . ' name="' . esc_attr( $args['name'] ) . '" id="' . esc_attr( $args['name'] ) . '" class="' . $args['class'] . ' ' . esc_attr( $args['name'] ) . '" ' . checked( 1, $args['current'], false ) . ' />';
+		$output = '<input type="checkbox"'.$options.' name="'.esc_attr($args['name']).'" id="'.esc_attr($args['name']).'" class="'.$args['class'].' '.esc_attr($args['name']).'" '.checked(1, $args['current'], false).' />';
 
 		return $output;
 	}
@@ -392,22 +392,22 @@ 
 	 *
 	 * @return string Text field
 	 */
-	public function text( $args = array() ) {
+	public function text($args = array()) {
 		// Backwards compatabliity
-		if ( func_num_args() > 1 ) {
+		if (func_num_args() > 1) {
 			$args = func_get_args();
 
 			$name  = $args[0];
-			$value = isset( $args[1] ) ? $args[1] : '';
-			$label = isset( $args[2] ) ? $args[2] : '';
-			$desc  = isset( $args[3] ) ? $args[3] : '';
+			$value = isset($args[1]) ? $args[1] : '';
+			$label = isset($args[2]) ? $args[2] : '';
+			$desc  = isset($args[3]) ? $args[3] : '';
 		}
 
 		$defaults = array(
-			'name'         => isset( $name ) ? $name : 'text',
-			'value'        => isset( $value ) ? $value : null,
-			'label'        => isset( $label ) ? $label : null,
-			'desc'         => isset( $desc ) ? $desc : null,
+			'name'         => isset($name) ? $name : 'text',
+			'value'        => isset($value) ? $value : null,
+			'label'        => isset($label) ? $label : null,
+			'desc'         => isset($desc) ? $desc : null,
 			'placeholder'  => '',
 			'class'        => 'regular-text',
 			'disabled'     => false,
@@ -415,29 +415,29 @@ 
 			'data'         => false
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		$disabled = '';
-		if ( $args['disabled'] ) {
+		if ($args['disabled']) {
 			$disabled = ' disabled="disabled"';
 		}
 
 		$data = '';
-		if ( ! empty( $args['data'] ) ) {
-			foreach ( $args['data'] as $key => $value ) {
-				$data .= 'data-' . $key . '="' . $value . '" ';
+		if ( ! empty($args['data'])) {
+			foreach ($args['data'] as $key => $value) {
+				$data .= 'data-'.$key.'="'.$value.'" ';
 			}
 		}
 
-		$output = '<span id="give-' . sanitize_key( $args['name'] ) . '-wrap">';
+		$output = '<span id="give-'.sanitize_key($args['name']).'-wrap">';
 
-		$output .= '<label class="give-label" for="give-' . sanitize_key( $args['name'] ) . '">' . esc_html( $args['label'] ) . '</label>';
+		$output .= '<label class="give-label" for="give-'.sanitize_key($args['name']).'">'.esc_html($args['label']).'</label>';
 
-		if ( ! empty( $args['desc'] ) ) {
-			$output .= '<span class="give-description">' . esc_html( $args['desc'] ) . '</span>';
+		if ( ! empty($args['desc'])) {
+			$output .= '<span class="give-description">'.esc_html($args['desc']).'</span>';
 		}
 
-		$output .= '<input type="text" name="' . esc_attr( $args['name'] ) . '" id="' . esc_attr( $args['name'] ) . '" autocomplete="' . esc_attr( $args['autocomplete'] ) . '" value="' . esc_attr( $args['value'] ) . '" placeholder="' . esc_attr( $args['placeholder'] ) . '" class="' . $args['class'] . '" ' . $data . '' . $disabled . '/>';
+		$output .= '<input type="text" name="'.esc_attr($args['name']).'" id="'.esc_attr($args['name']).'" autocomplete="'.esc_attr($args['autocomplete']).'" value="'.esc_attr($args['value']).'" placeholder="'.esc_attr($args['placeholder']).'" class="'.$args['class'].'" '.$data.''.$disabled.'/>';
 
 		$output .= '</span>';
 
@@ -453,7 +453,7 @@ 
 	 *
 	 * @return string textarea
 	 */
-	public function textarea( $args = array() ) {
+	public function textarea($args = array()) {
 		$defaults = array(
 			'name'     => 'textarea',
 			'value'    => null,
@@ -463,21 +463,21 @@ 
 			'disabled' => false
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		$disabled = '';
-		if ( $args['disabled'] ) {
+		if ($args['disabled']) {
 			$disabled = ' disabled="disabled"';
 		}
 
-		$output = '<span id="give-' . sanitize_key( $args['name'] ) . '-wrap">';
+		$output = '<span id="give-'.sanitize_key($args['name']).'-wrap">';
 
-		$output .= '<label class="give-label" for="give-' . sanitize_key( $args['name'] ) . '">' . esc_html( $args['label'] ) . '</label>';
+		$output .= '<label class="give-label" for="give-'.sanitize_key($args['name']).'">'.esc_html($args['label']).'</label>';
 
-		$output .= '<textarea name="' . esc_attr( $args['name'] ) . '" id="' . esc_attr( $args['name'] ) . '" class="' . $args['class'] . '"' . $disabled . '>' . esc_attr( $args['value'] ) . '</textarea>';
+		$output .= '<textarea name="'.esc_attr($args['name']).'" id="'.esc_attr($args['name']).'" class="'.$args['class'].'"'.$disabled.'>'.esc_attr($args['value']).'</textarea>';
 
-		if ( ! empty( $args['desc'] ) ) {
-			$output .= '<span class="give-description">' . esc_html( $args['desc'] ) . '</span>';
+		if ( ! empty($args['desc'])) {
+			$output .= '<span class="give-description">'.esc_html($args['desc']).'</span>';
 		}
 
 		$output .= '</span>';
@@ -494,12 +494,12 @@ 
 	 *
 	 * @return string text field with ajax search
 	 */
-	public function ajax_user_search( $args = array() ) {
+	public function ajax_user_search($args = array()) {
 
 		$defaults = array(
 			'name'         => 'user_id',
 			'value'        => null,
-			'placeholder'  => __( 'Enter username', 'give' ),
+			'placeholder'  => __('Enter username', 'give'),
 			'label'        => null,
 			'desc'         => null,
 			'class'        => '',
@@ -508,13 +508,13 @@ 
 			'data'         => false
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$args['class'] = 'give-ajax-user-search ' . $args['class'];
+		$args['class'] = 'give-ajax-user-search '.$args['class'];
 
 		$output = '<span class="give_user_search_wrap">';
-		$output .= $this->text( $args );
-		$output .= '<span class="give_user_search_results hidden"><a class="give-ajax-user-cancel" title="' . __( 'Cancel', 'give' ) . '" aria-label="' . __( 'Cancel', 'give' ) . '" href="#">x</a><span></span></span>';
+		$output .= $this->text($args);
+		$output .= '<span class="give_user_search_results hidden"><a class="give-ajax-user-cancel" title="'.__('Cancel', 'give').'" aria-label="'.__('Cancel', 'give').'" href="#">x</a><span></span></span>';
 		$output .= '</span>';
 
 		return $output;

includes/class-give-license-handler.php

Indentation +6 added lines, -6 removed lines

@@ -1,11 +1,11 @@
 <?php
 /**
- * Give License handler
- *
- * This class simplifies the process of adding license information to new Give add-ons.
- *
- * @version 1.0
- */
+	 * Give License handler
+	 *
+	 * This class simplifies the process of adding license information to new Give add-ons.
+	 *
+	 * @version 1.0
+	 */
 
 // Exit if accessed directly
 if ( ! defined( 'ABSPATH' ) ) {

Spacing +63 added lines, -63 removed lines

@@ -8,11 +8,11 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give_License' ) ) :
+if ( ! class_exists('Give_License')) :
 
 	/**
 	 * Give_License Class
@@ -38,16 +38,16 @@ 
 		 * @param string  $_optname
 		 * @param string  $_api_url
 		 */
-		public function __construct( $_file, $_item_name, $_version, $_author, $_optname = null, $_api_url = null ) {
+		public function __construct($_file, $_item_name, $_version, $_author, $_optname = null, $_api_url = null) {
 			global $give_options;
 
 			$this->file           = $_file;
 			$this->item_name      = $_item_name;
-			$this->item_shortname = 'give_' . preg_replace( '/[^a-zA-Z0-9_\s]/', '', str_replace( ' ', '_', strtolower( $this->item_name ) ) );
+			$this->item_shortname = 'give_'.preg_replace('/[^a-zA-Z0-9_\s]/', '', str_replace(' ', '_', strtolower($this->item_name)));
 			$this->version        = $_version;
-			$this->license        = isset( $give_options[ $this->item_shortname . '_license_key' ] ) ? trim( $give_options[ $this->item_shortname . '_license_key' ] ) : '';
+			$this->license        = isset($give_options[$this->item_shortname.'_license_key']) ? trim($give_options[$this->item_shortname.'_license_key']) : '';
 			$this->author         = $_author;
-			$this->api_url        = is_null( $_api_url ) ? $this->api_url : $_api_url;
+			$this->api_url        = is_null($_api_url) ? $this->api_url : $_api_url;
 
 
 			// Setup hooks
@@ -63,7 +63,7 @@ 
 		 * @return  void
 		 */
 		private function includes() {
-			if ( ! class_exists( 'EDD_SL_Plugin_Updater' ) ) {
+			if ( ! class_exists('EDD_SL_Plugin_Updater')) {
 				require_once 'admin/EDD_SL_Plugin_Updater.php';
 			}
 		}
@@ -77,18 +77,18 @@ 
 		private function hooks() {
 
 			// Register settings
-			add_filter( 'give_settings_licenses', array( $this, 'settings' ), 1 );
+			add_filter('give_settings_licenses', array($this, 'settings'), 1);
 
 			// Activate license key on settings save
-			add_action( 'admin_init', array( $this, 'activate_license' ) );
+			add_action('admin_init', array($this, 'activate_license'));
 
 			// Deactivate license key
-			add_action( 'admin_init', array( $this, 'deactivate_license' ) );
+			add_action('admin_init', array($this, 'deactivate_license'));
 
 			// Updater
-			add_action( 'admin_init', array( $this, 'auto_updater' ), 0 );
+			add_action('admin_init', array($this, 'auto_updater'), 0);
 
-			add_action( 'admin_notices', array( $this, 'notices' ) );
+			add_action('admin_notices', array($this, 'notices'));
 		}
 
 		/**
@@ -100,7 +100,7 @@ 
 		 */
 		public function auto_updater() {
 
-			if ( 'valid' !== get_option( $this->item_shortname . '_license_active' ) ) {
+			if ('valid' !== get_option($this->item_shortname.'_license_active')) {
 				return;
 			}
 
@@ -127,20 +127,20 @@ 
 		 *
 		 * @return  array
 		 */
-		public function settings( $settings ) {
+		public function settings($settings) {
 
 			$give_license_settings = array(
 				array(
-					'name'    => sprintf( __( '%1$s', 'give' ), $this->item_name ),
-					'id'      => $this->item_shortname . '_license_key',
+					'name'    => sprintf(__('%1$s', 'give'), $this->item_name),
+					'id'      => $this->item_shortname.'_license_key',
 					'desc'    => '',
 					'type'    => 'license_key',
-					'options' => array( 'is_valid_license_option' => $this->item_shortname . '_license_active' ),
+					'options' => array('is_valid_license_option' => $this->item_shortname.'_license_active'),
 					'size'    => 'regular'
 				)
 			);
 
-			return array_merge( $settings, $give_license_settings );
+			return array_merge($settings, $give_license_settings);
 		}
 
 		/**
@@ -152,18 +152,18 @@ 
 		 *
 		 * @return  array
 		 */
-		public function license_settings_content( $settings ) {
+		public function license_settings_content($settings) {
 
 			$give_license_settings = array(
 				array(
-					'name' => __( 'Add-on Licenses', 'give' ),
+					'name' => __('Add-on Licenses', 'give'),
 					'desc' => '<hr>',
 					'type' => 'give_title',
 					'id'   => 'give_title'
 				),
 			);
 
-			return array_merge( $settings, $give_license_settings );
+			return array_merge($settings, $give_license_settings);
 		}
 
 
@@ -175,34 +175,34 @@ 
 		 */
 		public function activate_license() {
 
-			if ( ! isset( $_POST[ $this->item_shortname . '_license_key' ] ) ) {
+			if ( ! isset($_POST[$this->item_shortname.'_license_key'])) {
 				return;
 			}
 
-			foreach ( $_POST as $key => $value ) {
-				if ( false !== strpos( $key, 'license_key_deactivate' ) ) {
+			foreach ($_POST as $key => $value) {
+				if (false !== strpos($key, 'license_key_deactivate')) {
 					// Don't activate a key when deactivating a different key
 					return;
 				}
 			}
 
-			if ( ! wp_verify_nonce( $_REQUEST[ $this->item_shortname . '_license_key-nonce' ], $this->item_shortname . '_license_key-nonce' ) ) {
+			if ( ! wp_verify_nonce($_REQUEST[$this->item_shortname.'_license_key-nonce'], $this->item_shortname.'_license_key-nonce')) {
 
-				wp_die( __( 'Nonce verification failed', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+				wp_die(__('Nonce verification failed', 'give'), __('Error', 'give'), array('response' => 403));
 
 			}
 
-			if ( ! current_user_can( 'manage_give_settings' ) ) {
+			if ( ! current_user_can('manage_give_settings')) {
 				return;
 			}
 
-			if ( 'valid' === get_option( $this->item_shortname . '_license_active' ) ) {
+			if ('valid' === get_option($this->item_shortname.'_license_active')) {
 				return;
 			}
 
-			$license = sanitize_text_field( $_POST[ $this->item_shortname . '_license_key' ] );
+			$license = sanitize_text_field($_POST[$this->item_shortname.'_license_key']);
 
-			if ( empty( $license ) ) {
+			if (empty($license)) {
 				return;
 			}
 
@@ -210,7 +210,7 @@ 
 			$api_params = array(
 				'edd_action' => 'activate_license', //never change from "edd_" to "give_"!
 				'license'    => $license,
-				'item_name'  => urlencode( $this->item_name ),
+				'item_name'  => urlencode($this->item_name),
 				'url'        => home_url()
 			);
 
@@ -225,22 +225,22 @@ 
 			);
 
 			// Make sure there are no errors
-			if ( is_wp_error( $response ) ) {
+			if (is_wp_error($response)) {
 				return;
 			}
 
 			// Tell WordPress to look for updates
-			set_site_transient( 'update_plugins', null );
+			set_site_transient('update_plugins', null);
 
 			// Decode license data
-			$license_data = json_decode( wp_remote_retrieve_body( $response ) );
+			$license_data = json_decode(wp_remote_retrieve_body($response));
 
-			update_option( $this->item_shortname . '_license_active', $license_data->license );
+			update_option($this->item_shortname.'_license_active', $license_data->license);
 
-			if ( ! (bool) $license_data->success ) {
-				set_transient( 'give_license_error', $license_data, 1000 );
+			if ( ! (bool) $license_data->success) {
+				set_transient('give_license_error', $license_data, 1000);
 			} else {
-				delete_transient( 'give_license_error' );
+				delete_transient('give_license_error');
 			}
 		}
 
@@ -253,28 +253,28 @@ 
 		 */
 		public function deactivate_license() {
 
-			if ( ! isset( $_POST[ $this->item_shortname . '_license_key' ] ) ) {
+			if ( ! isset($_POST[$this->item_shortname.'_license_key'])) {
 				return;
 			}
 
-			if ( ! wp_verify_nonce( $_REQUEST[ $this->item_shortname . '_license_key-nonce' ], $this->item_shortname . '_license_key-nonce' ) ) {
+			if ( ! wp_verify_nonce($_REQUEST[$this->item_shortname.'_license_key-nonce'], $this->item_shortname.'_license_key-nonce')) {
 
-				wp_die( __( 'Nonce verification failed', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+				wp_die(__('Nonce verification failed', 'give'), __('Error', 'give'), array('response' => 403));
 
 			}
 
-			if ( ! current_user_can( 'manage_give_settings' ) ) {
+			if ( ! current_user_can('manage_give_settings')) {
 				return;
 			}
 
 			// Run on deactivate button press
-			if ( isset( $_POST[ $this->item_shortname . '_license_key_deactivate' ] ) ) {
+			if (isset($_POST[$this->item_shortname.'_license_key_deactivate'])) {
 
 				// Data to send to the API
 				$api_params = array(
 					'edd_action' => 'deactivate_license', //never change from "edd_" to "give_"!
 					'license'    => $this->license,
-					'item_name'  => urlencode( $this->item_name ),
+					'item_name'  => urlencode($this->item_name),
 					'url'        => home_url()
 				);
 
@@ -290,19 +290,19 @@ 
 
 
 				// Make sure there are no errors
-				if ( is_wp_error( $response ) ) {
+				if (is_wp_error($response)) {
 					return;
 				}
 
 				// Decode the license data
-				$license_data = json_decode( wp_remote_retrieve_body( $response ) );
+				$license_data = json_decode(wp_remote_retrieve_body($response));
 
-				delete_option( $this->item_shortname . '_license_active' );
+				delete_option($this->item_shortname.'_license_active');
 
-				if ( ! (bool) $license_data->success ) {
-					set_transient( 'give_license_error', $license_data, 1000 );
+				if ( ! (bool) $license_data->success) {
+					set_transient('give_license_error', $license_data, 1000);
 				} else {
-					delete_transient( 'give_license_error' );
+					delete_transient('give_license_error');
 				}
 			}
 		}
@@ -316,57 +316,57 @@ 
 		 */
 		public function notices() {
 
-			if ( ! isset( $_GET['page'] ) || 'give-settings' !== $_GET['page'] ) {
+			if ( ! isset($_GET['page']) || 'give-settings' !== $_GET['page']) {
 				return;
 			}
 
-			if ( ! isset( $_GET['tab'] ) || 'licenses' !== $_GET['tab'] ) {
+			if ( ! isset($_GET['tab']) || 'licenses' !== $_GET['tab']) {
 				return;
 			}
 
-			$license_error = get_transient( 'give_license_error' );
+			$license_error = get_transient('give_license_error');
 
-			if ( false === $license_error ) {
+			if (false === $license_error) {
 				return;
 			}
 
-			if ( ! empty( $license_error->error ) ) {
+			if ( ! empty($license_error->error)) {
 
-				switch ( $license_error->error ) {
+				switch ($license_error->error) {
 
 					case 'item_name_mismatch' :
 
-						$message = __( 'This license does not belong to the product you have entered it for.', 'give' );
+						$message = __('This license does not belong to the product you have entered it for.', 'give');
 						break;
 
 					case 'no_activations_left' :
 
-						$message = __( 'This license does not have any activations left', 'give' );
+						$message = __('This license does not have any activations left', 'give');
 						break;
 
 					case 'expired' :
 
-						$message = __( 'This license key is expired. Please renew it.', 'give' );
+						$message = __('This license key is expired. Please renew it.', 'give');
 						break;
 
 					default :
 
-						$message = sprintf( __( 'There was a problem activating your license key, please try again or contact support. Error code: %s', 'give' ), $license_error->error );
+						$message = sprintf(__('There was a problem activating your license key, please try again or contact support. Error code: %s', 'give'), $license_error->error);
 						break;
 
 				}
 
 			}
 
-			if ( ! empty( $message ) ) {
+			if ( ! empty($message)) {
 
 				echo '<div class="error">';
-				echo '<p>' . $message . '</p>';
+				echo '<p>'.$message.'</p>';
 				echo '</div>';
 
 			}
 
-			delete_transient( 'give_license_error' );
+			delete_transient('give_license_error');
 
 		}
 	}

includes/class-give-roles.php

Indentation +10 added lines, -10 removed lines

@@ -1,12 +1,12 @@ 
 <?php
 /**
- * Roles and Capabilities
- *
- * @package     Give
- * @subpackage  Classes/Roles
- * @copyright   Copyright (c) 2015, WordImpress
- * @license     http://opensource.org/licenses/gpl-2.0.php GNU Public License
- * @since       1.0
+	 * Roles and Capabilities
+	 *
+	 * @package     Give
+	 * @subpackage  Classes/Roles
+	 * @copyright   Copyright (c) 2015, WordImpress
+	 * @license     http://opensource.org/licenses/gpl-2.0.php GNU Public License
+	 * @since       1.0
 */
 
 /**
@@ -71,9 +71,9 @@ 
 		) );
 
 		add_role( 'give_accountant', __( 'Give Accountant', 'give' ), array(
-		    'read'                   => true,
-		    'edit_posts'             => false,
-		    'delete_posts'           => false
+			'read'                   => true,
+			'edit_posts'             => false,
+			'delete_posts'           => false
 		) );
 
 		add_role( 'give_worker', __( 'Give Worker', 'give' ), array(

Spacing +57 added lines, -57 removed lines

@@ -28,7 +28,7 @@ 
 	 */
 	public function __construct() {
 
-		add_filter( 'give_map_meta_cap', array( $this, 'meta_caps' ), 10, 4 );
+		add_filter('give_map_meta_cap', array($this, 'meta_caps'), 10, 4);
 	}
 
 	/**
@@ -39,7 +39,7 @@ 
 	 * @return void
 	 */
 	public function add_roles() {
-		add_role( 'give_manager', __( 'Give Manager', 'give' ), array(
+		add_role('give_manager', __('Give Manager', 'give'), array(
 			'read'                   => true,
 			'edit_posts'             => true,
 			'delete_posts'           => true,
@@ -68,20 +68,20 @@ 
 			'publish_posts'          => true,
 			'read_private_pages'     => true,
 			'read_private_posts'     => true
-		) );
+		));
 
-		add_role( 'give_accountant', __( 'Give Accountant', 'give' ), array(
+		add_role('give_accountant', __('Give Accountant', 'give'), array(
 		    'read'                   => true,
 		    'edit_posts'             => false,
 		    'delete_posts'           => false
-		) );
+		));
 
-		add_role( 'give_worker', __( 'Give Worker', 'give' ), array(
+		add_role('give_worker', __('Give Worker', 'give'), array(
 			'read'                   => true,
 			'edit_posts'             => false,
 			'upload_files'           => true,
 			'delete_posts'           => false
-		) );
+		));
 
 	}
 
@@ -96,38 +96,38 @@ 
 	public function add_caps() {
 		global $wp_roles;
 
-		if ( class_exists('WP_Roles') ) {
-			if ( ! isset( $wp_roles ) ) {
+		if (class_exists('WP_Roles')) {
+			if ( ! isset($wp_roles)) {
 				$wp_roles = new WP_Roles();
 			}
 		}
 
-		if ( is_object( $wp_roles ) ) {
-			$wp_roles->add_cap( 'give_manager', 'view_give_reports' );
-			$wp_roles->add_cap( 'give_manager', 'view_give_sensitive_data' );
-			$wp_roles->add_cap( 'give_manager', 'export_give_reports' );
-			$wp_roles->add_cap( 'give_manager', 'manage_give_settings' );
+		if (is_object($wp_roles)) {
+			$wp_roles->add_cap('give_manager', 'view_give_reports');
+			$wp_roles->add_cap('give_manager', 'view_give_sensitive_data');
+			$wp_roles->add_cap('give_manager', 'export_give_reports');
+			$wp_roles->add_cap('give_manager', 'manage_give_settings');
 
-			$wp_roles->add_cap( 'administrator', 'view_give_reports' );
-			$wp_roles->add_cap( 'administrator', 'view_give_sensitive_data' );
-			$wp_roles->add_cap( 'administrator', 'export_give_reports' );
-			$wp_roles->add_cap( 'administrator', 'manage_give_settings' );
+			$wp_roles->add_cap('administrator', 'view_give_reports');
+			$wp_roles->add_cap('administrator', 'view_give_sensitive_data');
+			$wp_roles->add_cap('administrator', 'export_give_reports');
+			$wp_roles->add_cap('administrator', 'manage_give_settings');
 
 			// Add the main post type capabilities
 			$capabilities = $this->get_core_caps();
-			foreach ( $capabilities as $cap_group ) {
-				foreach ( $cap_group as $cap ) {
-					$wp_roles->add_cap( 'give_manager', $cap );
-					$wp_roles->add_cap( 'administrator', $cap );
-					$wp_roles->add_cap( 'give_worker', $cap );
+			foreach ($capabilities as $cap_group) {
+				foreach ($cap_group as $cap) {
+					$wp_roles->add_cap('give_manager', $cap);
+					$wp_roles->add_cap('administrator', $cap);
+					$wp_roles->add_cap('give_worker', $cap);
 				}
 			}
 
-			$wp_roles->add_cap( 'give_accountant', 'edit_give_forms' );
-			$wp_roles->add_cap( 'give_accountant', 'read_private_forms' );
-			$wp_roles->add_cap( 'give_accountant', 'view_give_reports' );
-			$wp_roles->add_cap( 'give_accountant', 'export_give_reports' );
-			$wp_roles->add_cap( 'give_accountant', 'edit_give_payments' );
+			$wp_roles->add_cap('give_accountant', 'edit_give_forms');
+			$wp_roles->add_cap('give_accountant', 'read_private_forms');
+			$wp_roles->add_cap('give_accountant', 'view_give_reports');
+			$wp_roles->add_cap('give_accountant', 'export_give_reports');
+			$wp_roles->add_cap('give_accountant', 'edit_give_payments');
 
 		}
 	}
@@ -142,10 +142,10 @@ 
 	public function get_core_caps() {
 		$capabilities = array();
 
-		$capability_types = array( 'give_forms', 'give_campaigns', 'give_payments' );
+		$capability_types = array('give_forms', 'give_campaigns', 'give_payments');
 
-		foreach ( $capability_types as $capability_type ) {
-			$capabilities[ $capability_type ] = array(
+		foreach ($capability_types as $capability_type) {
+			$capabilities[$capability_type] = array(
 				// Post type
 				"edit_{$capability_type}",
 				"read_{$capability_type}",
@@ -182,22 +182,22 @@ 
 	 * @since  2.0
 	 * @return array $caps
 	 */
-	public function meta_caps( $caps, $cap, $user_id, $args ) {
+	public function meta_caps($caps, $cap, $user_id, $args) {
 
-		switch( $cap ) {
+		switch ($cap) {
 
 			case 'view_give_forms_stats' :
 				
-				if( empty( $args[0] ) ) {
+				if (empty($args[0])) {
 					break;
 				}
 				
-				$form = get_post( $args[0] );
-				if ( empty( $form ) ) {
+				$form = get_post($args[0]);
+				if (empty($form)) {
 					break;
 				}
 
-				if( user_can( $user_id, 'view_give_reports' ) || $user_id == $form->post_author ) {
+				if (user_can($user_id, 'view_give_reports') || $user_id == $form->post_author) {
 					$caps = array();
 				}
 
@@ -219,41 +219,41 @@ 
 		
 		global $wp_roles;
 
-		if ( class_exists( 'WP_Roles' ) ) {
-			if ( ! isset( $wp_roles ) ) {
+		if (class_exists('WP_Roles')) {
+			if ( ! isset($wp_roles)) {
 				$wp_roles = new WP_Roles();
 			}
 		}
 
-		if ( is_object( $wp_roles ) ) {
+		if (is_object($wp_roles)) {
 			/** Give Manager Capabilities */
-			$wp_roles->remove_cap( 'give_manager', 'view_give_reports' );
-			$wp_roles->remove_cap( 'give_manager', 'view_give_sensitive_data' );
-			$wp_roles->remove_cap( 'give_manager', 'export_give_reports' );
-			$wp_roles->remove_cap( 'give_manager', 'manage_give_settings' );
+			$wp_roles->remove_cap('give_manager', 'view_give_reports');
+			$wp_roles->remove_cap('give_manager', 'view_give_sensitive_data');
+			$wp_roles->remove_cap('give_manager', 'export_give_reports');
+			$wp_roles->remove_cap('give_manager', 'manage_give_settings');
 
 			/** Site Administrator Capabilities */
-			$wp_roles->remove_cap( 'administrator', 'view_give_reports' );
-			$wp_roles->remove_cap( 'administrator', 'view_give_sensitive_data' );
-			$wp_roles->remove_cap( 'administrator', 'export_give_reports' );
-			$wp_roles->remove_cap( 'administrator', 'manage_give_settings' );
+			$wp_roles->remove_cap('administrator', 'view_give_reports');
+			$wp_roles->remove_cap('administrator', 'view_give_sensitive_data');
+			$wp_roles->remove_cap('administrator', 'export_give_reports');
+			$wp_roles->remove_cap('administrator', 'manage_give_settings');
 
 			/** Remove the Main Post Type Capabilities */
 			$capabilities = $this->get_core_caps();
 
-			foreach ( $capabilities as $cap_group ) {
-				foreach ( $cap_group as $cap ) {
-					$wp_roles->remove_cap( 'give_manager', $cap );
-					$wp_roles->remove_cap( 'administrator', $cap );
-					$wp_roles->remove_cap( 'give_worker', $cap );
+			foreach ($capabilities as $cap_group) {
+				foreach ($cap_group as $cap) {
+					$wp_roles->remove_cap('give_manager', $cap);
+					$wp_roles->remove_cap('administrator', $cap);
+					$wp_roles->remove_cap('give_worker', $cap);
 				}
 			}
 
 			/** Shop Accountant Capabilities */
-			$wp_roles->remove_cap( 'give_accountant', 'edit_give_forms' );
-			$wp_roles->remove_cap( 'give_accountant', 'read_private_forms' );
-			$wp_roles->remove_cap( 'give_accountant', 'view_give_reports' );
-			$wp_roles->remove_cap( 'give_accountant', 'export_give_reports' );
+			$wp_roles->remove_cap('give_accountant', 'edit_give_forms');
+			$wp_roles->remove_cap('give_accountant', 'read_private_forms');
+			$wp_roles->remove_cap('give_accountant', 'view_give_reports');
+			$wp_roles->remove_cap('give_accountant', 'export_give_reports');
 
 		}
 	}

includes/class-give-template-loader.php

Spacing +24 added lines, -24 removed lines

@@ -13,7 +13,7 @@ 
 

 	public function __construct() {

 

-		add_filter( 'template_include', array( __CLASS__, 'template_loader' ) );

+		add_filter('template_include', array(__CLASS__, 'template_loader'));

 

 

 		/**

@@ -28,19 +28,19 @@ 
 		 * @see 3give_output_content_wrapper()

 		 * @see give_output_content_wrapper_end()

 		 */

-		add_action( 'give_before_main_content', 'give_output_content_wrapper', 10 );

-		add_action( 'give_after_main_content', 'give_output_content_wrapper_end', 10 );

+		add_action('give_before_main_content', 'give_output_content_wrapper', 10);

+		add_action('give_after_main_content', 'give_output_content_wrapper_end', 10);

 

 		/**

 		 * Entry Summary Classes

 		 */

-		add_filter( 'give_forms_single_summary_classes', array( $this, 'give_set_single_summary_classes' ) );

+		add_filter('give_forms_single_summary_classes', array($this, 'give_set_single_summary_classes'));

 

 

 		/**

 		 * Sidebar

 		 */

-		add_action( 'give_before_single_form_summary', array( $this, 'give_output_sidebar_option' ), 1 );

+		add_action('give_before_single_form_summary', array($this, 'give_output_sidebar_option'), 1);

 

 

 		/**

@@ -48,8 +48,8 @@ 
 		 *

 		 * @see give_template_single_title()

 		 */

-		add_action( 'give_single_form_summary', 'give_template_single_title', 5 );

-		add_action( 'give_single_form_summary', 'give_get_donation_form', 10 );

+		add_action('give_single_form_summary', 'give_template_single_title', 5);

+		add_action('give_single_form_summary', 'give_get_donation_form', 10);

 

 

 	}

@@ -64,12 +64,12 @@ 
 	 *

 	 * @return string

 	 */

-	public function give_set_single_summary_classes( $classes ) {

+	public function give_set_single_summary_classes($classes) {

 

-		$sidebar_option = give_get_option( 'disable_form_sidebar' );

+		$sidebar_option = give_get_option('disable_form_sidebar');

 

 		//Add full width class when feature image is disabled AND no widgets are present

-		if ( $sidebar_option == 'on' ) {

+		if ($sidebar_option == 'on') {

 			$classes .= ' give-full-width';

 		}

 

@@ -86,13 +86,13 @@ 
 	 *

 	 */

 	public function give_output_sidebar_option() {

-		$sidebar_option = give_get_option( 'disable_form_sidebar' );

+		$sidebar_option = give_get_option('disable_form_sidebar');

 		//Add full width class when feature image is disabled AND no widgets are present

-		if ( $sidebar_option !== 'on' ) {

-			add_action( 'give_before_single_form_summary', 'give_left_sidebar_pre_wrap', 5 );

-			add_action( 'give_before_single_form_summary', 'give_show_form_images', 10 );

-			add_action( 'give_before_single_form_summary', 'give_get_forms_sidebar', 20 );

-			add_action( 'give_before_single_form_summary', 'give_left_sidebar_post_wrap', 30 );

+		if ($sidebar_option !== 'on') {

+			add_action('give_before_single_form_summary', 'give_left_sidebar_pre_wrap', 5);

+			add_action('give_before_single_form_summary', 'give_show_form_images', 10);

+			add_action('give_before_single_form_summary', 'give_get_forms_sidebar', 20);

+			add_action('give_before_single_form_summary', 'give_left_sidebar_post_wrap', 30);

 		}

 

 	}

@@ -114,22 +114,22 @@ 
 	 *

 	 * @return string

 	 */

-	public static function template_loader( $template ) {

-		$find = array( 'give.php' );

+	public static function template_loader($template) {

+		$find = array('give.php');

 		$file = '';

 

-		if ( is_single() && get_post_type() == 'give_forms' ) {

+		if (is_single() && get_post_type() == 'give_forms') {

 

 			$file   = 'single-give-form.php';

 			$find[] = $file;

-			$find[] = apply_filters( 'give_template_path', 'give/' ) . $file;

+			$find[] = apply_filters('give_template_path', 'give/').$file;

 

 		}

 

-		if ( $file ) {

-			$template = locate_template( array_unique( $find ) );

-			if ( ! $template ) {

-				$template = GIVE_PLUGIN_DIR . '/templates/' . $file;

+		if ($file) {

+			$template = locate_template(array_unique($find));

+			if ( ! $template) {

+				$template = GIVE_PLUGIN_DIR.'/templates/'.$file;

 			}

 		}

 

includes/country-functions.php

Spacing +76 added lines, -76 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,9 +23,9 @@ 
  */
 function give_get_country() {
 	global $give_options;
-	$country = isset( $give_options['base_country'] ) ? $give_options['base_country'] : 'US';
+	$country = isset($give_options['base_country']) ? $give_options['base_country'] : 'US';
 
-	return apply_filters( 'give_give_country', $country );
+	return apply_filters('give_give_country', $country);
 }
 
 /**
@@ -36,9 +36,9 @@ 
  */
 function give_get_state() {
 	global $give_options;
-	$state = isset( $give_options['base_state'] ) ? $give_options['base_state'] : false;
+	$state = isset($give_options['base_state']) ? $give_options['base_state'] : false;
 
-	return apply_filters( 'give_give_state', $state );
+	return apply_filters('give_give_state', $state);
 }
 
 /**
@@ -50,14 +50,14 @@ 
  *
  * @return mixed|void  A list of states for the shop's base country
  */
-function give_get_states( $country = null ) {
+function give_get_states($country = null) {
 	global $give_options;
 
-	if ( empty( $country ) ) {
+	if (empty($country)) {
 		$country = give_get_country();
 	}
 
-	switch ( $country ) :
+	switch ($country) :
 
 		case 'US' :
 			$states = give_get_states_list();
@@ -107,7 +107,7 @@ 
 
 	endswitch;
 
-	return apply_filters( 'give_give_states', $states );
+	return apply_filters('give_give_states', $states);
 }
 
 
@@ -366,7 +366,7 @@ 
 		'ZW' => 'Zimbabwe'
 	);
 
-	return apply_filters( 'give_countries', $countries );
+	return apply_filters('give_countries', $countries);
 }
 
 /**
@@ -447,7 +447,7 @@ 
 		'AP' => 'Armed Forces - Pacific'
 	);
 
-	return apply_filters( 'give_us_states', $states );
+	return apply_filters('give_us_states', $states);
 }
 
 /**
@@ -475,7 +475,7 @@ 
 		'YT' => 'Yukon'
 	);
 
-	return apply_filters( 'give_canada_provinces', $provinces );
+	return apply_filters('give_canada_provinces', $provinces);
 }
 
 /**
@@ -497,7 +497,7 @@ 
 		'WA'  => 'Western Australia'
 	);
 
-	return apply_filters( 'give_australian_states', $states );
+	return apply_filters('give_australian_states', $states);
 }
 
 /**
@@ -538,7 +538,7 @@ 
 		'TO' => 'Tocantins'
 	);
 
-	return apply_filters( 'give_brazil_states', $states );
+	return apply_filters('give_brazil_states', $states);
 }
 
 /**
@@ -555,7 +555,7 @@ 
 		'NEW TERRITORIES' => 'New Territories'
 	);
 
-	return apply_filters( 'give_hong_kong_states', $states );
+	return apply_filters('give_hong_kong_states', $states);
 }
 
 /**
@@ -589,7 +589,7 @@ 
 		'ZA' => 'Zala'
 	);
 
-	return apply_filters( 'give_hungary_states', $states );
+	return apply_filters('give_hungary_states', $states);
 }
 
 /**
@@ -635,7 +635,7 @@ 
 		'CN32' => 'Xinjiang / 新疆'
 	);
 
-	return apply_filters( 'give_chinese_states', $states );
+	return apply_filters('give_chinese_states', $states);
 }
 
 /**
@@ -664,7 +664,7 @@ 
 		'WC' => 'West Coast'
 	);
 
-	return apply_filters( 'give_new_zealand_states', $states );
+	return apply_filters('give_new_zealand_states', $states);
 }
 
 /**
@@ -712,7 +712,7 @@ 
 		'PB' => 'Papua Barat'
 	);
 
-	return apply_filters( 'give_indonesia_states', $states );
+	return apply_filters('give_indonesia_states', $states);
 }
 
 /**
@@ -762,7 +762,7 @@ 
 		'PY' => 'Pondicherry (Puducherry)'
 	);
 
-	return apply_filters( 'give_indian_states', $states );
+	return apply_filters('give_indian_states', $states);
 }
 
 /**
@@ -792,7 +792,7 @@ 
 		'PJY' => 'W.P. Putrajaya'
 	);
 
-	return apply_filters( 'give_malaysian_states', $states );
+	return apply_filters('give_malaysian_states', $states);
 }
 
 /**
@@ -815,7 +815,7 @@ 
 		'WC'  => 'Western Cape'
 	);
 
-	return apply_filters( 'give_south_african_states', $states );
+	return apply_filters('give_south_african_states', $states);
 }
 
 /**
@@ -906,7 +906,7 @@ 
 		'TH-35' => 'Yasothon (ยโสธร)'
 	);
 
-	return apply_filters( 'give_thailand_states', $states );
+	return apply_filters('give_thailand_states', $states);
 }
 
 /**
@@ -918,59 +918,59 @@ 
 function give_get_spain_states_list() {
 	$states = array(
 		''   => '',
-		'C'  => __( 'A Coruña', 'give' ),
-		'VI' => __( 'Araba', 'give' ),
-		'AB' => __( 'Albacete', 'give' ),
-		'A'  => __( 'Alicante', 'give' ),
-		'AL' => __( 'Almería', 'give' ),
-		'O'  => __( 'Asturias', 'give' ),
-		'AV' => __( 'Ávila', 'give' ),
-		'BA' => __( 'Badajoz', 'give' ),
-		'PM' => __( 'Baleares', 'give' ),
-		'B'  => __( 'Barcelona', 'give' ),
-		'BU' => __( 'Burgos', 'give' ),
-		'CC' => __( 'Cáceres', 'give' ),
-		'CA' => __( 'Cádiz', 'give' ),
-		'S'  => __( 'Cantabria', 'give' ),
-		'CS' => __( 'Castellón', 'give' ),
-		'CE' => __( 'Ceuta', 'give' ),
-		'CR' => __( 'Ciudad Real', 'give' ),
-		'CO' => __( 'Córdoba', 'give' ),
-		'CU' => __( 'Cuenca', 'give' ),
-		'GI' => __( 'Girona', 'give' ),
-		'GR' => __( 'Granada', 'give' ),
-		'GU' => __( 'Guadalajara', 'give' ),
-		'SS' => __( 'Gipuzkoa', 'give' ),
-		'H'  => __( 'Huelva', 'give' ),
-		'HU' => __( 'Huesca', 'give' ),
-		'J'  => __( 'Jaén', 'give' ),
-		'LO' => __( 'La Rioja', 'give' ),
-		'GC' => __( 'Las Palmas', 'give' ),
-		'LE' => __( 'León', 'give' ),
-		'L'  => __( 'Lleida', 'give' ),
-		'LU' => __( 'Lugo', 'give' ),
-		'M'  => __( 'Madrid', 'give' ),
-		'MA' => __( 'Málaga', 'give' ),
-		'ML' => __( 'Melilla', 'give' ),
-		'MU' => __( 'Murcia', 'give' ),
-		'NA' => __( 'Navarra', 'give' ),
-		'OR' => __( 'Ourense', 'give' ),
-		'P'  => __( 'Palencia', 'give' ),
-		'PO' => __( 'Pontevedra', 'give' ),
-		'SA' => __( 'Salamanca', 'give' ),
-		'TF' => __( 'Santa Cruz de Tenerife', 'give' ),
-		'SG' => __( 'Segovia', 'give' ),
-		'SE' => __( 'Sevilla', 'give' ),
-		'SO' => __( 'Soria', 'give' ),
-		'T'  => __( 'Tarragona', 'give' ),
-		'TE' => __( 'Teruel', 'give' ),
-		'TO' => __( 'Toledo', 'give' ),
-		'V'  => __( 'Valencia', 'give' ),
-		'VA' => __( 'Valladolid', 'give' ),
-		'BI' => __( 'Bizkaia', 'give' ),
-		'ZA' => __( 'Zamora', 'give' ),
-		'Z'  => __( 'Zaragoza', 'give' )
+		'C'  => __('A Coruña', 'give'),
+		'VI' => __('Araba', 'give'),
+		'AB' => __('Albacete', 'give'),
+		'A'  => __('Alicante', 'give'),
+		'AL' => __('Almería', 'give'),
+		'O'  => __('Asturias', 'give'),
+		'AV' => __('Ávila', 'give'),
+		'BA' => __('Badajoz', 'give'),
+		'PM' => __('Baleares', 'give'),
+		'B'  => __('Barcelona', 'give'),
+		'BU' => __('Burgos', 'give'),
+		'CC' => __('Cáceres', 'give'),
+		'CA' => __('Cádiz', 'give'),
+		'S'  => __('Cantabria', 'give'),
+		'CS' => __('Castellón', 'give'),
+		'CE' => __('Ceuta', 'give'),
+		'CR' => __('Ciudad Real', 'give'),
+		'CO' => __('Córdoba', 'give'),
+		'CU' => __('Cuenca', 'give'),
+		'GI' => __('Girona', 'give'),
+		'GR' => __('Granada', 'give'),
+		'GU' => __('Guadalajara', 'give'),
+		'SS' => __('Gipuzkoa', 'give'),
+		'H'  => __('Huelva', 'give'),
+		'HU' => __('Huesca', 'give'),
+		'J'  => __('Jaén', 'give'),
+		'LO' => __('La Rioja', 'give'),
+		'GC' => __('Las Palmas', 'give'),
+		'LE' => __('León', 'give'),
+		'L'  => __('Lleida', 'give'),
+		'LU' => __('Lugo', 'give'),
+		'M'  => __('Madrid', 'give'),
+		'MA' => __('Málaga', 'give'),
+		'ML' => __('Melilla', 'give'),
+		'MU' => __('Murcia', 'give'),
+		'NA' => __('Navarra', 'give'),
+		'OR' => __('Ourense', 'give'),
+		'P'  => __('Palencia', 'give'),
+		'PO' => __('Pontevedra', 'give'),
+		'SA' => __('Salamanca', 'give'),
+		'TF' => __('Santa Cruz de Tenerife', 'give'),
+		'SG' => __('Segovia', 'give'),
+		'SE' => __('Sevilla', 'give'),
+		'SO' => __('Soria', 'give'),
+		'T'  => __('Tarragona', 'give'),
+		'TE' => __('Teruel', 'give'),
+		'TO' => __('Toledo', 'give'),
+		'V'  => __('Valencia', 'give'),
+		'VA' => __('Valladolid', 'give'),
+		'BI' => __('Bizkaia', 'give'),
+		'ZA' => __('Zamora', 'give'),
+		'Z'  => __('Zaragoza', 'give')
 	);
 
-	return apply_filters( 'give_spain_states', $states );
+	return apply_filters('give_spain_states', $states);
 }

includes/emails/actions.php

Spacing +18 added lines, -18 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,17 +23,17 @@ 
  *
  * @return void
  */
-function give_trigger_donation_receipt( $payment_id ) {
+function give_trigger_donation_receipt($payment_id) {
 	// Make sure we don't send a purchase receipt while editing a payment
-	if ( isset( $_POST['give-action'] ) && 'edit_payment' == $_POST['give-action'] ) {
+	if (isset($_POST['give-action']) && 'edit_payment' == $_POST['give-action']) {
 		return;
 	}
 
 	// Send email
-	give_email_donation_receipt( $payment_id );
+	give_email_donation_receipt($payment_id);
 }
 
-add_action( 'give_complete_purchase', 'give_trigger_donation_receipt', 999, 1 );
+add_action('give_complete_purchase', 'give_trigger_donation_receipt', 999, 1);
 
 /**
  * Resend the Email Purchase Receipt. (This can be done from the Payment History page)
@@ -44,29 +44,29 @@ 
  *
  * @return void
  */
-function give_resend_donation_receipt( $data ) {
+function give_resend_donation_receipt($data) {
 
-	$purchase_id = absint( $data['purchase_id'] );
+	$purchase_id = absint($data['purchase_id']);
 
-	if ( empty( $purchase_id ) ) {
+	if (empty($purchase_id)) {
 		return;
 	}
 
-	if ( ! current_user_can( 'edit_give_payments', $purchase_id ) ) {
-		wp_die( __( 'You do not have permission to edit this payment record', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! current_user_can('edit_give_payments', $purchase_id)) {
+		wp_die(__('You do not have permission to edit this payment record', 'give'), __('Error', 'give'), array('response' => 403));
 	}
 
-	give_email_donation_receipt( $purchase_id, false );
+	give_email_donation_receipt($purchase_id, false);
 
-	wp_redirect( add_query_arg( array(
+	wp_redirect(add_query_arg(array(
 		'give-message' => 'email_sent',
 		'give-action'  => false,
 		'purchase_id'  => false
-	) ) );
+	)));
 	exit;
 }
 
-add_action( 'give_email_links', 'give_resend_donation_receipt' );
+add_action('give_email_links', 'give_resend_donation_receipt');
 
 /**
  * Trigger the sending of a Test Email
@@ -77,8 +77,8 @@ 
  *
  * @return void
  */
-function give_send_test_email( $data ) {
-	if ( ! wp_verify_nonce( $data['_wpnonce'], 'give-test-email' ) ) {
+function give_send_test_email($data) {
+	if ( ! wp_verify_nonce($data['_wpnonce'], 'give-test-email')) {
 		return;
 	}
 
@@ -86,8 +86,8 @@ 
 	give_email_test_donation_receipt();
 
 	// Remove the test email query arg
-	wp_redirect( remove_query_arg( 'give_action' ) );
+	wp_redirect(remove_query_arg('give_action'));
 	exit;
 }
 
-add_action( 'give_send_test_email', 'give_send_test_email' );
+add_action('give_send_test_email', 'give_send_test_email');

includes/emails/class-give-emails.php

Indentation +10 added lines, -10 removed lines

@@ -1,15 +1,15 @@
 <?php
 /**
- * Emails
- *
- * This class handles all emails sent through Give
- *
- * @package     Give
- * @subpackage  Classes/Emails
- * @copyright   Copyright (c) 2015, WordImpress
- * @license     http://opensource.org/licenses/gpl-2.1.php GNU Public License
- * @since       1.0
- */
+		 * Emails
+		 *
+		 * This class handles all emails sent through Give
+		 *
+		 * @package     Give
+		 * @subpackage  Classes/Emails
+		 * @copyright   Copyright (c) 2015, WordImpress
+		 * @license     http://opensource.org/licenses/gpl-2.1.php GNU Public License
+		 * @since       1.0
+		 */
 
 // Exit if accessed directly
 if ( ! defined( 'ABSPATH' ) ) {

Spacing +58 added lines, -58 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -79,12 +79,12 @@ 
 	 */
 	public function __construct() {
 
-		if ( 'none' === $this->get_template() ) {
+		if ('none' === $this->get_template()) {
 			$this->html = false;
 		}
 
-		add_action( 'give_email_send_before', array( $this, 'send_before' ) );
-		add_action( 'give_email_send_after', array( $this, 'send_after' ) );
+		add_action('give_email_send_before', array($this, 'send_before'));
+		add_action('give_email_send_after', array($this, 'send_after'));
 
 	}
 
@@ -93,7 +93,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	public function __set( $key, $value ) {
+	public function __set($key, $value) {
 		$this->$key = $value;
 	}
 
@@ -103,11 +103,11 @@ 
 	 * @since 1.0
 	 */
 	public function get_from_name() {
-		if ( ! $this->from_name ) {
-			$this->from_name = give_get_option( 'from_name', get_bloginfo( 'name' ) );
+		if ( ! $this->from_name) {
+			$this->from_name = give_get_option('from_name', get_bloginfo('name'));
 		}
 
-		return apply_filters( 'give_email_from_name', wp_specialchars_decode( $this->from_name ), $this );
+		return apply_filters('give_email_from_name', wp_specialchars_decode($this->from_name), $this);
 	}
 
 	/**
@@ -116,11 +116,11 @@ 
 	 * @since 1.0
 	 */
 	public function get_from_address() {
-		if ( ! $this->from_address ) {
-			$this->from_address = give_get_option( 'from_email', get_option( 'admin_email' ) );
+		if ( ! $this->from_address) {
+			$this->from_address = give_get_option('from_email', get_option('admin_email'));
 		}
 
-		return apply_filters( 'give_email_from_address', $this->from_address, $this );
+		return apply_filters('give_email_from_address', $this->from_address, $this);
 	}
 
 	/**
@@ -129,13 +129,13 @@ 
 	 * @since 1.0
 	 */
 	public function get_content_type() {
-		if ( ! $this->content_type && $this->html ) {
-			$this->content_type = apply_filters( 'give_email_default_content_type', 'text/html', $this );
-		} else if ( ! $this->html ) {
+		if ( ! $this->content_type && $this->html) {
+			$this->content_type = apply_filters('give_email_default_content_type', 'text/html', $this);
+		} else if ( ! $this->html) {
 			$this->content_type = 'text/plain';
 		}
 
-		return apply_filters( 'give_email_content_type', $this->content_type, $this );
+		return apply_filters('give_email_content_type', $this->content_type, $this);
 	}
 
 	/**
@@ -144,13 +144,13 @@ 
 	 * @since 1.0
 	 */
 	public function get_headers() {
-		if ( ! $this->headers ) {
+		if ( ! $this->headers) {
 			$this->headers = "From: {$this->get_from_name()} <{$this->get_from_address()}>\r\n";
 			$this->headers .= "Reply-To: {$this->get_from_address()}\r\n";
 			$this->headers .= "Content-Type: {$this->get_content_type()}; charset=utf-8\r\n";
 		}
 
-		return apply_filters( 'give_email_headers', $this->headers, $this );
+		return apply_filters('give_email_headers', $this->headers, $this);
 	}
 
 	/**
@@ -160,11 +160,11 @@ 
 	 */
 	public function get_templates() {
 		$templates = array(
-			'default' => __( 'Default Template', 'give' ),
-			'none'    => __( 'No template, plain text only', 'give' )
+			'default' => __('Default Template', 'give'),
+			'none'    => __('No template, plain text only', 'give')
 		);
 
-		return apply_filters( 'give_email_templates', $templates );
+		return apply_filters('give_email_templates', $templates);
 	}
 
 	/**
@@ -173,11 +173,11 @@ 
 	 * @since 1.0
 	 */
 	public function get_template() {
-		if ( ! $this->template ) {
-			$this->template = give_get_option( 'email_template', 'default' );
+		if ( ! $this->template) {
+			$this->template = give_get_option('email_template', 'default');
 		}
 
-		return apply_filters( 'give_email_template', $this->template );
+		return apply_filters('give_email_template', $this->template);
 	}
 
 	/**
@@ -186,7 +186,7 @@ 
 	 * @since 1.0
 	 */
 	public function get_heading() {
-		return apply_filters( 'give_email_heading', $this->heading );
+		return apply_filters('give_email_heading', $this->heading);
 	}
 
 	/**
@@ -196,7 +196,7 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function parse_tags( $content ) {
+	public function parse_tags($content) {
 
 		// The email tags are parsed during setup for purchase receipts and sale notifications
 		// Once tags are not restricted to payments, we'll expand this. @see https://github.com/easydigitaldownloads/Easy-Digital-Downloads/issues/2151
@@ -209,36 +209,36 @@ 
 	 *
 	 * @since 1.0
 	 */
-	public function build_email( $message ) {
+	public function build_email($message) {
 
-		if ( false === $this->html ) {
-			return apply_filters( 'give_email_message', wp_strip_all_tags( $message ), $this );
+		if (false === $this->html) {
+			return apply_filters('give_email_message', wp_strip_all_tags($message), $this);
 		}
 
-		$message = $this->text_to_html( $message );
+		$message = $this->text_to_html($message);
 
 		ob_start();
 
-		give_get_template_part( 'emails/header', $this->get_template(), true );
+		give_get_template_part('emails/header', $this->get_template(), true);
 
-		do_action( 'give_email_header', $this );
+		do_action('give_email_header', $this);
 
-		if ( has_action( 'give_email_template_' . $this->get_template() ) ) {
-			do_action( 'give_email_template_' . $this->get_template() );
+		if (has_action('give_email_template_'.$this->get_template())) {
+			do_action('give_email_template_'.$this->get_template());
 		} else {
-			give_get_template_part( 'emails/body', $this->get_template(), true );
+			give_get_template_part('emails/body', $this->get_template(), true);
 		}
 
-		do_action( 'give_email_body', $this );
+		do_action('give_email_body', $this);
 
-		give_get_template_part( 'emails/footer', $this->get_template(), true );
+		give_get_template_part('emails/footer', $this->get_template(), true);
 
-		do_action( 'give_email_footer', $this );
+		do_action('give_email_footer', $this);
 
 		$body    = ob_get_clean();
-		$message = str_replace( '{email}', $message, $body );
+		$message = str_replace('{email}', $message, $body);
 
-		return apply_filters( 'give_email_message', $message, $this );
+		return apply_filters('give_email_message', $message, $this);
 	}
 
 	/**
@@ -251,26 +251,26 @@ 
 	 *
 	 * @return bool
 	 */
-	public function send( $to, $subject, $message, $attachments = '' ) {
+	public function send($to, $subject, $message, $attachments = '') {
 
-		if ( ! did_action( 'init' ) && ! did_action( 'admin_init' ) ) {
-			_doing_it_wrong( __FUNCTION__, __( 'You cannot send email with Give_Emails until init/admin_init has been reached', 'give' ), null );
+		if ( ! did_action('init') && ! did_action('admin_init')) {
+			_doing_it_wrong(__FUNCTION__, __('You cannot send email with Give_Emails until init/admin_init has been reached', 'give'), null);
 
 			return false;
 		}
 
-		do_action( 'give_email_send_before', $this );
+		do_action('give_email_send_before', $this);
 
-		$subject = $this->parse_tags( $subject );
-		$message = $this->parse_tags( $message );
+		$subject = $this->parse_tags($subject);
+		$message = $this->parse_tags($message);
 
-		$message = $this->build_email( $message );
+		$message = $this->build_email($message);
 
-		$attachments = apply_filters( 'give_email_attachments', $attachments, $this );
+		$attachments = apply_filters('give_email_attachments', $attachments, $this);
 
-		$sent = wp_mail( $to, $subject, $message, $this->get_headers(), $attachments );
+		$sent = wp_mail($to, $subject, $message, $this->get_headers(), $attachments);
 
-		do_action( 'give_email_send_after', $this );
+		do_action('give_email_send_after', $this);
 
 		return $sent;
 
@@ -282,9 +282,9 @@ 
 	 * @since 1.0
 	 */
 	public function send_before() {
-		add_filter( 'wp_mail_from', array( $this, 'get_from_address' ) );
-		add_filter( 'wp_mail_from_name', array( $this, 'get_from_name' ) );
-		add_filter( 'wp_mail_content_type', array( $this, 'get_content_type' ) );
+		add_filter('wp_mail_from', array($this, 'get_from_address'));
+		add_filter('wp_mail_from_name', array($this, 'get_from_name'));
+		add_filter('wp_mail_content_type', array($this, 'get_content_type'));
 	}
 
 	/**
@@ -293,9 +293,9 @@ 
 	 * @since 1.0
 	 */
 	public function send_after() {
-		remove_filter( 'wp_mail_from', array( $this, 'get_from_address' ) );
-		remove_filter( 'wp_mail_from_name', array( $this, 'get_from_name' ) );
-		remove_filter( 'wp_mail_content_type', array( $this, 'get_content_type' ) );
+		remove_filter('wp_mail_from', array($this, 'get_from_address'));
+		remove_filter('wp_mail_from_name', array($this, 'get_from_name'));
+		remove_filter('wp_mail_content_type', array($this, 'get_content_type'));
 
 		// Reset heading to an empty string
 		$this->heading = '';
@@ -306,10 +306,10 @@ 
 	 *
 	 * @since 1.0
 	 */
-	public function text_to_html( $message ) {
+	public function text_to_html($message) {
 
-		if ( 'text/html' == $this->content_type || true === $this->html ) {
-			$message = wpautop( $message );
+		if ('text/html' == $this->content_type || true === $this->html) {
+			$message = wpautop($message);
 		}
 
 		return $message;