WordImpress / Give

includes/error-tracking.php

Doc Comments +1 added lines, -1 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' );

includes/login-register.php

Indentation +24 added lines, -24 removed lines

@@ -33,20 +33,20 @@ 
 		$login_redirect = add_query_arg('give-login-success', 'true', give_get_current_page_url());
 	}
 
-    if ( empty( $logout_redirect ) ) {
-        $logout_redirect = add_query_arg( 'give-logout-success', 'true', give_get_current_page_url() );
-    }
+	if ( empty( $logout_redirect ) ) {
+		$logout_redirect = add_query_arg( 'give-logout-success', 'true', give_get_current_page_url() );
+	}
 
 
-    // Add user_logout action to logout url.
-    $logout_redirect = add_query_arg(
-        array(
-            'give_action'       => 'user_logout',
-            'give_logout_nonce' => wp_create_nonce( 'give-logout-nonce' ),
-            'give_logout_redirect' => urlencode( $logout_redirect )
-        ),
-        home_url('/')
-    );
+	// Add user_logout action to logout url.
+	$logout_redirect = add_query_arg(
+		array(
+			'give_action'       => 'user_logout',
+			'give_logout_nonce' => wp_create_nonce( 'give-logout-nonce' ),
+			'give_logout_redirect' => urlencode( $logout_redirect )
+		),
+		home_url('/')
+	);
 
 	$give_login_redirect = $login_redirect;
 	$give_logout_redirect = $logout_redirect;
@@ -136,23 +136,23 @@ 
  * @return void
  */
 function give_process_user_logout( $data ) {
-    if ( wp_verify_nonce( $data['give_logout_nonce'], 'give-logout-nonce' ) && is_user_logged_in() ) {
+	if ( wp_verify_nonce( $data['give_logout_nonce'], 'give-logout-nonce' ) && is_user_logged_in() ) {
 
-        // Prevent occurring of any custom action on wp_logout.
-        remove_all_actions( 'wp_logout' );
+		// Prevent occurring of any custom action on wp_logout.
+		remove_all_actions( 'wp_logout' );
 
-        // Before logout give action.
-        do_action( 'give_before_user_logout' );
+		// Before logout give action.
+		do_action( 'give_before_user_logout' );
 
-        // Logout user.
-        wp_logout();
+		// Logout user.
+		wp_logout();
 
-        // After logout give action.
-        do_action( 'give_after_user_logout' );
+		// After logout give action.
+		do_action( 'give_after_user_logout' );
 
-        wp_redirect( $data['give_logout_redirect'] );
-        give_die();
-    }
+		wp_redirect( $data['give_logout_redirect'] );
+		give_die();
+	}
 }
 
 add_action( 'give_user_logout', 'give_process_user_logout' );

Spacing +71 added lines, -71 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,15 +26,15 @@ 
  *
  * @return string Login form
  */
-function give_login_form( $login_redirect = '', $logout_redirect = '' ) {
-	global $give_login_redirect, $give_logout_redirect;;
+function give_login_form($login_redirect = '', $logout_redirect = '') {
+	global $give_login_redirect, $give_logout_redirect; ;
 
-	if ( empty( $login_redirect ) ) {
+	if (empty($login_redirect)) {
 		$login_redirect = add_query_arg('give-login-success', 'true', give_get_current_page_url());
 	}
 
-    if ( empty( $logout_redirect ) ) {
-        $logout_redirect = add_query_arg( 'give-logout-success', 'true', give_get_current_page_url() );
+    if (empty($logout_redirect)) {
+        $logout_redirect = add_query_arg('give-logout-success', 'true', give_get_current_page_url());
     }
 
 
@@ -42,8 +42,8 @@ 
     $logout_redirect = add_query_arg(
         array(
             'give_action'       => 'user_logout',
-            'give_logout_nonce' => wp_create_nonce( 'give-logout-nonce' ),
-            'give_logout_redirect' => urlencode( $logout_redirect )
+            'give_logout_nonce' => wp_create_nonce('give-logout-nonce'),
+            'give_logout_redirect' => urlencode($logout_redirect)
         ),
         home_url('/')
     );
@@ -53,9 +53,9 @@ 
 
 	ob_start();
 
-	give_get_template_part( 'shortcode', 'login' );
+	give_get_template_part('shortcode', 'login');
 
-	return apply_filters( 'give_login_form', ob_get_clean() );
+	return apply_filters('give_login_form', ob_get_clean());
 }
 
 /**
@@ -69,10 +69,10 @@ 
  *
  * @return string Register form
  */
-function give_register_form( $redirect = '' ) {
+function give_register_form($redirect = '') {
 	global $give_options, $give_register_redirect;
 
-	if ( empty( $redirect ) ) {
+	if (empty($redirect)) {
 		$redirect = give_get_current_page_url();
 	}
 
@@ -80,11 +80,11 @@ 
 
 	ob_start();
 
-	if ( ! is_user_logged_in() ) {
-		give_get_template_part( 'shortcode', 'register' );
+	if ( ! is_user_logged_in()) {
+		give_get_template_part('shortcode', 'register');
 	}
 
-	return apply_filters( 'give_register_form', ob_get_clean() );
+	return apply_filters('give_register_form', ob_get_clean());
 }
 
 /**
@@ -96,34 +96,34 @@ 
  *
  * @return void
  */
-function give_process_login_form( $data ) {
-	if ( wp_verify_nonce( $data['give_login_nonce'], 'give-login-nonce' ) ) {
-		$user_data = get_user_by( 'login', $data['give_user_login'] );
-		if ( ! $user_data ) {
-			$user_data = get_user_by( 'email', $data['give_user_login'] );
+function give_process_login_form($data) {
+	if (wp_verify_nonce($data['give_login_nonce'], 'give-login-nonce')) {
+		$user_data = get_user_by('login', $data['give_user_login']);
+		if ( ! $user_data) {
+			$user_data = get_user_by('email', $data['give_user_login']);
 		}
-		if ( $user_data ) {
+		if ($user_data) {
 			$user_ID    = $user_data->ID;
 			$user_email = $user_data->user_email;
-			if ( wp_check_password( $data['give_user_pass'], $user_data->user_pass, $user_data->ID ) ) {
-				give_log_user_in( $user_data->ID, $data['give_user_login'], $data['give_user_pass'] );
+			if (wp_check_password($data['give_user_pass'], $user_data->user_pass, $user_data->ID)) {
+				give_log_user_in($user_data->ID, $data['give_user_login'], $data['give_user_pass']);
 			} else {
-				give_set_error( 'password_incorrect', __( 'The password you entered is incorrect.', 'give' ) );
+				give_set_error('password_incorrect', __('The password you entered is incorrect.', 'give'));
 			}
 		} else {
-			give_set_error( 'username_incorrect', __( 'The username you entered does not exist.', 'give' ) );
+			give_set_error('username_incorrect', __('The username you entered does not exist.', 'give'));
 		}
 		// Check for errors and redirect if none present
 		$errors = give_get_errors();
-		if ( ! $errors ) {
-			$redirect = apply_filters( 'give_login_redirect', $data['give_login_redirect'], $user_ID );
-			wp_redirect( $redirect );
+		if ( ! $errors) {
+			$redirect = apply_filters('give_login_redirect', $data['give_login_redirect'], $user_ID);
+			wp_redirect($redirect);
 			give_die();
 		}
 	}
 }
 
-add_action( 'give_user_login', 'give_process_login_form' );
+add_action('give_user_login', 'give_process_login_form');
 
 
 /**
@@ -135,27 +135,27 @@ 
  *
  * @return void
  */
-function give_process_user_logout( $data ) {
-    if ( wp_verify_nonce( $data['give_logout_nonce'], 'give-logout-nonce' ) && is_user_logged_in() ) {
+function give_process_user_logout($data) {
+    if (wp_verify_nonce($data['give_logout_nonce'], 'give-logout-nonce') && is_user_logged_in()) {
 
         // Prevent occurring of any custom action on wp_logout.
-        remove_all_actions( 'wp_logout' );
+        remove_all_actions('wp_logout');
 
         // Before logout give action.
-        do_action( 'give_before_user_logout' );
+        do_action('give_before_user_logout');
 
         // Logout user.
         wp_logout();
 
         // After logout give action.
-        do_action( 'give_after_user_logout' );
+        do_action('give_after_user_logout');
 
-        wp_redirect( $data['give_logout_redirect'] );
+        wp_redirect($data['give_logout_redirect']);
         give_die();
     }
 }
 
-add_action( 'give_user_logout', 'give_process_user_logout' );
+add_action('give_user_logout', 'give_process_user_logout');
 
 /**
  * Log User In
@@ -168,15 +168,15 @@ 
  *
  * @return void
  */
-function give_log_user_in( $user_id, $user_login, $user_pass ) {
-	if ( $user_id < 1 ) {
+function give_log_user_in($user_id, $user_login, $user_pass) {
+	if ($user_id < 1) {
 		return;
 	}
 
-	wp_set_auth_cookie( $user_id );
-	wp_set_current_user( $user_id, $user_login );
-	do_action( 'wp_login', $user_login, get_userdata( $user_id ) );
-	do_action( 'give_log_user_in', $user_id, $user_login, $user_pass );
+	wp_set_auth_cookie($user_id);
+	wp_set_current_user($user_id, $user_login);
+	do_action('wp_login', $user_login, get_userdata($user_id));
+	do_action('give_log_user_in', $user_id, $user_login, $user_pass);
 }
 
 
@@ -189,70 +189,70 @@ 
  *
  * @return void
  */
-function give_process_register_form( $data ) {
+function give_process_register_form($data) {
 
-	if ( is_user_logged_in() ) {
+	if (is_user_logged_in()) {
 		return;
 	}
 
-	if ( empty( $_POST['give_register_submit'] ) ) {
+	if (empty($_POST['give_register_submit'])) {
 		return;
 	}
 
-	do_action( 'give_pre_process_register_form' );
+	do_action('give_pre_process_register_form');
 
-	if ( empty( $data['give_user_login'] ) ) {
-		give_set_error( 'empty_username', __( 'Invalid username.', 'give' ) );
+	if (empty($data['give_user_login'])) {
+		give_set_error('empty_username', __('Invalid username.', 'give'));
 	}
 
-	if ( username_exists( $data['give_user_login'] ) ) {
-		give_set_error( 'username_unavailable', __( 'Username already taken.', 'give' ) );
+	if (username_exists($data['give_user_login'])) {
+		give_set_error('username_unavailable', __('Username already taken.', 'give'));
 	}
 
-	if ( ! validate_username( $data['give_user_login'] ) ) {
-		give_set_error( 'username_invalid', __( 'Invalid username.', 'give' ) );
+	if ( ! validate_username($data['give_user_login'])) {
+		give_set_error('username_invalid', __('Invalid username.', 'give'));
 	}
 
-	if ( email_exists( $data['give_user_email'] ) ) {
-		give_set_error( 'email_unavailable', __( 'Email address already taken.', 'give' ) );
+	if (email_exists($data['give_user_email'])) {
+		give_set_error('email_unavailable', __('Email address already taken.', 'give'));
 	}
 
-	if ( empty( $data['give_user_email'] ) || ! is_email( $data['give_user_email'] ) ) {
-		give_set_error( 'email_invalid', __( 'Invalid email.', 'give' ) );
+	if (empty($data['give_user_email']) || ! is_email($data['give_user_email'])) {
+		give_set_error('email_invalid', __('Invalid email.', 'give'));
 	}
 
-	if ( ! empty( $data['give_payment_email'] ) && $data['give_payment_email'] != $data['give_user_email'] && ! is_email( $data['give_payment_email'] ) ) {
-		give_set_error( 'payment_email_invalid', __( 'Invalid payment email.', 'give' ) );
+	if ( ! empty($data['give_payment_email']) && $data['give_payment_email'] != $data['give_user_email'] && ! is_email($data['give_payment_email'])) {
+		give_set_error('payment_email_invalid', __('Invalid payment email.', 'give'));
 	}
 
-	if ( empty( $_POST['give_user_pass'] ) ) {
-		give_set_error( 'empty_password', __( 'Please enter a password.', 'give' ) );
+	if (empty($_POST['give_user_pass'])) {
+		give_set_error('empty_password', __('Please enter a password.', 'give'));
 	}
 
-	if ( ( ! empty( $_POST['give_user_pass'] ) && empty( $_POST['give_user_pass2'] ) ) || ( $_POST['give_user_pass'] !== $_POST['give_user_pass2'] ) ) {
-		give_set_error( 'password_mismatch', __( 'Passwords don\'t match.', 'give' ) );
+	if (( ! empty($_POST['give_user_pass']) && empty($_POST['give_user_pass2'])) || ($_POST['give_user_pass'] !== $_POST['give_user_pass2'])) {
+		give_set_error('password_mismatch', __('Passwords don\'t match.', 'give'));
 	}
 
-	do_action( 'give_process_register_form' );
+	do_action('give_process_register_form');
 
 	// Check for errors and redirect if none present
 	$errors = give_get_errors();
 
-	if ( empty( $errors ) ) {
+	if (empty($errors)) {
 
-		$redirect = apply_filters( 'give_register_redirect', $data['give_redirect'] );
+		$redirect = apply_filters('give_register_redirect', $data['give_redirect']);
 
-		give_register_and_login_new_user( array(
+		give_register_and_login_new_user(array(
 			'user_login'      => $data['give_user_login'],
 			'user_pass'       => $data['give_user_pass'],
 			'user_email'      => $data['give_user_email'],
-			'user_registered' => date( 'Y-m-d H:i:s' ),
-			'role'            => get_option( 'default_role' )
-		) );
+			'user_registered' => date('Y-m-d H:i:s'),
+			'role'            => get_option('default_role')
+		));
 
-		wp_redirect( $redirect );
+		wp_redirect($redirect);
 		give_die();
 	}
 }
 
-add_action( 'give_user_register', 'give_process_register_form' );
\ No newline at end of file
+add_action('give_user_register', 'give_process_register_form');
\ No newline at end of file

includes/api/class-give-api.php

Indentation +12 added lines, -12 removed lines

@@ -1437,7 +1437,7 @@ 
 	 * Determines whether results should be displayed in XML or JSON
 	 *
 	 * @since 1.1
-     * @access public
+	 * @access public
 	 *
 	 * @return mixed|void
 	 */
@@ -1455,7 +1455,7 @@ 
 	 *
 	 * @access private
 	 * @since  1.1
-     *
+	 *
 	 * @global Give_Logging $give_logs
 	 * @global WP_Query     $wp_query
 	 *
@@ -1468,15 +1468,15 @@ 
 			return;
 		}
 
-        /**
-         * @var Give_Logging $give_logs
-         */
+		/**
+		 * @var Give_Logging $give_logs
+		 */
 		global $give_logs;
 
-        /**
-         * @var WP_Query $wp_query
-         */
-        global $wp_query;
+		/**
+		 * @var WP_Query $wp_query
+		 */
+		global $wp_query;
 
 		$query = array(
 			'give-api'    => $wp_query->query_vars['give-api'],
@@ -1534,9 +1534,9 @@ 
 	 * @param int $status_code
 	 */
 	public function output( $status_code = 200 ) {
-        /**
-         * @var WP_Query $wp_query
-         */
+		/**
+		 * @var WP_Query $wp_query
+		 */
 		global $wp_query;
 
 		$format = $this->get_output_format();

includes/admin/shortcodes/shortcode-give-login.php

Indentation +10 added lines, -10 removed lines

@@ -42,16 +42,16 @@
 				'minWidth' => 320,
 				'tooltip'  => __( 'Enter an URL here to redirect to after login.', 'give' ),
 			),
-            array(
-                'type' => 'container',
-                'html' => sprintf( '<p class="no-margin">%s</p>', __( 'Logout Redirect URL (optional):', 'give' ) ),
-            ),
-            array(
-                'type'     => 'textbox',
-                'name'     => 'logout-redirect',
-                'minWidth' => 320,
-                'tooltip'  => __( 'Enter an URL here to redirect to after logout.', 'give' ),
-            ),
+			array(
+				'type' => 'container',
+				'html' => sprintf( '<p class="no-margin">%s</p>', __( 'Logout Redirect URL (optional):', 'give' ) ),
+			),
+			array(
+				'type'     => 'textbox',
+				'name'     => 'logout-redirect',
+				'minWidth' => 320,
+				'tooltip'  => __( 'Enter an URL here to redirect to after logout.', 'give' ),
+			),
 		);
 	}
 }

Spacing +8 added lines, -8 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.3.0
  */
 
-defined( 'ABSPATH' ) or exit;
+defined('ABSPATH') or exit;
 
 class Give_Shortcode_Login extends Give_Shortcode_Generator {
 
@@ -18,10 +18,10 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['title'] = __( 'Login', 'give' );
-		$this->shortcode['label'] = __( 'Login', 'give' );
+		$this->shortcode['title'] = __('Login', 'give');
+		$this->shortcode['label'] = __('Login', 'give');
 
-		parent::__construct( 'give_login' );
+		parent::__construct('give_login');
 	}
 
 	/**
@@ -34,23 +34,23 @@ 
 		return array(
 			array(
 				'type' => 'container',
-				'html' => sprintf( '<p class="no-margin">%s</p>', __( 'Login Redirect URL (optional):', 'give' ) ),
+				'html' => sprintf('<p class="no-margin">%s</p>', __('Login Redirect URL (optional):', 'give')),
 			),
 			array(
 				'type'     => 'textbox',
 				'name'     => 'login-redirect',
 				'minWidth' => 320,
-				'tooltip'  => __( 'Enter an URL here to redirect to after login.', 'give' ),
+				'tooltip'  => __('Enter an URL here to redirect to after login.', 'give'),
 			),
             array(
                 'type' => 'container',
-                'html' => sprintf( '<p class="no-margin">%s</p>', __( 'Logout Redirect URL (optional):', 'give' ) ),
+                'html' => sprintf('<p class="no-margin">%s</p>', __('Logout Redirect URL (optional):', 'give')),
             ),
             array(
                 'type'     => 'textbox',
                 'name'     => 'logout-redirect',
                 'minWidth' => 320,
-                'tooltip'  => __( 'Enter an URL here to redirect to after logout.', 'give' ),
+                'tooltip'  => __('Enter an URL here to redirect to after logout.', 'give'),
             ),
 		);
 	}

includes/shortcodes.php

Indentation +5 added lines, -5 removed lines

@@ -91,7 +91,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 ] = '';
@@ -169,15 +169,15 @@ 
  */
 function give_login_form_shortcode( $atts, $content = null ) {
 	$atts = shortcode_atts( array(
-        // Add backward compatibility for redirect attribute.
-        'redirect'          => '',
+		// Add backward compatibility for redirect attribute.
+		'redirect'          => '',
 
 		'login-redirect'    => '',
 		'logout-redirect'   => '',
 	), $atts, 'give_login' );
 
-    // Check login-redirect attribute first, if it empty or not found then check for redirect attribute and add value of this to login-redirect attribute.
-    $atts['login-redirect'] = ! empty( $atts['login-redirect'] ) ? $atts['login-redirect'] : ( ! empty( $atts['redirect' ] ) ? $atts['redirect'] : '' );
+	// Check login-redirect attribute first, if it empty or not found then check for redirect attribute and add value of this to login-redirect attribute.
+	$atts['login-redirect'] = ! empty( $atts['login-redirect'] ) ? $atts['login-redirect'] : ( ! empty( $atts['redirect' ] ) ? $atts['redirect'] : '' );
 
 	return give_login_form( $atts['login-redirect'], $atts['logout-redirect'] );
 }

Spacing +109 added lines, -109 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,22 +167,22 @@ 
  * @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(
         // Add backward compatibility for redirect attribute.
         'redirect'          => '',
 
 		'login-redirect'    => '',
 		'logout-redirect'   => '',
-	), $atts, 'give_login' );
+	), $atts, 'give_login');
 
     // Check login-redirect attribute first, if it empty or not found then check for redirect attribute and add value of this to login-redirect attribute.
-    $atts['login-redirect'] = ! empty( $atts['login-redirect'] ) ? $atts['login-redirect'] : ( ! empty( $atts['redirect' ] ) ? $atts['redirect'] : '' );
+    $atts['login-redirect'] = ! empty($atts['login-redirect']) ? $atts['login-redirect'] : ( ! empty($atts['redirect']) ? $atts['redirect'] : '');
 
-	return give_login_form( $atts['login-redirect'], $atts['logout-redirect'] );
+	return give_login_form($atts['login-redirect'], $atts['logout-redirect']);
 }
 
-add_shortcode( 'give_login', 'give_login_form_shortcode' );
+add_shortcode('give_login', 'give_login_form_shortcode');
 
 /**
  * Register Shortcode
@@ -197,15 +197,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');
 
 
 /**
@@ -220,62 +220,62 @@ 
  *
  * @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,
 		'donor'          => 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;
 
@@ -283,9 +283,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();
 
@@ -302,13 +302,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();
 
@@ -317,7 +317,7 @@ 
 
 }
 
-add_shortcode( 'give_receipt', 'give_receipt_shortcode' );
+add_shortcode('give_receipt', 'give_receipt_shortcode');
 
 /**
  * Profile Editor Shortcode
@@ -337,18 +337,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
@@ -361,30 +361,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,
@@ -404,45 +404,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');
 

includes/class-give-donate-form.php

Doc Comments +1 added lines, -1 removed lines

@@ -182,7 +182,7 @@
 	 *
 	 * @param  array $data Array of attributes for a donation form
 	 *
-	 * @return mixed  false if data isn't passed and class not instantiated for creation, or New Form ID
+	 * @return boolean  false if data isn't passed and class not instantiated for creation, or New Form ID
 	 */
 	public function create( $data = array() ) {
 

Spacing +100 added lines, -100 removed lines

@@ -100,12 +100,12 @@ 
 	 * @param bool $_id
 	 * @param array $_args
 	 */
-	public function __construct( $_id = false, $_args = array() ) {
+	public function __construct($_id = false, $_args = array()) {
 
 
-		$donation_form = WP_Post::get_instance( $_id );
+		$donation_form = WP_Post::get_instance($_id);
 
-		return $this->setup_donation_form( $donation_form );
+		return $this->setup_donation_form($donation_form);
 	}
 
 	/**
@@ -119,23 +119,23 @@ 
 	 *
 	 * @return bool             If the setup was successful or not
 	 */
-	private function setup_donation_form( $donation_form ) {
+	private function setup_donation_form($donation_form) {
 
-		if ( ! is_object( $donation_form ) ) {
+		if ( ! is_object($donation_form)) {
 			return false;
 		}
 
-		if ( ! is_a( $donation_form, 'WP_Post' ) ) {
+		if ( ! is_a($donation_form, 'WP_Post')) {
 			return false;
 		}
 
-		if ( 'give_forms' !== $donation_form->post_type ) {
+		if ('give_forms' !== $donation_form->post_type) {
 			return false;
 		}
 
-		foreach ( $donation_form as $key => $value ) {
+		foreach ($donation_form as $key => $value) {
 
-			switch ( $key ) {
+			switch ($key) {
 
 				default:
 					$this->$key = $value;
@@ -159,16 +159,16 @@ 
 	 * @return mixed
 	 * @throws Exception
 	 */
-	public function __get( $key ) {
+	public function __get($key) {
 
-		if ( method_exists( $this, 'get_' . $key ) ) {
+		if (method_exists($this, 'get_'.$key)) {
 
-			return call_user_func( array( $this, 'get_' . $key ) );
+			return call_user_func(array($this, 'get_'.$key));
 
 		} else {
 
 			/* translators: %s: property key */
-			return new WP_Error( 'give-form-invalid-property', sprintf( __( 'Can\'t get property %s.', 'give' ), $key ) );
+			return new WP_Error('give-form-invalid-property', sprintf(__('Can\'t get property %s.', 'give'), $key));
 
 		}
 
@@ -184,30 +184,30 @@ 
 	 *
 	 * @return mixed  false if data isn't passed and class not instantiated for creation, or New Form ID
 	 */
-	public function create( $data = array() ) {
+	public function create($data = array()) {
 
-		if ( $this->id != 0 ) {
+		if ($this->id != 0) {
 			return false;
 		}
 
 		$defaults = array(
 			'post_type'   => 'give_forms',
 			'post_status' => 'draft',
-			'post_title'  => __( 'New Donation Form', 'give' )
+			'post_title'  => __('New Donation Form', 'give')
 		);
 
-		$args = wp_parse_args( $data, $defaults );
+		$args = wp_parse_args($data, $defaults);
 
 		/**
 		 * Fired before a donation form is created
 		 *
 		 * @param array $args The post object arguments used for creation.
 		 */
-		do_action( 'give_form_pre_create', $args );
+		do_action('give_form_pre_create', $args);
 
-		$id = wp_insert_post( $args, true );
+		$id = wp_insert_post($args, true);
 
-		$donation_form = WP_Post::get_instance( $id );
+		$donation_form = WP_Post::get_instance($id);
 
 		/**
 		 * Fired after a donation form is created
@@ -215,9 +215,9 @@ 
 		 * @param int $id The post ID of the created item.
 		 * @param array $args The post object arguments used for creation.
 		 */
-		do_action( 'give_form_post_create', $id, $args );
+		do_action('give_form_post_create', $id, $args);
 
-		return $this->setup_donation_form( $donation_form );
+		return $this->setup_donation_form($donation_form);
 
 	}
 
@@ -240,7 +240,7 @@ 
 	 * @return string Name of the donation form
 	 */
 	public function get_name() {
-		return get_the_title( $this->ID );
+		return get_the_title($this->ID);
 	}
 
 	/**
@@ -251,13 +251,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 {
 
@@ -275,7 +275,7 @@ 
 		 * @param string $price The donation form price.
 		 * @param string|int $id The form ID.
 		 */
-		return apply_filters( 'give_get_set_price', $this->price, $this->ID );
+		return apply_filters('give_get_set_price', $this->price, $this->ID);
 	}
 
 	/**
@@ -286,14 +286,14 @@ 
 	 */
 	public function get_minimum_price() {
 
-		if ( ! isset( $this->minimum_price ) ) {
+		if ( ! isset($this->minimum_price)) {
 
-			$allow_custom_amount = get_post_meta( $this->ID, '_give_custom_amount', true );
-			$this->minimum_price = get_post_meta( $this->ID, '_give_custom_amount_minimum', true );
+			$allow_custom_amount = get_post_meta($this->ID, '_give_custom_amount', true);
+			$this->minimum_price = get_post_meta($this->ID, '_give_custom_amount_minimum', true);
 
-			if ( $allow_custom_amount != 'no' && $this->minimum_price ) {
+			if ($allow_custom_amount != 'no' && $this->minimum_price) {
 
-				$this->minimum_price = give_sanitize_amount( $this->minimum_price );
+				$this->minimum_price = give_sanitize_amount($this->minimum_price);
 
 			} else {
 
@@ -303,7 +303,7 @@ 
 
 		}
 
-		return apply_filters( 'give_get_set_minimum_price', $this->minimum_price, $this->ID );
+		return apply_filters('give_get_set_minimum_price', $this->minimum_price, $this->ID);
 	}
 
 	/**
@@ -314,9 +314,9 @@ 
 	 */
 	public function get_prices() {
 
-		if ( ! isset( $this->prices ) ) {
+		if ( ! isset($this->prices)) {
 
-			$this->prices = get_post_meta( $this->ID, '_give_donation_levels', true );
+			$this->prices = get_post_meta($this->ID, '_give_donation_levels', true);
 
 		}
 
@@ -328,7 +328,7 @@ 
 		 * @param array $prices The array of mulit-level prices.
 		 * @param int|string The ID of the form.
 		 */
-		return apply_filters( 'give_get_donation_levels', $this->prices, $this->ID );
+		return apply_filters('give_get_donation_levels', $this->prices, $this->ID);
 
 	}
 
@@ -340,13 +340,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 {
 
@@ -356,7 +356,7 @@ 
 
 		}
 
-		return apply_filters( 'give_get_set_goal', $this->goal, $this->ID );
+		return apply_filters('give_get_set_goal', $this->goal, $this->ID);
 
 	}
 
@@ -368,10 +368,10 @@ 
 	 */
 	public function is_single_price_mode() {
 
-		$option = get_post_meta( $this->ID, '_give_price_options_mode', true );
+		$option = get_post_meta($this->ID, '_give_price_options_mode', true);
 		$ret    = 0;
 
-		if ( empty( $option ) || $option === 'set' ) {
+		if (empty($option) || $option === 'set') {
 			$ret = 1;
 		}
 		
@@ -383,7 +383,7 @@ 
 		 * @param bool $ret Is donation form in single price mode?
 		 * @param int|string The ID of the donation form.
 		 */
-		return (bool) apply_filters( 'give_single_price_option_mode', $ret, $this->ID );
+		return (bool) apply_filters('give_single_price_option_mode', $ret, $this->ID);
 
 	}
 
@@ -395,10 +395,10 @@ 
 	 */
 	public function is_custom_price_mode() {
 
-		$option = get_post_meta( $this->ID, '_give_custom_amount', true );
+		$option = get_post_meta($this->ID, '_give_custom_amount', true);
 		$ret    = 0;
 
-		if ( $option === 'yes' ) {
+		if ($option === 'yes') {
 			$ret = 1;
 		}
 
@@ -410,7 +410,7 @@ 
 		 * @param bool $ret Is donation form in custom price mode?
 		 * @param int|string The ID of the donation form.
 		 */
-		return (bool) apply_filters( 'give_custom_price_option_mode', $ret, $this->ID );
+		return (bool) apply_filters('give_custom_price_option_mode', $ret, $this->ID);
 
 	}
 
@@ -424,10 +424,10 @@ 
 	 */
 	public function has_variable_prices() {
 
-		$option = get_post_meta( $this->ID, '_give_price_option', true );
+		$option = get_post_meta($this->ID, '_give_price_option', true);
 		$ret    = 0;
 
-		if ( $option === 'multi' ) {
+		if ($option === 'multi') {
 			$ret = 1;
 		}
 
@@ -437,7 +437,7 @@ 
 		 * @param bool $ret Does donation form have variable prices?
 		 * @param int|string The ID of the donation form.
 		 */
-		return (bool) apply_filters( 'give_has_variable_prices', $ret, $this->ID );
+		return (bool) apply_filters('give_has_variable_prices', $ret, $this->ID);
 
 	}
 
@@ -449,17 +449,17 @@ 
 	 */
 	public function get_type() {
 
-		if ( ! isset( $this->type ) ) {
+		if ( ! isset($this->type)) {
 
-			$this->type = get_post_meta( $this->ID, '_give_price_option', true );
+			$this->type = get_post_meta($this->ID, '_give_price_option', true);
 
-			if ( empty( $this->type ) ) {
+			if (empty($this->type)) {
 				$this->type = 'set';
 			}
 
 		}
 
-		return apply_filters( 'give_get_form_type', $this->type, $this->ID );
+		return apply_filters('give_get_form_type', $this->type, $this->ID);
 
 	}
 
@@ -471,15 +471,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;
 			}
@@ -499,13 +499,13 @@ 
 	 *
 	 * @return int|false  New number of total sales
 	 */
-	public function increase_sales( $quantity = 1 ) {
+	public function increase_sales($quantity = 1) {
 
-		$sales       = give_get_form_sales_stats( $this->ID );
-		$quantity    = absint( $quantity );
+		$sales       = give_get_form_sales_stats($this->ID);
+		$quantity    = absint($quantity);
 		$total_sales = $sales + $quantity;
 
-		if ( $this->update_meta( '_give_form_sales', $total_sales ) ) {
+		if ($this->update_meta('_give_form_sales', $total_sales)) {
 
 			$this->sales = $total_sales;
 
@@ -525,17 +525,17 @@ 
 	 *
 	 * @return int|false  New number of total sales
 	 */
-	public function decrease_sales( $quantity = 1 ) {
+	public function decrease_sales($quantity = 1) {
 
-		$sales = give_get_form_sales_stats( $this->ID );
+		$sales = give_get_form_sales_stats($this->ID);
 
 		// Only decrease if not already zero
-		if ( $sales > 0 ) {
+		if ($sales > 0) {
 
-			$quantity    = absint( $quantity );
+			$quantity    = absint($quantity);
 			$total_sales = $sales - $quantity;
 
-			if ( $this->update_meta( '_give_form_sales', $total_sales ) ) {
+			if ($this->update_meta('_give_form_sales', $total_sales)) {
 
 				$this->sales = $sales;
 
@@ -557,15 +557,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;
 			}
@@ -585,12 +585,12 @@ 
 	 * 
 	 * @return float|false
 	 */
-	public function increase_earnings( $amount = 0 ) {
+	public function increase_earnings($amount = 0) {
 
-		$earnings   = give_get_form_earnings_stats( $this->ID );
+		$earnings   = give_get_form_earnings_stats($this->ID);
 		$new_amount = $earnings + (float) $amount;
 
-		if ( $this->update_meta( '_give_form_earnings', $new_amount ) ) {
+		if ($this->update_meta('_give_form_earnings', $new_amount)) {
 
 			$this->earnings = $new_amount;
 
@@ -608,16 +608,16 @@ 
 	 * @since 1.0
 	 * @return float|false
 	 */
-	public function decrease_earnings( $amount ) {
+	public function decrease_earnings($amount) {
 
-		$earnings = give_get_form_earnings_stats( $this->ID );
+		$earnings = give_get_form_earnings_stats($this->ID);
 
-		if ( $earnings > 0 ) {
+		if ($earnings > 0) {
 			// Only decrease if greater than zero
 			$new_amount = $earnings - (float) $amount;
 
 
-			if ( $this->update_meta( '_give_form_earnings', $new_amount ) ) {
+			if ($this->update_meta('_give_form_earnings', $new_amount)) {
 
 				$this->earnings = $new_amount;
 
@@ -637,22 +637,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);
 
 	}
 
@@ -669,9 +669,9 @@ 
 	 */
 	public function is_close_donation_form() {
 		return (
-				'yes' === get_post_meta( $this->ID, '_give_goal_option', true ) )
-				&& ( 'yes' === get_post_meta( $this->ID, '_give_close_form_when_goal_achieved', true ) )
-				&& ( $this->get_goal() <= $this->get_earnings()
+				'yes' === get_post_meta($this->ID, '_give_goal_option', true) )
+				&& ('yes' === get_post_meta($this->ID, '_give_close_form_when_goal_achieved', true))
+				&& ($this->get_goal() <= $this->get_earnings()
 		);
 	}
 
@@ -687,29 +687,29 @@ 
 	 *
 	 * @return bool                             The result of the update query
 	 */
-	private function update_meta( $meta_key = '', $meta_value = '' ) {
+	private function update_meta($meta_key = '', $meta_value = '') {
 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		if ( empty( $meta_key ) || empty( $meta_value ) ) {
+		if (empty($meta_key) || empty($meta_value)) {
 			return false;
 		}
 
 		// Make sure if it needs to be serialized, we do
-		$meta_value = maybe_serialize( $meta_value );
+		$meta_value = maybe_serialize($meta_value);
 
-		if ( is_numeric( $meta_value ) ) {
-			$value_type = is_float( $meta_value ) ? '%f' : '%d';
+		if (is_numeric($meta_value)) {
+			$value_type = is_float($meta_value) ? '%f' : '%d';
 		} else {
 			$value_type = "'%s'";
 		}
 
-		$sql = $wpdb->prepare( "UPDATE $wpdb->postmeta SET meta_value = $value_type WHERE post_id = $this->ID AND meta_key = '%s'", $meta_value, $meta_key );
+		$sql = $wpdb->prepare("UPDATE $wpdb->postmeta SET meta_value = $value_type WHERE post_id = $this->ID AND meta_key = '%s'", $meta_value, $meta_key);
 
-		if ( $wpdb->query( $sql ) ) {
+		if ($wpdb->query($sql)) {
 
-			clean_post_cache( $this->ID );
+			clean_post_cache($this->ID);
 
 			return true;
 

includes/class-give-logging.php

Doc Comments +3 added lines, -3 removed lines

@@ -214,7 +214,7 @@ 
 	 * @param array $log_data Log entry data
 	 * @param array $log_meta Log entry meta
 	 *
-	 * @return bool True if successful, false otherwise
+	 * @return boolean|null True if successful, false otherwise
 	 */
 	public function update_log( $log_data = array(), $log_meta = array() ) {
 
@@ -392,13 +392,13 @@ 
  * @param string $title
  * @param string $message
  * @param int $parent
- * @param null $type
+ * @param string $type
  *
  * @global $give_logs GIVE Logs Object
  *
  * @uses Give_Logging::add()
  *
- * @return mixed ID of the new log entry
+ * @return integer ID of the new log entry
  */
 function give_record_log( $title = '', $message = '', $parent = 0, $type = null ) {
 	/* @var Give_Logging $give_logs */

includes/class-give-session.php

Doc Comments +2 added lines, -3 removed lines

@@ -175,10 +175,9 @@
 	 * @access public
 	 * @since 1.0
 	 *
-	 * @param $key $_SESSION key
-	 * @param $value $_SESSION variable
 	 *
-	 * @return mixed Session variable
+	 * @param string $key
+	 * @return string Session variable
 	 */
 	public function set( $key, $value ) {
 

Indentation +8 added lines, -8 removed lines

@@ -171,7 +171,7 @@ 
 
 	/**
 	 * Set a session variable
-     *
+	 *
 	 * @access public
 	 * @since 1.0
 	 *
@@ -294,7 +294,7 @@ 
 	/**
 	 * Determines if we should start sessions
 	 *
-     * @access public
+	 * @access public
 	 * @since  1.4
 	 * @return bool
 	 */
@@ -332,8 +332,8 @@ 
 	/**
 	 * Maybe Start Session
 	 * Starts a new session if one hasn't started yet.
-     *
-     * @access public
+	 *
+	 * @access public
 	 * @see    http://php.net/manual/en/function.session-set-cookie-params.php
 	 */
 	public function maybe_start_session() {
@@ -352,10 +352,10 @@ 
 	/**
 	 * Get Session Expiration
 	 * Looks at the session cookies and returns the expiration date for this session if applicable
-     *
-     * @access public
-     *
-     * @return string Formatted date string
+	 *
+	 * @access public
+	 *
+	 * @return string Formatted date string
 	 */
 	public function get_session_expiration() {
 

includes/formatting.php

Doc Comments +3 added lines, -3 removed lines

@@ -163,7 +163,7 @@ 
  * @use  give_get_price_thousand_separator Get thousand separator.
  *
  * @param string $amount formatted amount number.
- * @return float|string  formatted amount number with large number names.
+ * @return string  formatted amount number with large number names.
  */
 function give_human_format_large_amount( $amount ) {
 
@@ -220,7 +220,7 @@ 
  * @param $field_args
  * @param $field
  *
- * @return bool
+ * @return false|null
  */
 function give_format_admin_multilevel_amount( $field_args, $field ) {
 
@@ -381,7 +381,7 @@ 
  * @param array  $field_args
  * @param object $field
  *
- * @return mixed
+ * @return string
  */
 function give_sanitize_thousand_separator( $value, $field_args, $field ){
     return $value;

Indentation +48 added lines, -48 removed lines

@@ -23,7 +23,7 @@ 
  * @return mixed
  */
 function give_get_price_decimals() {
-    return apply_filters( 'give_sanitize_amount_decimals', 2 );
+	return apply_filters( 'give_sanitize_amount_decimals', 2 );
 }
 
 /**
@@ -34,7 +34,7 @@ 
  * @return mixed
  */
 function give_get_price_thousand_separator() {
-    return give_get_option( 'thousands_separator', ',' );
+	return give_get_option( 'thousands_separator', ',' );
 }
 
 /**
@@ -45,7 +45,7 @@ 
  * @return mixed
  */
 function give_get_price_decimal_separator() {
-    return give_get_option( 'decimal_separator', '.' );
+	return give_get_option( 'decimal_separator', '.' );
 }
 
 /**
@@ -62,39 +62,39 @@ 
  * @return string $amount Newly sanitized amount
  */
 function give_sanitize_amount( $number, $trim_zeros = false ) {
-    $thousand_separator = give_get_price_thousand_separator();
+	$thousand_separator = give_get_price_thousand_separator();
 
-    $locale   = localeconv();
-    $decimals = array( give_get_price_decimal_separator(), $locale['decimal_point'], $locale['mon_decimal_point'] );
+	$locale   = localeconv();
+	$decimals = array( give_get_price_decimal_separator(), $locale['decimal_point'], $locale['mon_decimal_point'] );
 
-    // Remove locale from string
-    if ( ! is_float( $number ) ) {
-        $number = str_replace( $decimals, '.', $number );
-    }
+	// Remove locale from string
+	if ( ! is_float( $number ) ) {
+		$number = str_replace( $decimals, '.', $number );
+	}
 
-    // Remove thousand amount formatting if amount has.
-    // This condition use to add backward compatibility to version before 1.6, because before version 1.6 we were saving formatted amount to db.
-    if( false !== strpos( $number, $thousand_separator ) ) {
-        $number = str_replace( $thousand_separator, '', $number );
-    }
+	// Remove thousand amount formatting if amount has.
+	// This condition use to add backward compatibility to version before 1.6, because before version 1.6 we were saving formatted amount to db.
+	if( false !== strpos( $number, $thousand_separator ) ) {
+		$number = str_replace( $thousand_separator, '', $number );
+	}
 
-    // Remove non numeric entity before decimal separator.
-    $number   = preg_replace( '/[^0-9\.]/', '', $number );
+	// Remove non numeric entity before decimal separator.
+	$number   = preg_replace( '/[^0-9\.]/', '', $number );
 
-    $decimals = give_get_price_decimals();
-    $decimals = apply_filters( 'give_sanitize_amount_decimals', $decimals, $number );
+	$decimals = give_get_price_decimals();
+	$decimals = apply_filters( 'give_sanitize_amount_decimals', $decimals, $number );
 
-    $number = number_format( floatval( $number ), $decimals, '.', '' );
+	$number = number_format( floatval( $number ), $decimals, '.', '' );
 
-    // Reset negative amount to zero.
+	// Reset negative amount to zero.
 	if ( 0 > $number ) {
 		$number = number_format( 0, 2, '.' );
 	}
 
-    // Trim zeros.
-    if ( $trim_zeros && strstr( $number, '.' ) ) {
-        $number = rtrim( rtrim( $number, '0' ), '.' );
-    }
+	// Trim zeros.
+	if ( $trim_zeros && strstr( $number, '.' ) ) {
+		$number = rtrim( rtrim( $number, '0' ), '.' );
+	}
 
 	return apply_filters( 'give_sanitize_amount', $number );
 }
@@ -167,28 +167,28 @@ 
  */
 function give_human_format_large_amount( $amount ) {
 
-    // Get thousand separator.
-    $thousands_sep = give_get_price_thousand_separator();
+	// Get thousand separator.
+	$thousands_sep = give_get_price_thousand_separator();
 
-    // Sanitize amount.
-    $sanitize_amount = give_sanitize_amount( $amount );
+	// Sanitize amount.
+	$sanitize_amount = give_sanitize_amount( $amount );
 
-    // Explode amount to calculate name of large numbers.
+	// Explode amount to calculate name of large numbers.
 	$amount_array = explode( $thousands_sep, $amount );
 
-    // Calculate amount parts count.
-    $amount_count_parts = count( $amount_array );
+	// Calculate amount parts count.
+	$amount_count_parts = count( $amount_array );
 
-    // Calculate large number formatted amount.
-    if ( 4 < $amount_count_parts ){
-        $sanitize_amount =  sprintf( __( '%s trillion', 'give' ), round( ( $sanitize_amount / 1000000000000 ), 2 ) );
-    } elseif ( 3 < $amount_count_parts ){
-        $sanitize_amount =  sprintf( __( '%s billion', 'give' ), round( ( $sanitize_amount / 1000000000 ), 2 ));
-    } elseif ( 2 < $amount_count_parts  ) {
-        $sanitize_amount =  sprintf( __( '%s million', 'give' ), round( ( $sanitize_amount / 1000000), 2 ) );
-    }
+	// Calculate large number formatted amount.
+	if ( 4 < $amount_count_parts ){
+		$sanitize_amount =  sprintf( __( '%s trillion', 'give' ), round( ( $sanitize_amount / 1000000000000 ), 2 ) );
+	} elseif ( 3 < $amount_count_parts ){
+		$sanitize_amount =  sprintf( __( '%s billion', 'give' ), round( ( $sanitize_amount / 1000000000 ), 2 ));
+	} elseif ( 2 < $amount_count_parts  ) {
+		$sanitize_amount =  sprintf( __( '%s million', 'give' ), round( ( $sanitize_amount / 1000000), 2 ) );
+	}
 
-    return apply_filters( 'give_human_format_large_amount', $sanitize_amount, $amount );
+	return apply_filters( 'give_human_format_large_amount', $sanitize_amount, $amount );
 }
 
 /**
@@ -201,14 +201,14 @@ 
  * @return string $amount Newly formatted amount or Price Not Available
  */
 function give_format_decimal( $amount ){
-    $decimal_separator = give_get_price_decimal_separator();
-    $formatted_amount  = give_sanitize_amount( $amount );
+	$decimal_separator = give_get_price_decimal_separator();
+	$formatted_amount  = give_sanitize_amount( $amount );
 
-    if( false !== strpos( $formatted_amount, '.' ) ) {
-        $formatted_amount = str_replace( '.', $decimal_separator, $formatted_amount );
-    }
+	if( false !== strpos( $formatted_amount, '.' ) ) {
+		$formatted_amount = str_replace( '.', $decimal_separator, $formatted_amount );
+	}
 
-    return apply_filters( 'give_format_decimal', $formatted_amount, $amount, $decimal_separator );
+	return apply_filters( 'give_format_decimal', $formatted_amount, $amount, $decimal_separator );
 }
 
 
@@ -384,5 +384,5 @@ 
  * @return mixed
  */
 function give_sanitize_thousand_separator( $value, $field_args, $field ){
-    return $value;
+	return $value;
 }
\ No newline at end of file

Spacing +81 added lines, -81 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,7 +23,7 @@ 
  * @return mixed
  */
 function give_get_price_decimals() {
-    return apply_filters( 'give_sanitize_amount_decimals', 2 );
+    return apply_filters('give_sanitize_amount_decimals', 2);
 }
 
 /**
@@ -34,7 +34,7 @@ 
  * @return mixed
  */
 function give_get_price_thousand_separator() {
-    return give_get_option( 'thousands_separator', ',' );
+    return give_get_option('thousands_separator', ',');
 }
 
 /**
@@ -45,7 +45,7 @@ 
  * @return mixed
  */
 function give_get_price_decimal_separator() {
-    return give_get_option( 'decimal_separator', '.' );
+    return give_get_option('decimal_separator', '.');
 }
 
 /**
@@ -61,42 +61,42 @@ 
  *
  * @return string $amount Newly sanitized amount
  */
-function give_sanitize_amount( $number, $trim_zeros = false ) {
+function give_sanitize_amount($number, $trim_zeros = false) {
     $thousand_separator = give_get_price_thousand_separator();
 
     $locale   = localeconv();
-    $decimals = array( give_get_price_decimal_separator(), $locale['decimal_point'], $locale['mon_decimal_point'] );
+    $decimals = array(give_get_price_decimal_separator(), $locale['decimal_point'], $locale['mon_decimal_point']);
 
     // Remove locale from string
-    if ( ! is_float( $number ) ) {
-        $number = str_replace( $decimals, '.', $number );
+    if ( ! is_float($number)) {
+        $number = str_replace($decimals, '.', $number);
     }
 
     // Remove thousand amount formatting if amount has.
     // This condition use to add backward compatibility to version before 1.6, because before version 1.6 we were saving formatted amount to db.
-    if( false !== strpos( $number, $thousand_separator ) ) {
-        $number = str_replace( $thousand_separator, '', $number );
+    if (false !== strpos($number, $thousand_separator)) {
+        $number = str_replace($thousand_separator, '', $number);
     }
 
     // Remove non numeric entity before decimal separator.
-    $number   = preg_replace( '/[^0-9\.]/', '', $number );
+    $number   = preg_replace('/[^0-9\.]/', '', $number);
 
     $decimals = give_get_price_decimals();
-    $decimals = apply_filters( 'give_sanitize_amount_decimals', $decimals, $number );
+    $decimals = apply_filters('give_sanitize_amount_decimals', $decimals, $number);
 
-    $number = number_format( floatval( $number ), $decimals, '.', '' );
+    $number = number_format(floatval($number), $decimals, '.', '');
 
     // Reset negative amount to zero.
-	if ( 0 > $number ) {
-		$number = number_format( 0, 2, '.' );
+	if (0 > $number) {
+		$number = number_format(0, 2, '.');
 	}
 
     // Trim zeros.
-    if ( $trim_zeros && strstr( $number, '.' ) ) {
-        $number = rtrim( rtrim( $number, '0' ), '.' );
+    if ($trim_zeros && strstr($number, '.')) {
+        $number = rtrim(rtrim($number, '0'), '.');
     }
 
-	return apply_filters( 'give_sanitize_amount', $number );
+	return apply_filters('give_sanitize_amount', $number);
 }
 
 /**
@@ -109,47 +109,47 @@ 
  *
  * @return string $amount Newly formatted amount or Price Not Available
  */
-function give_format_amount( $amount, $decimals = true ) {
+function give_format_amount($amount, $decimals = true) {
 
-	$thousands_sep = give_get_option( 'thousands_separator', ',' );
-	$decimal_sep   = give_get_option( 'decimal_separator', '.' );
+	$thousands_sep = give_get_option('thousands_separator', ',');
+	$decimal_sep   = give_get_option('decimal_separator', '.');
 
 	// Format the amount
-	if ( $decimal_sep == ',' && false !== ( $sep_found = strpos( $amount, $decimal_sep ) ) ) {
-		$whole  = substr( $amount, 0, $sep_found );
-		$part   = substr( $amount, $sep_found + 1, ( strlen( $amount ) - 1 ) );
-		$amount = $whole . '.' . $part;
+	if ($decimal_sep == ',' && false !== ($sep_found = strpos($amount, $decimal_sep))) {
+		$whole  = substr($amount, 0, $sep_found);
+		$part   = substr($amount, $sep_found + 1, (strlen($amount) - 1));
+		$amount = $whole.'.'.$part;
 	}
 
 	// Strip , from the amount (if set as the thousands separator)
-	if ( $thousands_sep == ',' && false !== ( $found = strpos( $amount, $thousands_sep ) ) ) {
-		$amount = str_replace( ',', '', $amount );
+	if ($thousands_sep == ',' && false !== ($found = strpos($amount, $thousands_sep))) {
+		$amount = str_replace(',', '', $amount);
 	}
 
 	// Strip . from the amount (if set as the thousands separator) AND , set to decimal separator
-	if ( $thousands_sep == '.' && $decimal_sep == ',' && false !== ( $found = strpos( $amount, $thousands_sep ) ) ) {
-		$amount      = explode( '.', $amount );
-		$array_count = count( $amount );
-		if ( $decimals == true ) {
-			unset( $amount[ $array_count - 1 ] );
+	if ($thousands_sep == '.' && $decimal_sep == ',' && false !== ($found = strpos($amount, $thousands_sep))) {
+		$amount      = explode('.', $amount);
+		$array_count = count($amount);
+		if ($decimals == true) {
+			unset($amount[$array_count - 1]);
 		}
-		$amount = implode( '', $amount );
+		$amount = implode('', $amount);
 	}
 
 	// Strip ' ' from the amount (if set as the thousands separator)
-	if ( $thousands_sep == ' ' && false !== ( $found = strpos( $amount, $thousands_sep ) ) ) {
-		$amount = str_replace( ' ', '', $amount );
+	if ($thousands_sep == ' ' && false !== ($found = strpos($amount, $thousands_sep))) {
+		$amount = str_replace(' ', '', $amount);
 	}
 
-	if ( empty( $amount ) ) {
+	if (empty($amount)) {
 		$amount = 0;
 	}
 
 	$decimals = give_get_price_decimals();
 
-	$formatted = number_format( $amount, $decimals, $decimal_sep, $thousands_sep );
+	$formatted = number_format($amount, $decimals, $decimal_sep, $thousands_sep);
 
-	return apply_filters( 'give_format_amount', $formatted, $amount, $decimals, $decimal_sep, $thousands_sep );
+	return apply_filters('give_format_amount', $formatted, $amount, $decimals, $decimal_sep, $thousands_sep);
 }
 
 
@@ -165,30 +165,30 @@ 
  * @param string $amount formatted amount number.
  * @return float|string  formatted amount number with large number names.
  */
-function give_human_format_large_amount( $amount ) {
+function give_human_format_large_amount($amount) {
 
     // Get thousand separator.
     $thousands_sep = give_get_price_thousand_separator();
 
     // Sanitize amount.
-    $sanitize_amount = give_sanitize_amount( $amount );
+    $sanitize_amount = give_sanitize_amount($amount);
 
     // Explode amount to calculate name of large numbers.
-	$amount_array = explode( $thousands_sep, $amount );
+	$amount_array = explode($thousands_sep, $amount);
 
     // Calculate amount parts count.
-    $amount_count_parts = count( $amount_array );
+    $amount_count_parts = count($amount_array);
 
     // Calculate large number formatted amount.
-    if ( 4 < $amount_count_parts ){
-        $sanitize_amount =  sprintf( __( '%s trillion', 'give' ), round( ( $sanitize_amount / 1000000000000 ), 2 ) );
-    } elseif ( 3 < $amount_count_parts ){
-        $sanitize_amount =  sprintf( __( '%s billion', 'give' ), round( ( $sanitize_amount / 1000000000 ), 2 ));
-    } elseif ( 2 < $amount_count_parts  ) {
-        $sanitize_amount =  sprintf( __( '%s million', 'give' ), round( ( $sanitize_amount / 1000000), 2 ) );
+    if (4 < $amount_count_parts) {
+        $sanitize_amount = sprintf(__('%s trillion', 'give'), round(($sanitize_amount / 1000000000000), 2));
+    } elseif (3 < $amount_count_parts) {
+        $sanitize_amount = sprintf(__('%s billion', 'give'), round(($sanitize_amount / 1000000000), 2));
+    } elseif (2 < $amount_count_parts) {
+        $sanitize_amount = sprintf(__('%s million', 'give'), round(($sanitize_amount / 1000000), 2));
     }
 
-    return apply_filters( 'give_human_format_large_amount', $sanitize_amount, $amount );
+    return apply_filters('give_human_format_large_amount', $sanitize_amount, $amount);
 }
 
 /**
@@ -200,15 +200,15 @@ 
  *
  * @return string $amount Newly formatted amount or Price Not Available
  */
-function give_format_decimal( $amount ){
+function give_format_decimal($amount) {
     $decimal_separator = give_get_price_decimal_separator();
-    $formatted_amount  = give_sanitize_amount( $amount );
+    $formatted_amount  = give_sanitize_amount($amount);
 
-    if( false !== strpos( $formatted_amount, '.' ) ) {
-        $formatted_amount = str_replace( '.', $decimal_separator, $formatted_amount );
+    if (false !== strpos($formatted_amount, '.')) {
+        $formatted_amount = str_replace('.', $decimal_separator, $formatted_amount);
     }
 
-    return apply_filters( 'give_format_decimal', $formatted_amount, $amount, $decimal_separator );
+    return apply_filters('give_format_decimal', $formatted_amount, $amount, $decimal_separator);
 }
 
 
@@ -222,13 +222,13 @@ 
  *
  * @return bool
  */
-function give_format_admin_multilevel_amount( $field_args, $field ) {
+function give_format_admin_multilevel_amount($field_args, $field) {
 
-	if ( empty( $field->value ) ) {
+	if (empty($field->value)) {
 		return false;
 	}
 
-	$field->value = give_format_decimal( $field->value );
+	$field->value = give_format_decimal($field->value);
 }
 
 /**
@@ -241,24 +241,24 @@ 
  *
  * @return mixed|string|void
  */
-function give_currency_filter( $price = '', $currency = '' ) {
+function give_currency_filter($price = '', $currency = '') {
 
-	if ( empty( $currency ) ) {
+	if (empty($currency)) {
 		$currency = give_get_currency();
 	}
 
-	$position = give_get_option( 'currency_position', 'before' );
+	$position = give_get_option('currency_position', 'before');
 
 	$negative = $price < 0;
 
-	if ( $negative ) {
-		$price = substr( $price, 1 ); // Remove proceeding "-" -
+	if ($negative) {
+		$price = substr($price, 1); // Remove proceeding "-" -
 	}
 
-	$symbol = give_currency_symbol( $currency );
+	$symbol = give_currency_symbol($currency);
 
-	if ( $position == 'before' ):
-		switch ( $currency ):
+	if ($position == 'before'):
+		switch ($currency):
 			case 'GBP' :
 			case 'BRL' :
 			case 'EUR' :
@@ -287,18 +287,18 @@ 
 			case 'MAD' :
 			case 'KRW' :
 			case 'ZAR' :
-				$formatted = $symbol . $price;
+				$formatted = $symbol.$price;
 				break;
 			case 'NOK' :
-				$formatted = $symbol . ' ' . $price;
+				$formatted = $symbol.' '.$price;
 				break;
 			default :
-				$formatted = $currency . ' ' . $price;
+				$formatted = $currency.' '.$price;
 				break;
 		endswitch;
-		$formatted = apply_filters( 'give_' . strtolower( $currency ) . '_currency_filter_before', $formatted, $currency, $price );
+		$formatted = apply_filters('give_'.strtolower($currency).'_currency_filter_before', $formatted, $currency, $price);
 	else :
-		switch ( $currency ) :
+		switch ($currency) :
 			case 'GBP' :
 			case 'BRL' :
 			case 'EUR' :
@@ -326,18 +326,18 @@ 
 			case 'MAD' :
 			case 'KRW' :
 			case 'ZAR' :
-				$formatted = $price . $symbol;
+				$formatted = $price.$symbol;
 				break;
 			default :
-				$formatted = $price . ' ' . $currency;
+				$formatted = $price.' '.$currency;
 				break;
 		endswitch;
-		$formatted = apply_filters( 'give_' . strtolower( $currency ) . '_currency_filter_after', $formatted, $currency, $price );
+		$formatted = apply_filters('give_'.strtolower($currency).'_currency_filter_after', $formatted, $currency, $price);
 	endif;
 
-	if ( $negative ) {
+	if ($negative) {
 		// Prepend the mins sign before the currency sign
-		$formatted = '-' . $formatted;
+		$formatted = '-'.$formatted;
 	}
 
 	return $formatted;
@@ -352,11 +352,11 @@ 
  *
  * @return int $decimals
  */
-function give_currency_decimal_filter( $decimals = 2 ) {
+function give_currency_decimal_filter($decimals = 2) {
 
 	$currency = give_get_currency();
 
-	switch ( $currency ) {
+	switch ($currency) {
 		case 'RIAL' :
 		case 'JPY' :
 		case 'TWD' :
@@ -366,11 +366,11 @@ 
 			break;
 	}
 
-	return apply_filters( 'give_currency_decimal_count', $decimals, $currency );
+	return apply_filters('give_currency_decimal_count', $decimals, $currency);
 }
 
-add_filter( 'give_sanitize_amount_decimals', 'give_currency_decimal_filter' );
-add_filter( 'give_format_amount_decimals', 'give_currency_decimal_filter' );
+add_filter('give_sanitize_amount_decimals', 'give_currency_decimal_filter');
+add_filter('give_format_amount_decimals', 'give_currency_decimal_filter');
 
 /**
  * Sanitize thousand separator
@@ -383,6 +383,6 @@ 
  *
  * @return mixed
  */
-function give_sanitize_thousand_separator( $value, $field_args, $field ){
+function give_sanitize_thousand_separator($value, $field_args, $field) {
     return $value;
 }
\ No newline at end of file