WordImpress / Give

includes/admin/upgrades/upgrades.php

Spacing +17 added lines, -17 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,12 +21,12 @@ 
  * @return void
  */
 function give_upgrades_screen() {
-	$action = isset( $_GET['give-upgrade'] ) ? sanitize_text_field( $_GET['give-upgrade'] ) : '';
-	$step   = isset( $_GET['step'] ) ? absint( $_GET['step'] ) : 1;
-	$total  = isset( $_GET['total'] ) ? absint( $_GET['total'] ) : false;
-	$custom = isset( $_GET['custom'] ) ? absint( $_GET['custom'] ) : 0;
-	$number = isset( $_GET['number'] ) ? absint( $_GET['number'] ) : 100;
-	$steps  = round( ( $total / $number ), 0 );
+	$action = isset($_GET['give-upgrade']) ? sanitize_text_field($_GET['give-upgrade']) : '';
+	$step   = isset($_GET['step']) ? absint($_GET['step']) : 1;
+	$total  = isset($_GET['total']) ? absint($_GET['total']) : false;
+	$custom = isset($_GET['custom']) ? absint($_GET['custom']) : 0;
+	$number = isset($_GET['number']) ? absint($_GET['number']) : 100;
+	$steps  = round(($total / $number), 0);
 
 	$doing_upgrade_args = array(
 		'page'         => 'give-upgrades',
@@ -36,25 +36,25 @@ 
 		'custom'       => $custom,
 		'steps'        => $steps
 	);
-	update_option( 'give_doing_upgrade', $doing_upgrade_args );
-	if ( $step > $steps ) {
+	update_option('give_doing_upgrade', $doing_upgrade_args);
+	if ($step > $steps) {
 		// Prevent a weird case where the estimate was off. Usually only a couple.
 		$steps = $step;
 	}
 	?>
 	<div class="wrap">
-		<h2><?php _e( 'Give - Upgrades', 'give' ); ?></h2>
+		<h2><?php _e('Give - Upgrades', 'give'); ?></h2>
 
-		<?php if ( ! empty( $action ) ) : ?>
+		<?php if ( ! empty($action)) : ?>
 
 			<div id="give-upgrade-status">
-				<p style="font-size: 20px;max-width: 900px;"><?php _e( 'The upgrade process has started, please be patient and do not close this window or navigate away from this page. This could take several minutes depending on the upgrade and the size of your website. You will be automatically redirected when the upgrade is finished.', 'give' ); ?>
-					<img src="<?php echo GIVE_PLUGIN_URL . '/assets/images/spinner.gif'; ?>" id="give-upgrade-loader" style="  position: relative; top: 3px; left: 6px;" />
+				<p style="font-size: 20px;max-width: 900px;"><?php _e('The upgrade process has started, please be patient and do not close this window or navigate away from this page. This could take several minutes depending on the upgrade and the size of your website. You will be automatically redirected when the upgrade is finished.', 'give'); ?>
+					<img src="<?php echo GIVE_PLUGIN_URL.'/assets/images/spinner.gif'; ?>" id="give-upgrade-loader" style="  position: relative; top: 3px; left: 6px;" />
 				</p>
 
-				<?php if ( ! empty( $total ) ) : ?>
+				<?php if ( ! empty($total)) : ?>
 					<p>
-						<strong><?php printf( __( 'Step %d of approximately %d running', 'give' ), $step, $steps ); ?></strong>
+						<strong><?php printf(__('Step %d of approximately %d running', 'give'), $step, $steps); ?></strong>
 					</p>
 				<?php endif; ?>
 			</div>
@@ -68,8 +68,8 @@ 
 
 			<div id="give-upgrade-status">
 				<p style="font-size: 20px;max-width: 900px;">
-					<?php _e( 'The upgrade process has started, please be patient and do not close this window or navigate away from this page. This could take several minutes depending on the upgrade and the size of your website. You will be automatically redirected when the upgrade is finished.', 'give' ); ?>
-					<img src="<?php echo GIVE_PLUGIN_URL . '/assets/images/spinner.gif'; ?>" id="give-upgrade-loader" style="  position: relative; top: 3px; left: 6px;" />
+					<?php _e('The upgrade process has started, please be patient and do not close this window or navigate away from this page. This could take several minutes depending on the upgrade and the size of your website. You will be automatically redirected when the upgrade is finished.', 'give'); ?>
+					<img src="<?php echo GIVE_PLUGIN_URL.'/assets/images/spinner.gif'; ?>" id="give-upgrade-loader" style="  position: relative; top: 3px; left: 6px;" />
 				</p>
 			</div>
 			<script type="text/javascript">

Braces +5 added lines, -2 removed lines

@@ -64,11 +64,14 @@
 				}, 250 );
 			</script>
 
-		<?php else : ?>
+		<?php else {
+	: ?>
 
 			<div id="give-upgrade-status">
 				<p style="font-size: 20px;max-width: 900px;">
-					<?php _e( 'The upgrade process has started, please be patient and do not close this window or navigate away from this page. This could take several minutes depending on the upgrade and the size of your website. You will be automatically redirected when the upgrade is finished.', 'give' ); ?>
+					<?php _e( 'The upgrade process has started, please be patient and do not close this window or navigate away from this page. This could take several minutes depending on the upgrade and the size of your website. You will be automatically redirected when the upgrade is finished.', 'give' );
+}
+?>
 					<img src="<?php echo GIVE_PLUGIN_URL . '/assets/images/spinner.gif'; ?>" id="give-upgrade-loader" style="  position: relative; top: 3px; left: 6px;" />
 				</p>
 			</div>

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

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined('ABSPATH')) exit;
 
 /**
  * Give_API_V1 Class

Braces +3 added lines, -1 removed lines

@@ -10,7 +10,9 @@
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
 
 /**
  * Give_API_V1 Class

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/error-tracking.php

Spacing +30 added lines, -30 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -29,33 +29,33 @@ 
  *
  * @return void
  */
-function give_print_errors( $form_id ) {
+function give_print_errors($form_id) {
 
 	$errors = give_get_errors();
 
-	$request_form_id = isset( $_REQUEST['form-id'] ) ? intval( $_REQUEST['form-id'] ) : 0;
+	$request_form_id = isset($_REQUEST['form-id']) ? intval($_REQUEST['form-id']) : 0;
 
 	//Sanity checks first: Ensure that gateway returned errors display on the appropriate form
-	if ( ! isset( $_POST['give_ajax'] ) && $request_form_id !== $form_id ) {
+	if ( ! isset($_POST['give_ajax']) && $request_form_id !== $form_id) {
 		return;
 	}
 
-	if ( $errors ) {
-		$classes = apply_filters( 'give_error_class', array(
+	if ($errors) {
+		$classes = apply_filters('give_error_class', array(
 			'give_errors'
-		) );
-		echo '<div class="' . implode( ' ', $classes ) . '">';
+		));
+		echo '<div class="'.implode(' ', $classes).'">';
 		// Loop error codes and display errors
-		foreach ( $errors as $error_id => $error ) {
-			echo '<div class="give_error" id="give_error_' . $error_id . '"><p><strong>' . __( 'Error', 'give' ) . '</strong>: ' . $error . '</p></div>';
+		foreach ($errors as $error_id => $error) {
+			echo '<div class="give_error" id="give_error_'.$error_id.'"><p><strong>'.__('Error', 'give').'</strong>: '.$error.'</p></div>';
 		}
 		echo '</div>';
 		give_clear_errors();
 	}
 }
 
-add_action( 'give_purchase_form_before_personal_info', 'give_print_errors' );
-add_action( 'give_ajax_checkout_errors', 'give_print_errors' );
+add_action('give_purchase_form_before_personal_info', 'give_print_errors');
+add_action('give_ajax_checkout_errors', 'give_print_errors');
 
 /**
  * Get Errors
@@ -68,7 +68,7 @@ 
  * @return mixed array if errors are present, false if none found
  */
 function give_get_errors() {
-	return Give()->session->get( 'give_errors' );
+	return Give()->session->get('give_errors');
 }
 
 /**
@@ -84,13 +84,13 @@ 
  *
  * @return void
  */
-function give_set_error( $error_id, $error_message ) {
+function give_set_error($error_id, $error_message) {
 	$errors = give_get_errors();
-	if ( ! $errors ) {
+	if ( ! $errors) {
 		$errors = array();
 	}
-	$errors[ $error_id ] = $error_message;
-	Give()->session->set( 'give_errors', $errors );
+	$errors[$error_id] = $error_message;
+	Give()->session->set('give_errors', $errors);
 }
 
 /**
@@ -101,7 +101,7 @@ 
  * @return void
  */
 function give_clear_errors() {
-	Give()->session->set( 'give_errors', null );
+	Give()->session->set('give_errors', null);
 }
 
 /**
@@ -114,11 +114,11 @@ 
  *
  * @return string
  */
-function give_unset_error( $error_id ) {
+function give_unset_error($error_id) {
 	$errors = give_get_errors();
-	if ( $errors ) {
-		unset( $errors[ $error_id ] );
-		Give()->session->set( 'give_errors', $errors );
+	if ($errors) {
+		unset($errors[$error_id]);
+		Give()->session->set('give_errors', $errors);
 	}
 }
 
@@ -129,7 +129,7 @@ 
  * @return void
  */
 function _give_die_handler() {
-	if ( defined( 'GIVE_UNIT_TESTS' ) ) {
+	if (defined('GIVE_UNIT_TESTS')) {
 		return '_give_die_handler';
 	} else {
 		die();
@@ -144,10 +144,10 @@ 
  * @since  1.0
  * @return void
  */
-function give_die( $message = '', $title = '', $status = 400 ) {
-	add_filter( 'wp_die_ajax_handler', '_give_die_handler', 10, 3 );
-	add_filter( 'wp_die_handler', '_give_die_handler', 10, 3 );
-	wp_die( $message, $title, array( 'response' => $status ) );
+function give_die($message = '', $title = '', $status = 400) {
+	add_filter('wp_die_ajax_handler', '_give_die_handler', 10, 3);
+	add_filter('wp_die_handler', '_give_die_handler', 10, 3);
+	wp_die($message, $title, array('response' => $status));
 }
 
 /**
@@ -162,10 +162,10 @@ 
  *
  * @return   string  $error
  */
-function give_output_error( $message, $echo = true, $error_id = 'warning' ) {
-	$error = '<div class="give_errors" id="give_error_' . $error_id . '"><p class="give_error  give_' . $error_id . '">' . $message . '</p></div>';
+function give_output_error($message, $echo = true, $error_id = 'warning') {
+	$error = '<div class="give_errors" id="give_error_'.$error_id.'"><p class="give_error  give_'.$error_id.'">'.$message.'</p></div>';
 
-	if ( $echo ) {
+	if ($echo) {
 		echo $error;
 	} else {
 		return $error;

Doc Comments +2 added lines, -2 removed lines

@@ -65,7 +65,7 @@ 
  *
  * @since 1.0
  * @uses  Give_Session::get()
- * @return mixed array if errors are present, false if none found
+ * @return string array if errors are present, false if none found
  */
 function give_get_errors() {
 	return Give()->session->get( 'give_errors' );
@@ -126,7 +126,7 @@ 
  * Register die handler for give_die()
  *
  * @since  1.0
- * @return void
+ * @return string|null
  */
 function _give_die_handler() {
 	if ( defined( 'GIVE_UNIT_TESTS' ) ) {

includes/forms/widget.php

Spacing +47 added lines, -47 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-defined( 'ABSPATH' ) or exit;
+defined('ABSPATH') or exit;
 
 /**
  * Give Form widget
@@ -31,18 +31,18 @@ 
 	 */
 	public function __construct()
 	{
-		$this->self = get_class( $this );
+		$this->self = get_class($this);
 
 		parent::__construct(
-			strtolower( $this->self ),
-			__( 'Give - Donation Form', 'give' ),
+			strtolower($this->self),
+			__('Give - Donation Form', 'give'),
 			array(
-				'description' => __( 'Display a Give Donation Form in your theme\'s widget powered sidebar.', 'give' )
+				'description' => __('Display a Give Donation Form in your theme\'s widget powered sidebar.', 'give')
 			)
 		);
 
-		add_action( 'widgets_init',          array( $this, 'widget_init' ) );
-		add_action( 'admin_enqueue_scripts', array( $this, 'admin_widget_scripts' ) );
+		add_action('widgets_init', array($this, 'widget_init'));
+		add_action('admin_enqueue_scripts', array($this, 'admin_widget_scripts'));
 	}
 
 	/**
@@ -52,24 +52,24 @@ 
 	 *
 	 * @return void
 	 */
-	public function admin_widget_scripts( $hook )
+	public function admin_widget_scripts($hook)
 	{
 		// Directories of assets
-		$js_dir     = GIVE_PLUGIN_URL . 'assets/js/admin/';
-		$js_plugins = GIVE_PLUGIN_URL . 'assets/js/plugins/';
-		$css_dir    = GIVE_PLUGIN_URL . 'assets/css/';
+		$js_dir     = GIVE_PLUGIN_URL.'assets/js/admin/';
+		$js_plugins = GIVE_PLUGIN_URL.'assets/js/plugins/';
+		$css_dir    = GIVE_PLUGIN_URL.'assets/css/';
 
 		// Use minified libraries if SCRIPT_DEBUG is turned off
-		$suffix = ( defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ) ? '' : '.min';
+		$suffix = (defined('SCRIPT_DEBUG') && SCRIPT_DEBUG) ? '' : '.min';
 
 		// Widget Script
-		if ( $hook == 'widgets.php' ) {
+		if ($hook == 'widgets.php') {
 
-			wp_enqueue_style( 'give-qtip-css', $css_dir . 'jquery.qtip' . $suffix . '.css' );
+			wp_enqueue_style('give-qtip-css', $css_dir.'jquery.qtip'.$suffix.'.css');
 
-			wp_enqueue_script( 'give-qtip', $js_plugins . 'jquery.qtip' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION );
+			wp_enqueue_script('give-qtip', $js_plugins.'jquery.qtip'.$suffix.'.js', array('jquery'), GIVE_VERSION);
 
-			wp_enqueue_script( 'give-admin-widgets-scripts', $js_dir . 'admin-widgets' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, false );
+			wp_enqueue_script('give-admin-widgets-scripts', $js_dir.'admin-widgets'.$suffix.'.js', array('jquery'), GIVE_VERSION, false);
 		}
 	}
 
@@ -80,24 +80,24 @@ 
 	 *                        before_widget, and after_widget.
 	 * @param array $instance The settings for the particular instance of the widget.
 	 */
-	public function widget( $args, $instance )
+	public function widget($args, $instance)
 	{
-		extract( $args );
+		extract($args);
 
-		$title = !empty( $instance['title'] ) ? $instance['title'] : '';
-		$title = apply_filters( 'widget_title', $title, $instance, $this->id_base );
+		$title = ! empty($instance['title']) ? $instance['title'] : '';
+		$title = apply_filters('widget_title', $title, $instance, $this->id_base);
 
 		echo $before_widget;
 
-		do_action( 'give_before_forms_widget' );
+		do_action('give_before_forms_widget');
 
-		echo $title ? $before_title . $title . $after_title : '';
+		echo $title ? $before_title.$title.$after_title : '';
 
-		give_get_donation_form( $instance );
+		give_get_donation_form($instance);
 
 		echo $after_widget;
 
-		do_action( 'give_after_forms_widget' );
+		do_action('give_after_forms_widget');
 	}
 
 	/**
@@ -107,7 +107,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function form( $instance )
+	public function form($instance)
 	{
 		$defaults = array(
 			'title'        => '',
@@ -115,51 +115,51 @@ 
 			'float_labels' => '',
 		);
 
-		$instance = wp_parse_args( (array) $instance, $defaults );
+		$instance = wp_parse_args((array) $instance, $defaults);
 
-		extract( $instance );
+		extract($instance);
 
 		// Query Give Forms
 
 		$args = array(
 			'post_type'      => 'give_forms',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'publish',
 		);
 
-		$give_forms = get_posts( $args );
+		$give_forms = get_posts($args);
 
 		// Widget: Title
 
 		?><p>
-			<label for="<?php echo $this->get_field_id( 'title' ); ?>"><?php _e( 'Title:', 'give' ); ?></label>
-			<input type="text" class="widefat" id="<?php echo $this->get_field_id( 'title' ); ?>" name="<?php echo $this->get_field_name( 'title' ); ?>" value="<?php echo esc_attr( $title ); ?>" /><br>
-			<small><?php _e( 'Leave blank to hide the widget title.', 'give' ); ?></small>
+			<label for="<?php echo $this->get_field_id('title'); ?>"><?php _e('Title:', 'give'); ?></label>
+			<input type="text" class="widefat" id="<?php echo $this->get_field_id('title'); ?>" name="<?php echo $this->get_field_name('title'); ?>" value="<?php echo esc_attr($title); ?>" /><br>
+			<small><?php _e('Leave blank to hide the widget title.', 'give'); ?></small>
 		</p><?php
 
 		// Widget: Give Form
 
 		?><p>
-			<label for="<?php echo esc_attr( $this->get_field_id( 'id' ) ); ?>"><?php printf( __( 'Give %s:', 'give' ), give_get_forms_label_singular() ); ?></label>
-			<select class="widefat" name="<?php echo esc_attr( $this->get_field_name( 'id' ) ); ?>" id="<?php echo esc_attr( $this->get_field_id( 'id' ) ); ?>">
-				<option value="current"><?php _e( '— Select —', 'give' ); ?></option>
-				<?php foreach ( $give_forms as $give_form ) { ?>
-					<option <?php selected( absint( $id ), $give_form->ID ); ?> value="<?php echo esc_attr( $give_form->ID ); ?>"><?php echo $give_form->post_title; ?></option>
+			<label for="<?php echo esc_attr($this->get_field_id('id')); ?>"><?php printf(__('Give %s:', 'give'), give_get_forms_label_singular()); ?></label>
+			<select class="widefat" name="<?php echo esc_attr($this->get_field_name('id')); ?>" id="<?php echo esc_attr($this->get_field_id('id')); ?>">
+				<option value="current"><?php _e('— Select —', 'give'); ?></option>
+				<?php foreach ($give_forms as $give_form) { ?>
+					<option <?php selected(absint($id), $give_form->ID); ?> value="<?php echo esc_attr($give_form->ID); ?>"><?php echo $give_form->post_title; ?></option>
 				<?php } ?>
 			</select><br>
-			<small><?php _e( 'Select a Give Form to embed in this widget.', 'give' ); ?></small>
+			<small><?php _e('Select a Give Form to embed in this widget.', 'give'); ?></small>
 		</p><?php
 
 		// Widget: Floating Labels
 
 		?><p>
-			<label for="<?php echo esc_attr( $this->get_field_id( 'float_labels' ) ); ?>"><?php _e( 'Floating Labels (optional):', 'give' ); ?></label>
-			<select class="widefat" name="<?php echo esc_attr( $this->get_field_name( 'float_labels' ) ); ?>" id="<?php echo esc_attr( $this->get_field_id( 'float_labels' ) ); ?>">
-				<option value="" <?php selected( esc_attr( $float_labels ), '' ) ?>>– <?php _e( 'Select', 'give' ); ?> –</option>
-				<option value="enabled" <?php selected( esc_attr( $float_labels ), 'enabled' ) ?>><?php _e( 'Enabled', 'give' ); ?></option>
-				<option value="disabled" <?php selected( esc_attr( $float_labels ), 'disabled' ) ?>><?php _e( 'Disabled', 'give' ); ?></option>
+			<label for="<?php echo esc_attr($this->get_field_id('float_labels')); ?>"><?php _e('Floating Labels (optional):', 'give'); ?></label>
+			<select class="widefat" name="<?php echo esc_attr($this->get_field_name('float_labels')); ?>" id="<?php echo esc_attr($this->get_field_id('float_labels')); ?>">
+				<option value="" <?php selected(esc_attr($float_labels), '') ?>>– <?php _e('Select', 'give'); ?> –</option>
+				<option value="enabled" <?php selected(esc_attr($float_labels), 'enabled') ?>><?php _e('Enabled', 'give'); ?></option>
+				<option value="disabled" <?php selected(esc_attr($float_labels), 'disabled') ?>><?php _e('Disabled', 'give'); ?></option>
 			</select><br>
-			<small><?php printf( __( 'Override the <a href="%s" target="_blank">floating labels</a> setting for this Give form.', 'give' ), esc_url( "http://bradfrost.com/blog/post/float-label-pattern/" ) ); ?></small>
+			<small><?php printf(__('Override the <a href="%s" target="_blank">floating labels</a> setting for this Give form.', 'give'), esc_url("http://bradfrost.com/blog/post/float-label-pattern/")); ?></small>
 		</p><?php
 	}
 
@@ -170,7 +170,7 @@ 
 	 */
 	function widget_init()
 	{
-		register_widget( $this->self );
+		register_widget($this->self);
 	}
 
 	/**
@@ -181,7 +181,7 @@ 
 	 *
 	 * @return array
 	 */
-	public function update( $new_instance, $old_instance )
+	public function update($new_instance, $old_instance)
 	{
 		$this->flush_widget_cache();
 
@@ -195,7 +195,7 @@ 
 	 */
 	public function flush_widget_cache()
 	{
-		wp_cache_delete( $this->self, 'widget' );
+		wp_cache_delete($this->self, 'widget');
 	}
 }
 

includes/gateways/functions.php

Indentation +4 added lines, -4 removed lines

@@ -97,10 +97,10 @@
 
 	//Single Form settings varies compared to the Global default settings
 	if ( ! empty( $form_default ) &&
-	     $form_id !== null &&
-	     $default !== $form_default &&
-	     $form_default !== 'global' &&
-	     give_is_gateway_active( $form_default )
+		 $form_id !== null &&
+		 $default !== $form_default &&
+		 $form_default !== 'global' &&
+		 give_is_gateway_active( $form_default )
 	) {
 		$default = $form_default;
 	}

Spacing +75 added lines, -75 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,17 +24,17 @@ 
 	// Default, built-in gateways
 	$gateways = array(
 		'paypal' => array(
-			'admin_label'    => __( 'PayPal Standard', 'give' ),
-			'checkout_label' => __( 'PayPal', 'give' ),
-			'supports'       => array( 'buy_now' )
+			'admin_label'    => __('PayPal Standard', 'give'),
+			'checkout_label' => __('PayPal', 'give'),
+			'supports'       => array('buy_now')
 		),
 		'manual' => array(
-			'admin_label'    => __( 'Test Payment', 'give' ),
-			'checkout_label' => __( 'Test Payment', 'give' )
+			'admin_label'    => __('Test Payment', 'give'),
+			'checkout_label' => __('Test Payment', 'give')
 		),
 	);
 
-	return apply_filters( 'give_payment_gateways', $gateways );
+	return apply_filters('give_payment_gateways', $gateways);
 
 }
 
@@ -48,20 +48,20 @@ 
 
 	$gateways = give_get_payment_gateways();
 
-	$enabled = isset( $_POST['gateways'] ) ? $_POST['gateways'] : give_get_option( 'gateways' );
+	$enabled = isset($_POST['gateways']) ? $_POST['gateways'] : give_get_option('gateways');
 
 	$gateway_list = array();
 
-	foreach ( $gateways as $key => $gateway ) {
-		if ( isset( $enabled[ $key ] ) && $enabled[ $key ] == 1 ) {
-			$gateway_list[ $key ] = $gateway;
+	foreach ($gateways as $key => $gateway) {
+		if (isset($enabled[$key]) && $enabled[$key] == 1) {
+			$gateway_list[$key] = $gateway;
 		}
 	}
 
 	// Set order of payment gateway in list.
-	$gateway_list = give_get_ordered_payment_gateways( $gateway_list );
+	$gateway_list = give_get_ordered_payment_gateways($gateway_list);
 
-	return apply_filters( 'give_enabled_payment_gateways', $gateway_list );
+	return apply_filters('give_enabled_payment_gateways', $gateway_list);
 }
 
 /**
@@ -73,12 +73,12 @@ 
  *
  * @return boolean true if enabled, false otherwise
  */
-function give_is_gateway_active( $gateway ) {
+function give_is_gateway_active($gateway) {
 	$gateways = give_get_enabled_payment_gateways();
 
-	$ret = array_key_exists( $gateway, $gateways );
+	$ret = array_key_exists($gateway, $gateways);
 
-	return apply_filters( 'give_is_gateway_active', $ret, $gateway, $gateways );
+	return apply_filters('give_is_gateway_active', $ret, $gateway, $gateways);
 }
 
 /**
@@ -91,24 +91,24 @@ 
  *
  * @return string Gateway ID
  */
-function give_get_default_gateway( $form_id ) {
+function give_get_default_gateway($form_id) {
 
 	global $give_options;
 
-	$default      = isset( $give_options['default_gateway'] ) && give_is_gateway_active( $give_options['default_gateway'] ) ? $give_options['default_gateway'] : 'paypal';
-	$form_default = get_post_meta( $form_id, '_give_default_gateway', true );
+	$default      = isset($give_options['default_gateway']) && give_is_gateway_active($give_options['default_gateway']) ? $give_options['default_gateway'] : 'paypal';
+	$form_default = get_post_meta($form_id, '_give_default_gateway', true);
 
 	//Single Form settings varies compared to the Global default settings
-	if ( ! empty( $form_default ) &&
+	if ( ! empty($form_default) &&
 	     $form_id !== null &&
 	     $default !== $form_default &&
 	     $form_default !== 'global' &&
-	     give_is_gateway_active( $form_default )
+	     give_is_gateway_active($form_default)
 	) {
 		$default = $form_default;
 	}
 
-	return apply_filters( 'give_default_gateway', $default );
+	return apply_filters('give_default_gateway', $default);
 }
 
 /**
@@ -120,18 +120,18 @@ 
  *
  * @return string Gateway admin label
  */
-function give_get_gateway_admin_label( $gateway ) {
+function give_get_gateway_admin_label($gateway) {
 	$gateways = give_get_enabled_payment_gateways();
-	$label    = isset( $gateways[ $gateway ] ) ? $gateways[ $gateway ]['admin_label'] : $gateway;
-	$payment  = isset( $_GET['id'] ) ? absint( $_GET['id'] ) : false;
+	$label    = isset($gateways[$gateway]) ? $gateways[$gateway]['admin_label'] : $gateway;
+	$payment  = isset($_GET['id']) ? absint($_GET['id']) : false;
 
-	if ( $gateway == 'manual' && $payment ) {
-		if ( give_get_payment_amount( $payment ) == 0 ) {
-			$label = __( 'Test Donation', 'give' );
+	if ($gateway == 'manual' && $payment) {
+		if (give_get_payment_amount($payment) == 0) {
+			$label = __('Test Donation', 'give');
 		}
 	}
 
-	return apply_filters( 'give_gateway_admin_label', $label, $gateway );
+	return apply_filters('give_gateway_admin_label', $label, $gateway);
 }
 
 /**
@@ -143,15 +143,15 @@ 
  *
  * @return string Checkout label for the gateway
  */
-function give_get_gateway_checkout_label( $gateway ) {
+function give_get_gateway_checkout_label($gateway) {
 	$gateways = give_get_enabled_payment_gateways();
-	$label    = isset( $gateways[ $gateway ] ) ? $gateways[ $gateway ]['checkout_label'] : $gateway;
+	$label    = isset($gateways[$gateway]) ? $gateways[$gateway]['checkout_label'] : $gateway;
 
-	if ( $gateway == 'manual' ) {
-		$label = __( 'Test Donation', 'give' );
+	if ($gateway == 'manual') {
+		$label = __('Test Donation', 'give');
 	}
 
-	return apply_filters( 'give_gateway_checkout_label', $label, $gateway );
+	return apply_filters('give_gateway_checkout_label', $label, $gateway);
 }
 
 /**
@@ -163,11 +163,11 @@ 
  *
  * @return array Options the gateway supports
  */
-function give_get_gateway_supports( $gateway ) {
+function give_get_gateway_supports($gateway) {
 	$gateways = give_get_enabled_payment_gateways();
-	$supports = isset( $gateways[ $gateway ]['supports'] ) ? $gateways[ $gateway ]['supports'] : array();
+	$supports = isset($gateways[$gateway]['supports']) ? $gateways[$gateway]['supports'] : array();
 
-	return apply_filters( 'give_gateway_supports', $supports, $gateway );
+	return apply_filters('give_gateway_supports', $supports, $gateway);
 }
 
 /**
@@ -179,11 +179,11 @@ 
  *
  * @return bool
  */
-function give_gateway_supports_buy_now( $gateway ) {
-	$supports = give_get_gateway_supports( $gateway );
-	$ret      = in_array( 'buy_now', $supports );
+function give_gateway_supports_buy_now($gateway) {
+	$supports = give_get_gateway_supports($gateway);
+	$ret      = in_array('buy_now', $supports);
 
-	return apply_filters( 'give_gateway_supports_buy_now', $ret, $gateway );
+	return apply_filters('give_gateway_supports_buy_now', $ret, $gateway);
 }
 
 /**
@@ -196,16 +196,16 @@ 
 	$gateways = give_get_enabled_payment_gateways();
 	$ret      = false;
 
-	if ( $gateways ) {
-		foreach ( $gateways as $gateway_id => $gateway ) {
-			if ( give_gateway_supports_buy_now( $gateway_id ) ) {
+	if ($gateways) {
+		foreach ($gateways as $gateway_id => $gateway) {
+			if (give_gateway_supports_buy_now($gateway_id)) {
 				$ret = true;
 				break;
 			}
 		}
 	}
 
-	return apply_filters( 'give_give_supports_buy_now', $ret );
+	return apply_filters('give_give_supports_buy_now', $ret);
 }
 
 /**
@@ -218,12 +218,12 @@ 
  *
  * @return void
  */
-function give_send_to_gateway( $gateway, $payment_data ) {
+function give_send_to_gateway($gateway, $payment_data) {
 
-	$payment_data['gateway_nonce'] = wp_create_nonce( 'give-gateway' );
+	$payment_data['gateway_nonce'] = wp_create_nonce('give-gateway');
 
 	// $gateway must match the ID used when registering the gateway
-	do_action( 'give_gateway_' . $gateway, $payment_data );
+	do_action('give_gateway_'.$gateway, $payment_data);
 }
 
 
@@ -240,32 +240,32 @@ 
  *
  * @return string $enabled_gateway The slug of the gateway
  */
-function give_get_chosen_gateway( $form_id ) {
+function give_get_chosen_gateway($form_id) {
 	$gateways        = give_get_enabled_payment_gateways();
-	$request_form_id = isset( $_REQUEST['give_form_id'] ) ? $_REQUEST['give_form_id'] : 0;
-	if ( empty( $request_form_id ) ) {
-		$request_form_id = isset( $_REQUEST['form-id'] ) ? $_REQUEST['form-id'] : 0;
+	$request_form_id = isset($_REQUEST['give_form_id']) ? $_REQUEST['give_form_id'] : 0;
+	if (empty($request_form_id)) {
+		$request_form_id = isset($_REQUEST['form-id']) ? $_REQUEST['form-id'] : 0;
 	}
-	$chosen          = give_get_default_gateway( $form_id );
+	$chosen          = give_get_default_gateway($form_id);
 	$enabled_gateway = '';
 
 	//Take into account request Form ID args
-	if ( ! empty( $request_form_id ) && $form_id == $request_form_id ) {
+	if ( ! empty($request_form_id) && $form_id == $request_form_id) {
 		$chosen = $_REQUEST['payment-mode'];
 	}
 
-	if ( $chosen ) {
-		$enabled_gateway = urldecode( $chosen );
-	} else if ( count( $gateways ) >= 1 && ! $chosen ) {
-		foreach ( $gateways as $gateway_id => $gateway ):
+	if ($chosen) {
+		$enabled_gateway = urldecode($chosen);
+	} else if (count($gateways) >= 1 && ! $chosen) {
+		foreach ($gateways as $gateway_id => $gateway):
 			$enabled_gateway = $gateway_id;
 		endforeach;
 	} else {
-		$enabled_gateway = give_get_default_gateway( $form_id );
+		$enabled_gateway = give_get_default_gateway($form_id);
 	}
 
 
-	return apply_filters( 'give_chosen_gateway', $enabled_gateway );
+	return apply_filters('give_chosen_gateway', $enabled_gateway);
 }
 
 /**
@@ -282,8 +282,8 @@ 
  *
  * @return int ID of the new log entry
  */
-function give_record_gateway_error( $title = '', $message = '', $parent = 0 ) {
-	return give_record_log( $title, $message, $parent, 'gateway_error' );
+function give_record_gateway_error($title = '', $message = '', $parent = 0) {
+	return give_record_log($title, $message, $parent, 'gateway_error');
 }
 
 /**
@@ -296,7 +296,7 @@ 
  *
  * @return int
  */
-function give_count_sales_by_gateway( $gateway_id = 'paypal', $status = 'publish' ) {
+function give_count_sales_by_gateway($gateway_id = 'paypal', $status = 'publish') {
 
 	$ret  = 0;
 	$args = array(
@@ -308,9 +308,9 @@ 
 		'fields'      => 'ids'
 	);
 
-	$payments = new WP_Query( $args );
+	$payments = new WP_Query($args);
 
-	if ( $payments ) {
+	if ($payments) {
 		$ret = $payments->post_count;
 	}
 
@@ -327,27 +327,27 @@ 
  *
  * @return array $gateways All the available gateways
  */
-function give_get_ordered_payment_gateways( $gateways ) {
+function give_get_ordered_payment_gateways($gateways) {
 
 	//  Get gateways setting.
-	$gateways_setting = isset( $_POST['gateways'] ) ? $_POST['gateways'] : give_get_option( 'gateways' );
+	$gateways_setting = isset($_POST['gateways']) ? $_POST['gateways'] : give_get_option('gateways');
 
 	// Return from here if we do not have gateways setting.
-	if ( empty( $gateways_setting ) ) {
+	if (empty($gateways_setting)) {
 		return $gateways;
 	}
 
 	// Reverse array to order payment gateways.
-	$gateways_setting = array_reverse( $gateways_setting );
+	$gateways_setting = array_reverse($gateways_setting);
 
 	// Reorder gateways array
-	foreach ( $gateways_setting as $gateway_key => $value ) {
+	foreach ($gateways_setting as $gateway_key => $value) {
 
-		$new_gateway_value = isset( $gateways[ $gateway_key ] ) ? $gateways[ $gateway_key ] : '';
-		unset( $gateways[ $gateway_key ] );
+		$new_gateway_value = isset($gateways[$gateway_key]) ? $gateways[$gateway_key] : '';
+		unset($gateways[$gateway_key]);
 
-		if(!empty($new_gateway_value)) {
-			$gateways = array_merge( array( $gateway_key => $new_gateway_value ), $gateways );
+		if ( ! empty($new_gateway_value)) {
+			$gateways = array_merge(array($gateway_key => $new_gateway_value), $gateways);
 		}
 	}
 
@@ -358,5 +358,5 @@ 
 	 *
 	 * @param array $gateways All the available gateways
 	 */
-	return apply_filters( 'give_payment_gateways_order', $gateways );
+	return apply_filters('give_payment_gateways_order', $gateways);
 }

includes/price-functions.php

Spacing +15 added lines, -15 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,13 +23,13 @@ 
  *
  * @return bool true if has variable prices, false otherwise
  */
-function give_has_variable_prices( $form_id = 0 ) {
+function give_has_variable_prices($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
 	return $form->has_variable_prices();
 }
@@ -44,13 +44,13 @@ 
  *
  * @return array Variable prices
  */
-function give_get_variable_prices( $form_id = 0 ) {
+function give_get_variable_prices($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
 	return $form->prices;
 
@@ -67,13 +67,13 @@ 
  *
  * @return string $default_price
  */
-function give_get_default_multilevel_amount( $form_id ) {
+function give_get_default_multilevel_amount($form_id) {
 	$default_price = '1.00';
-	$prices        = apply_filters( 'give_form_variable_prices', give_get_variable_prices( $form_id ), $form_id );
+	$prices        = apply_filters('give_form_variable_prices', give_get_variable_prices($form_id), $form_id);
 
-	foreach ( $prices as $price ) {
+	foreach ($prices as $price) {
 
-		if ( isset( $price['_give_default'] ) && $price['_give_default'] === 'default' ) {
+		if (isset($price['_give_default']) && $price['_give_default'] === 'default') {
 			$default_price = $price['_give_amount'];
 		}
 
@@ -94,15 +94,15 @@ 
  * @return string $default_price
  * @since      1.0
  */
-function give_get_default_form_amount( $form_id ) {
+function give_get_default_form_amount($form_id) {
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		$default_amount = give_get_default_multilevel_amount( $form_id );
+		$default_amount = give_get_default_multilevel_amount($form_id);
 
 	} else {
 
-		$default_amount = get_post_meta( $form_id, '_give_set_price', true );
+		$default_amount = get_post_meta($form_id, '_give_set_price', true);
 
 	}
 

includes/shortcodes.php

Indentation +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
 
 		//validate display_style and float_labels value
 		if ( ( $key == 'display_style' && ! in_array( $value, array( 'onpage', 'reveal', 'modal' ) ) )
-		     || ( $key == 'float_labels' && ! in_array( $value, array( 'enabled', 'disabled' ) ) )
+			 || ( $key == 'float_labels' && ! in_array( $value, array( 'enabled', 'disabled' ) ) )
 		) {
 
 			$atts[ $key ] = '';

Spacing +108 added lines, -108 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,28 +25,28 @@ 
  */
 function give_donation_history() {
 
-	$email_access = give_get_option( 'email_access' );
+	$email_access = give_get_option('email_access');
 
 	//Is user logged in? Does a session exist? Does an email-access token exist?
-	if ( is_user_logged_in() || Give()->session->get_session_expiration() !== false || ( $email_access == 'on' && Give()->email_access->token_exists ) ) {
+	if (is_user_logged_in() || Give()->session->get_session_expiration() !== false || ($email_access == 'on' && Give()->email_access->token_exists)) {
 		ob_start();
-		give_get_template_part( 'history', 'donations' );
+		give_get_template_part('history', 'donations');
 
 		return ob_get_clean();
 	} //Is Email-based access enabled?
-	elseif ( $email_access == 'on' ) {
+	elseif ($email_access == 'on') {
 
 		ob_start();
-		give_get_template_part( 'email', 'login-form' );
+		give_get_template_part('email', 'login-form');
 
 		return ob_get_clean();
 	} else {
-		$message = __( 'You must be logged in to view your donation history. Please login using your account or create an account using the same email you used to donate with.', 'give' );
-		echo apply_filters( 'give_donation_history_nonuser_message', give_output_error( $message, false ), $message );
+		$message = __('You must be logged in to view your donation history. Please login using your account or create an account using the same email you used to donate with.', 'give');
+		echo apply_filters('give_donation_history_nonuser_message', give_output_error($message, false), $message);
 	}
 }
 
-add_shortcode( 'donation_history', 'give_donation_history' );
+add_shortcode('donation_history', 'give_donation_history');
 
 /**
  * Donation Form Shortcode
@@ -60,53 +60,53 @@ 
  *
  * @return string
  */
-function give_form_shortcode( $atts, $content = null ) {
-	$atts = shortcode_atts( array(
+function give_form_shortcode($atts, $content = null) {
+	$atts = shortcode_atts(array(
 		'id'            => '',
 		'show_title'    => true,
 		'show_goal'     => true,
 		'show_content'  => '',
 		'float_labels'  => '',
 		'display_style' => '',
-	), $atts, 'give_form' );
+	), $atts, 'give_form');
 
-	foreach ( $atts as $key => $value ) {
+	foreach ($atts as $key => $value) {
 		//convert shortcode_atts values to booleans
-		if ( $key == 'show_title' ) {
-			$atts[ $key ] = filter_var( $atts[ $key ], FILTER_VALIDATE_BOOLEAN );
-		} elseif ( $key == 'show_goal' ) {
-			$atts[ $key ] = filter_var( $atts[ $key ], FILTER_VALIDATE_BOOLEAN );
+		if ($key == 'show_title') {
+			$atts[$key] = filter_var($atts[$key], FILTER_VALIDATE_BOOLEAN);
+		} elseif ($key == 'show_goal') {
+			$atts[$key] = filter_var($atts[$key], FILTER_VALIDATE_BOOLEAN);
 		}
 
 		//validate show_content value
-		if ( $key == 'show_content' ) {
-			if ( ! in_array( $value, array( 'none', 'above', 'below' ) ) ) {
-				$atts[ $key ] = '';
-			} else if ( $value == 'above' ) {
-				$atts[ $key ] = 'give_pre_form';
-			} else if ( $value == 'below' ) {
-				$atts[ $key ] = 'give_post_form';
+		if ($key == 'show_content') {
+			if ( ! in_array($value, array('none', 'above', 'below'))) {
+				$atts[$key] = '';
+			} else if ($value == 'above') {
+				$atts[$key] = 'give_pre_form';
+			} else if ($value == 'below') {
+				$atts[$key] = 'give_post_form';
 			}
 		}
 
 		//validate display_style and float_labels value
-		if ( ( $key == 'display_style' && ! in_array( $value, array( 'onpage', 'reveal', 'modal' ) ) )
-		     || ( $key == 'float_labels' && ! in_array( $value, array( 'enabled', 'disabled' ) ) )
+		if (($key == 'display_style' && ! in_array($value, array('onpage', 'reveal', 'modal')))
+		     || ($key == 'float_labels' && ! in_array($value, array('enabled', 'disabled')))
 		) {
 
-			$atts[ $key ] = '';
+			$atts[$key] = '';
 		}
 	}
 
 	//get the Give Form
 	ob_start();
-	give_get_donation_form( $atts );
+	give_get_donation_form($atts);
 	$final_output = ob_get_clean();
 
-	return apply_filters( 'give_donate_form', $final_output, $atts );
+	return apply_filters('give_donate_form', $final_output, $atts);
 }
 
-add_shortcode( 'give_form', 'give_form_shortcode' );
+add_shortcode('give_form', 'give_form_shortcode');
 
 /**
  * Donation Form Goal Shortcode
@@ -120,37 +120,37 @@ 
  *
  * @return string
  */
-function give_goal_shortcode( $atts, $content = null ) {
-	$atts = shortcode_atts( array(
+function give_goal_shortcode($atts, $content = null) {
+	$atts = shortcode_atts(array(
 		'id'        => '',
 		'show_text' => true,
 		'show_bar'  => true,
-	), $atts, 'give_goal' );
+	), $atts, 'give_goal');
 
 
 	//get the Give Form
 	ob_start();
 
 	//Sanity check 1: ensure there is an ID Provided
-	if ( empty( $atts['id'] ) ) {
-		give_output_error( __( 'Error: No Donation form ID for the shortcode provided.', 'give' ), true );
+	if (empty($atts['id'])) {
+		give_output_error(__('Error: No Donation form ID for the shortcode provided.', 'give'), true);
 	}
 
 	//Sanity check 2: Check that this form even has Goals enabled
-	$goal_option = get_post_meta( $atts['id'], '_give_goal_option', true );
-	if ( empty( $goal_option ) || $goal_option !== 'yes' ) {
-		give_output_error( __( 'Error: This form does not have Goals enabled.', 'give' ), true );
+	$goal_option = get_post_meta($atts['id'], '_give_goal_option', true);
+	if (empty($goal_option) || $goal_option !== 'yes') {
+		give_output_error(__('Error: This form does not have Goals enabled.', 'give'), true);
 	} else {
 		//Passed all sanity checks: output Goal
-		give_show_goal_progress( $atts['id'], $atts );
+		give_show_goal_progress($atts['id'], $atts);
 	}
 
 	$final_output = ob_get_clean();
 
-	return apply_filters( 'give_goal_shortcode_output', $final_output, $atts );
+	return apply_filters('give_goal_shortcode_output', $final_output, $atts);
 }
 
-add_shortcode( 'give_goal', 'give_goal_shortcode' );
+add_shortcode('give_goal', 'give_goal_shortcode');
 
 
 /**
@@ -167,15 +167,15 @@ 
  * @uses  give_login_form()
  * @return string
  */
-function give_login_form_shortcode( $atts, $content = null ) {
-	$atts = shortcode_atts( array(
+function give_login_form_shortcode($atts, $content = null) {
+	$atts = shortcode_atts(array(
 		'redirect' => '',
-	), $atts, 'give_login' );
+	), $atts, 'give_login');
 
-	return give_login_form( $atts['redirect'] );
+	return give_login_form($atts['redirect']);
 }
 
-add_shortcode( 'give_login', 'give_login_form_shortcode' );
+add_shortcode('give_login', 'give_login_form_shortcode');
 
 /**
  * Register Shortcode
@@ -190,15 +190,15 @@ 
  * @uses  give_register_form()
  * @return string
  */
-function give_register_form_shortcode( $atts, $content = null ) {
-	$atts = shortcode_atts( array(
+function give_register_form_shortcode($atts, $content = null) {
+	$atts = shortcode_atts(array(
 		'redirect' => '',
-	), $atts, 'give_register' );
+	), $atts, 'give_register');
 
-	return give_register_form( $atts['redirect'] );
+	return give_register_form($atts['redirect']);
 }
 
-add_shortcode( 'give_register', 'give_register_form_shortcode' );
+add_shortcode('give_register', 'give_register_form_shortcode');
 
 
 /**
@@ -213,61 +213,61 @@ 
  *
  * @return string
  */
-function give_receipt_shortcode( $atts, $content = null ) {
+function give_receipt_shortcode($atts, $content = null) {
 
 	global $give_receipt_args, $payment;
 
-	$give_receipt_args = shortcode_atts( array(
-		'error'          => esc_html__( 'Sorry, you are missing the payment key to view this donation receipt.', 'give' ),
+	$give_receipt_args = shortcode_atts(array(
+		'error'          => esc_html__('Sorry, you are missing the payment key to view this donation receipt.', 'give'),
 		'price'          => true,
 		'date'           => true,
 		'payment_key'    => false,
 		'payment_method' => true,
 		'payment_id'     => true
-	), $atts, 'give_receipt' );
+	), $atts, 'give_receipt');
 
 	//set $session var
 	$session = give_get_purchase_session();
 
 	//set payment key var
-	if ( isset( $_GET['payment_key'] ) ) {
-		$payment_key = urldecode( $_GET['payment_key'] );
-	} elseif ( $session ) {
+	if (isset($_GET['payment_key'])) {
+		$payment_key = urldecode($_GET['payment_key']);
+	} elseif ($session) {
 		$payment_key = $session['purchase_key'];
-	} elseif ( $give_receipt_args['payment_key'] ) {
+	} elseif ($give_receipt_args['payment_key']) {
 		$payment_key = $give_receipt_args['payment_key'];
 	}
 
-	$email_access = give_get_option( 'email_access' );
+	$email_access = give_get_option('email_access');
 
 	// No payment_key found & Email Access is Turned on:
-	if ( ! isset( $payment_key ) && $email_access == 'on' && ! Give()->email_access->token_exists ) {
+	if ( ! isset($payment_key) && $email_access == 'on' && ! Give()->email_access->token_exists) {
 
 		ob_start();
 
-		give_get_template_part( 'email-login-form' );
+		give_get_template_part('email-login-form');
 
 		return ob_get_clean();
 
-	} elseif ( ! isset( $payment_key ) ) {
+	} elseif ( ! isset($payment_key)) {
 
-		return give_output_error( $give_receipt_args['error'], false, 'error' );
+		return give_output_error($give_receipt_args['error'], false, 'error');
 
 	}
 
-	$payment_id    = give_get_purchase_id_by_key( $payment_key );
-	$user_can_view = give_can_view_receipt( $payment_key );
+	$payment_id    = give_get_purchase_id_by_key($payment_key);
+	$user_can_view = give_can_view_receipt($payment_key);
 
 	// Key was provided, but user is logged out. Offer them the ability to login and view the receipt
-	if ( ! $user_can_view && $email_access == 'on' && ! Give()->email_access->token_exists ) {
+	if ( ! $user_can_view && $email_access == 'on' && ! Give()->email_access->token_exists) {
 
 		ob_start();
 
-		give_get_template_part( 'email-login-form' );
+		give_get_template_part('email-login-form');
 
 		return ob_get_clean();
 
-	} elseif ( ! $user_can_view ) {
+	} elseif ( ! $user_can_view) {
 
 		global $give_login_redirect;
 
@@ -275,9 +275,9 @@ 
 
 		ob_start();
 
-		give_output_error( apply_filters( 'give_must_be_logged_in_error_message', esc_html__( 'You must be logged in to view this donation payment receipt.', 'give' ) ) );
+		give_output_error(apply_filters('give_must_be_logged_in_error_message', esc_html__('You must be logged in to view this donation payment receipt.', 'give')));
 
-		give_get_template_part( 'shortcode', 'login' );
+		give_get_template_part('shortcode', 'login');
 
 		$login_form = ob_get_clean();
 
@@ -294,13 +294,13 @@ 
 	 * Or if user is logged in and the user can view sensitive shop data
 	 *
 	 */
-	if ( ! apply_filters( 'give_user_can_view_receipt', $user_can_view, $give_receipt_args ) ) {
-		return give_output_error( $give_receipt_args['error'], false, 'error' );
+	if ( ! apply_filters('give_user_can_view_receipt', $user_can_view, $give_receipt_args)) {
+		return give_output_error($give_receipt_args['error'], false, 'error');
 	}
 
 	ob_start();
 
-	give_get_template_part( 'shortcode', 'receipt' );
+	give_get_template_part('shortcode', 'receipt');
 
 	$display = ob_get_clean();
 
@@ -309,7 +309,7 @@ 
 
 }
 
-add_shortcode( 'give_receipt', 'give_receipt_shortcode' );
+add_shortcode('give_receipt', 'give_receipt_shortcode');
 
 /**
  * Profile Editor Shortcode
@@ -329,18 +329,18 @@ 
  *
  * @return string Output generated from the profile editor
  */
-function give_profile_editor_shortcode( $atts, $content = null ) {
+function give_profile_editor_shortcode($atts, $content = null) {
 
 	ob_start();
 
-	give_get_template_part( 'shortcode', 'profile-editor' );
+	give_get_template_part('shortcode', 'profile-editor');
 
 	$display = ob_get_clean();
 
 	return $display;
 }
 
-add_shortcode( 'give_profile_editor', 'give_profile_editor_shortcode' );
+add_shortcode('give_profile_editor', 'give_profile_editor_shortcode');
 
 /**
  * Process Profile Updater Form
@@ -353,30 +353,30 @@ 
  *
  * @return false
  */
-function give_process_profile_editor_updates( $data ) {
+function give_process_profile_editor_updates($data) {
 	// Profile field change request
-	if ( empty( $_POST['give_profile_editor_submit'] ) && ! is_user_logged_in() ) {
+	if (empty($_POST['give_profile_editor_submit']) && ! is_user_logged_in()) {
 		return false;
 	}
 
 	// Nonce security
-	if ( ! wp_verify_nonce( $data['give_profile_editor_nonce'], 'give-profile-editor-nonce' ) ) {
+	if ( ! wp_verify_nonce($data['give_profile_editor_nonce'], 'give-profile-editor-nonce')) {
 		return false;
 	}
 
 	$user_id       = get_current_user_id();
-	$old_user_data = get_userdata( $user_id );
-
-	$display_name = isset( $data['give_display_name'] ) ? sanitize_text_field( $data['give_display_name'] ) : $old_user_data->display_name;
-	$first_name   = isset( $data['give_first_name'] ) ? sanitize_text_field( $data['give_first_name'] ) : $old_user_data->first_name;
-	$last_name    = isset( $data['give_last_name'] ) ? sanitize_text_field( $data['give_last_name'] ) : $old_user_data->last_name;
-	$email        = isset( $data['give_email'] ) ? sanitize_email( $data['give_email'] ) : $old_user_data->user_email;
-	$line1        = ( isset( $data['give_address_line1'] ) ? sanitize_text_field( $data['give_address_line1'] ) : '' );
-	$line2        = ( isset( $data['give_address_line2'] ) ? sanitize_text_field( $data['give_address_line2'] ) : '' );
-	$city         = ( isset( $data['give_address_city'] ) ? sanitize_text_field( $data['give_address_city'] ) : '' );
-	$state        = ( isset( $data['give_address_state'] ) ? sanitize_text_field( $data['give_address_state'] ) : '' );
-	$zip          = ( isset( $data['give_address_zip'] ) ? sanitize_text_field( $data['give_address_zip'] ) : '' );
-	$country      = ( isset( $data['give_address_country'] ) ? sanitize_text_field( $data['give_address_country'] ) : '' );
+	$old_user_data = get_userdata($user_id);
+
+	$display_name = isset($data['give_display_name']) ? sanitize_text_field($data['give_display_name']) : $old_user_data->display_name;
+	$first_name   = isset($data['give_first_name']) ? sanitize_text_field($data['give_first_name']) : $old_user_data->first_name;
+	$last_name    = isset($data['give_last_name']) ? sanitize_text_field($data['give_last_name']) : $old_user_data->last_name;
+	$email        = isset($data['give_email']) ? sanitize_email($data['give_email']) : $old_user_data->user_email;
+	$line1        = (isset($data['give_address_line1']) ? sanitize_text_field($data['give_address_line1']) : '');
+	$line2        = (isset($data['give_address_line2']) ? sanitize_text_field($data['give_address_line2']) : '');
+	$city         = (isset($data['give_address_city']) ? sanitize_text_field($data['give_address_city']) : '');
+	$state        = (isset($data['give_address_state']) ? sanitize_text_field($data['give_address_state']) : '');
+	$zip          = (isset($data['give_address_zip']) ? sanitize_text_field($data['give_address_zip']) : '');
+	$country      = (isset($data['give_address_country']) ? sanitize_text_field($data['give_address_country']) : '');
 
 	$userdata = array(
 		'ID'           => $user_id,
@@ -396,45 +396,45 @@ 
 		'country' => $country
 	);
 
-	do_action( 'give_pre_update_user_profile', $user_id, $userdata );
+	do_action('give_pre_update_user_profile', $user_id, $userdata);
 
 	// New password
-	if ( ! empty( $data['give_new_user_pass1'] ) ) {
-		if ( $data['give_new_user_pass1'] !== $data['give_new_user_pass2'] ) {
-			give_set_error( 'password_mismatch', __( 'The passwords you entered do not match. Please try again.', 'give' ) );
+	if ( ! empty($data['give_new_user_pass1'])) {
+		if ($data['give_new_user_pass1'] !== $data['give_new_user_pass2']) {
+			give_set_error('password_mismatch', __('The passwords you entered do not match. Please try again.', 'give'));
 		} else {
 			$userdata['user_pass'] = $data['give_new_user_pass1'];
 		}
 	}
 
 	// Make sure the new email doesn't belong to another user
-	if ( $email != $old_user_data->user_email ) {
-		if ( email_exists( $email ) ) {
-			give_set_error( 'email_exists', __( 'The email you entered belongs to another user. Please use another.', 'give' ) );
+	if ($email != $old_user_data->user_email) {
+		if (email_exists($email)) {
+			give_set_error('email_exists', __('The email you entered belongs to another user. Please use another.', 'give'));
 		}
 	}
 
 	// Check for errors
 	$errors = give_get_errors();
 
-	if ( $errors ) {
+	if ($errors) {
 		// Send back to the profile editor if there are errors
-		wp_redirect( $data['give_redirect'] );
+		wp_redirect($data['give_redirect']);
 		give_die();
 	}
 
 	// Update the user
-	$meta    = update_user_meta( $user_id, '_give_user_address', $address );
-	$updated = wp_update_user( $userdata );
+	$meta    = update_user_meta($user_id, '_give_user_address', $address);
+	$updated = wp_update_user($userdata);
 
-	if ( $updated ) {
-		do_action( 'give_user_profile_updated', $user_id, $userdata );
-		wp_redirect( add_query_arg( 'updated', 'true', $data['give_redirect'] ) );
+	if ($updated) {
+		do_action('give_user_profile_updated', $user_id, $userdata);
+		wp_redirect(add_query_arg('updated', 'true', $data['give_redirect']));
 		give_die();
 	}
 
 	return false;
 }
 
-add_action( 'give_edit_user_profile', 'give_process_profile_editor_updates' );
+add_action('give_edit_user_profile', 'give_process_profile_editor_updates');