gravityview / GravityView

trustedlogin/trustedlogin/trustedlogin-client/src/Logging.php

Spacing +13 added lines, -13 removed lines

@@ -68,7 +68,7 @@ 
 
 		$configured_logging_dir = $config->get_setting( 'logging/directory', '' );
 
-		if( ! empty( $configured_logging_dir ) ) {
+		if ( ! empty( $configured_logging_dir ) ) {
 
 			$logging_directory = $this->check_directory( $configured_logging_dir );
 
@@ -82,12 +82,12 @@ 
 		}
 
 		// Directory cannot be found or created. Cannot log.
-		if( ! $logging_directory ) {
+		if ( ! $logging_directory ) {
 			return false;
 		}
 
 		// Directory cannot be written to
-		if( ! $this->check_directory( $logging_directory ) ) {
+		if ( ! $this->check_directory( $logging_directory ) ) {
 			return false;
 		}
 
@@ -109,7 +109,7 @@ 
 
 			$options = wp_parse_args( $settings_options, $default_options );
 
-			$klogger = new Logger (
+			$klogger = new Logger(
 				$logging_directory,
 				$config->get_setting( 'logging/threshold', 'notice' ),
 				$options
@@ -134,21 +134,21 @@ 
 	 */
 	private function check_directory( $dirpath ) {
 
-		$dirpath = (string) $dirpath;
+		$dirpath = (string)$dirpath;
 		$file_exists = file_exists( $dirpath );
 		$is_writable = wp_is_writable( $dirpath );
 
 		// If the configured setting path exists and is writeable, use it.
-		if( $file_exists && $is_writable ) {
+		if ( $file_exists && $is_writable ) {
 			return $dirpath;
 		}
 
 		// Otherwise, try and log default errors
-		if( ! $file_exists ) {
+		if ( ! $file_exists ) {
 			$this->log( 'The defined logging directory does not exist: ' . $dirpath, __METHOD__, 'error' );
 		}
 
-		if( ! $is_writable ) {
+		if ( ! $is_writable ) {
 			$this->log( 'The defined logging directory exists but could not be written to: ' . $dirpath, __METHOD__, 'error' );
 		}
 
@@ -166,10 +166,10 @@ 
 
 		$upload_dir = wp_upload_dir();
 
-		$log_dir = trailingslashit( $upload_dir['basedir'] ) . self::DIRECTORY_PATH;
+		$log_dir = trailingslashit( $upload_dir[ 'basedir' ] ) . self::DIRECTORY_PATH;
 
 		// Directory exists; return early
-		if( file_exists( $log_dir ) ) {
+		if ( file_exists( $log_dir ) ) {
 
 			$this->prevent_directory_browsing( $log_dir );
 
@@ -180,7 +180,7 @@ 
 		$folder_created = wp_mkdir_p( $log_dir );
 
 		// Something went wrong mapping the directory
-		if( ! $folder_created ) {
+		if ( ! $folder_created ) {
 			$this->log( 'The log directory could not be created: ' . $log_dir, __METHOD__, 'error' );
 			return false;
 		}
@@ -241,7 +241,7 @@ 
 		 */
 		$is_enabled = apply_filters( 'trustedlogin/' . $this->ns . '/logging/enabled', $is_enabled );
 
-		return (bool) $is_enabled;
+		return (bool)$is_enabled;
 	}
 
 	/**
@@ -319,7 +319,7 @@ 
 			return;
 		}
 
-		$this->klogger->{$level}( $log_message, (array) $data );
+		$this->klogger->{$level}( $log_message, (array)$data );
 	}
 
 }

trustedlogin/trustedlogin/trustedlogin-client/src/ConfigException.php

Braces +1 added lines, -2 removed lines

@@ -17,6 +17,5 @@
 	exit;
 }
 
-class ConfigException extends \Exception
-{
+class ConfigException extends \Exception {
 }

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 namespace GravityView\TrustedLogin;
 
 // Exit if accessed directly
-if ( ! defined('ABSPATH') ) {
+if ( ! defined( 'ABSPATH' ) ) {
 	exit;
 }
 

trustedlogin/trustedlogin/trustedlogin-client/src/Config.php

Spacing +28 added lines, -28 removed lines

@@ -13,7 +13,7 @@ 
 namespace GravityView\TrustedLogin;
 
 // Exit if accessed directly
-if ( ! defined('ABSPATH') ) {
+if ( ! defined( 'ABSPATH' ) ) {
 	exit;
 }
 
@@ -112,51 +112,51 @@ 
 	 */
 	public function validate() {
 
-		if ( in_array( __NAMESPACE__, array( 'ReplaceMe', 'ReplaceMe\GravityView\GravityView\TrustedLogin' ) ) && ! defined('TL_DOING_TESTS') ) {
+		if ( in_array( __NAMESPACE__, array( 'ReplaceMe', 'ReplaceMe\GravityView\GravityView\TrustedLogin' ) ) && ! defined( 'TL_DOING_TESTS' ) ) {
 			throw new Exception( 'Developer: make sure to change the namespace for the TrustedLogin class. See https://trustedlogin.com/configuration/ for more information.', 501 );
 		}
 
 		$errors = array();
 
-		if ( ! isset( $this->settings['auth']['api_key'] ) ) {
-			$errors[] = new WP_Error( 'missing_configuration', 'You need to set an API key. Get yours at https://app.trustedlogin.com' );
+		if ( ! isset( $this->settings[ 'auth' ][ 'api_key' ] ) ) {
+			$errors[ ] = new WP_Error( 'missing_configuration', 'You need to set an API key. Get yours at https://app.trustedlogin.com' );
 		}
 
-		if ( isset( $this->settings['vendor']['website'] ) && 'https://www.example.com' === $this->settings['vendor']['website'] && ! defined('TL_DOING_TESTS') ) {
-			$errors[] = new WP_Error( 'missing_configuration', 'You need to configure the "website" URL to point to the URL where the Vendor plugin is installed.' );
+		if ( isset( $this->settings[ 'vendor' ][ 'website' ] ) && 'https://www.example.com' === $this->settings[ 'vendor' ][ 'website' ] && ! defined( 'TL_DOING_TESTS' ) ) {
+			$errors[ ] = new WP_Error( 'missing_configuration', 'You need to configure the "website" URL to point to the URL where the Vendor plugin is installed.' );
 		}
 
 		foreach ( array( 'namespace', 'title', 'website', 'support_url', 'email' ) as $required_vendor_field ) {
-			if ( ! isset( $this->settings['vendor'][ $required_vendor_field ] ) ) {
-				$errors[] = new WP_Error( 'missing_configuration', sprintf( 'Missing required configuration: `vendor/%s`', $required_vendor_field ) );
+			if ( ! isset( $this->settings[ 'vendor' ][ $required_vendor_field ] ) ) {
+				$errors[ ] = new WP_Error( 'missing_configuration', sprintf( 'Missing required configuration: `vendor/%s`', $required_vendor_field ) );
 			}
 		}
 
-		if ( isset( $this->settings['decay'] ) ) {
-			if ( ! is_int( $this->settings['decay'] ) ) {
-				$errors[] = new WP_Error( 'invalid_configuration', 'Decay must be an integer (number of seconds).' );
-			} elseif ( $this->settings['decay'] > MONTH_IN_SECONDS ) {
-				$errors[] = new WP_Error( 'invalid_configuration', 'Decay must be less than or equal to 30 days.' );
-			} elseif ( $this->settings['decay'] < DAY_IN_SECONDS ) {
-				$errors[] = new WP_Error( 'invalid_configuration', 'Decay must be greater than 1 day.' );
+		if ( isset( $this->settings[ 'decay' ] ) ) {
+			if ( ! is_int( $this->settings[ 'decay' ] ) ) {
+				$errors[ ] = new WP_Error( 'invalid_configuration', 'Decay must be an integer (number of seconds).' );
+			} elseif ( $this->settings[ 'decay' ] > MONTH_IN_SECONDS ) {
+				$errors[ ] = new WP_Error( 'invalid_configuration', 'Decay must be less than or equal to 30 days.' );
+			} elseif ( $this->settings[ 'decay' ] < DAY_IN_SECONDS ) {
+				$errors[ ] = new WP_Error( 'invalid_configuration', 'Decay must be greater than 1 day.' );
 			}
 		}
 
-		if ( isset( $this->settings['vendor']['namespace'] ) ) {
+		if ( isset( $this->settings[ 'vendor' ][ 'namespace' ] ) ) {
 
 			// This seems like a reasonable max limit on namespace length.
 			// @see https://developer.wordpress.org/reference/functions/set_transient/#more-information
-			if ( strlen( $this->settings['vendor']['namespace'] ) > 96 ) {
-				$errors[] = new WP_Error( 'invalid_configuration', 'Namespace length must be shorter than 96 characters.' );
+			if ( strlen( $this->settings[ 'vendor' ][ 'namespace' ] ) > 96 ) {
+				$errors[ ] = new WP_Error( 'invalid_configuration', 'Namespace length must be shorter than 96 characters.' );
 			}
 
-			if ( in_array( strtolower( $this->settings['vendor']['namespace'] ), self::$reserved_namespaces, true ) ) {
-				$errors[] = new WP_Error( 'invalid_configuration', 'The defined namespace is reserved.' );
+			if ( in_array( strtolower( $this->settings[ 'vendor' ][ 'namespace' ] ), self::$reserved_namespaces, true ) ) {
+				$errors[ ] = new WP_Error( 'invalid_configuration', 'The defined namespace is reserved.' );
 			}
 		}
 
-		if ( isset( $this->settings['vendor'][ 'email' ] ) && ! filter_var( $this->settings['vendor'][ 'email' ], FILTER_VALIDATE_EMAIL ) ) {
-			$errors[] = new WP_Error( 'invalid_configuration', 'An invalid `vendor/email` setting was passed to the TrustedLogin Client.' );
+		if ( isset( $this->settings[ 'vendor' ][ 'email' ] ) && ! filter_var( $this->settings[ 'vendor' ][ 'email' ], FILTER_VALIDATE_EMAIL ) ) {
+			$errors[ ] = new WP_Error( 'invalid_configuration', 'An invalid `vendor/email` setting was passed to the TrustedLogin Client.' );
 		}
 
 		// TODO: Add namespace collision check?
@@ -164,7 +164,7 @@ 
 			$value = $this->get_setting( $settings_key, null, $this->settings );
 			$url   = wp_kses_bad_protocol( $value, array( 'http', 'https' ) );
 			if ( $value && ! filter_var( $url, FILTER_VALIDATE_URL ) ) {
-				$errors[] = new WP_Error(
+				$errors[ ] = new WP_Error(
 					'invalid_configuration',
 					sprintf( 'An invalid `%s` setting was passed to the TrustedLogin Client: %s',
 						$settings_key,
@@ -178,7 +178,7 @@ 
 
 		foreach ( SupportRole::$prevented_caps as $invalid_cap ) {
 			if ( array_key_exists( $invalid_cap, $added_caps ) ) {
-				$errors[] = new WP_Error( 'invalid_configuration', 'TrustedLogin users cannot be allowed to: ' . $invalid_cap );
+				$errors[ ] = new WP_Error( 'invalid_configuration', 'TrustedLogin users cannot be allowed to: ' . $invalid_cap );
 			}
 		}
 
@@ -186,7 +186,7 @@ 
 			$error_text = array();
 			foreach ( $errors as $error ) {
 				if ( is_wp_error( $error ) ) {
-					$error_text[] = $error->get_error_message();
+					$error_text[ ] = $error->get_error_message();
 				}
 			}
 
@@ -221,7 +221,7 @@ 
 
 		$time = current_time( 'timestamp', $gmt );
 
-		return $time + (int) $decay_time;
+		return $time + (int)$decay_time;
 	}
 
 	/**
@@ -281,8 +281,8 @@ 
 
 		$default_settings = $this->default_settings;
 
-		$default_settings['paths']['css'] = plugin_dir_url( __FILE__ ) . 'assets/trustedlogin.css';
-		$default_settings['paths']['js']  = plugin_dir_url( __FILE__ ) . 'assets/trustedlogin.js';
+		$default_settings[ 'paths' ][ 'css' ] = plugin_dir_url( __FILE__ ) . 'assets/trustedlogin.css';
+		$default_settings[ 'paths' ][ 'js' ]  = plugin_dir_url( __FILE__ ) . 'assets/trustedlogin.js';
 
 		return $default_settings;
 	}

trustedlogin/trustedlogin/trustedlogin-client/src/SupportRole.php

Spacing +9 added lines, -9 removed lines

@@ -13,7 +13,7 @@ 
 namespace GravityView\TrustedLogin;
 
 // Exit if accessed directly
-if ( ! defined('ABSPATH') ) {
+if ( ! defined( 'ABSPATH' ) ) {
 	exit;
 }
 
@@ -72,14 +72,14 @@ 
 		// TODO: Support multiple roles
 		$role = is_array( $roles ) ? array_key_first( $roles ) : $roles;
 
-		return (string) $role;
+		return (string)$role;
 	}
 
 	/**
 	 * @return string
 	 */
 	public function get_name() {
-		return (string) $this->role_name;
+		return (string)$this->role_name;
 	}
 
 	/**
@@ -150,7 +150,7 @@ 
 
 		$add_caps = $this->config->get_setting( 'caps/add' );
 
-		foreach ( (array) $add_caps as $add_cap => $reason ) {
+		foreach ( (array)$add_caps as $add_cap => $reason ) {
 			$capabilities[ $add_cap ] = true;
 		}
 
@@ -166,7 +166,7 @@ 
 
 		$new_role = add_role( $new_role_slug, $role_display_name, $capabilities );
 
-		if ( ! $new_role ){
+		if ( ! $new_role ) {
 
 			return new WP_Error(
 				'add_role_failed',
@@ -181,11 +181,11 @@ 
 
 		$remove_caps = $this->config->get_setting( 'caps/remove' );
 
-		if ( ! empty( $remove_caps ) ){
+		if ( ! empty( $remove_caps ) ) {
 
-			foreach ( $remove_caps as $remove_cap => $description ){
+			foreach ( $remove_caps as $remove_cap => $description ) {
 				$new_role->remove_cap( $remove_cap );
-				$this->logging->log( 'Capability '. $remove_cap .' removed from role.', __METHOD__, 'info' );
+				$this->logging->log( 'Capability ' . $remove_cap . ' removed from role.', __METHOD__, 'info' );
 			}
 		}
 
@@ -204,7 +204,7 @@ 
 		// Returns void; no way to tell if successful
 		remove_role( $this->get_name() );
 
-		if( get_role( $this->get_name() ) ) {
+		if ( get_role( $this->get_name() ) ) {
 
 			$this->logging->log( "Role " . $this->get_name() . " was not removed successfully.", __METHOD__, 'error' );
 

Braces +3 added lines, -3 removed lines

@@ -166,7 +166,7 @@ 
 
 		$new_role = add_role( $new_role_slug, $role_display_name, $capabilities );
 
-		if ( ! $new_role ){
+		if ( ! $new_role ) {
 
 			return new WP_Error(
 				'add_role_failed',
@@ -181,9 +181,9 @@ 
 
 		$remove_caps = $this->config->get_setting( 'caps/remove' );
 
-		if ( ! empty( $remove_caps ) ){
+		if ( ! empty( $remove_caps ) ) {
 
-			foreach ( $remove_caps as $remove_cap => $description ){
+			foreach ( $remove_caps as $remove_cap => $description ) {
 				$new_role->remove_cap( $remove_cap );
 				$this->logging->log( 'Capability '. $remove_cap .' removed from role.', __METHOD__, 'info' );
 			}

trustedlogin/trustedlogin/trustedlogin-client/src/Client.php

Spacing +9 added lines, -9 removed lines

@@ -24,7 +24,7 @@ 
 namespace GravityView\TrustedLogin;
 
 // Exit if accessed directly
-if ( ! defined('ABSPATH') ) {
+if ( ! defined( 'ABSPATH' ) ) {
 	exit;
 }
 
@@ -159,7 +159,7 @@ 
 		$ns = $config->ns();
 
 		// Namespace isn't set; allow Config
-		if( empty( $ns ) ) {
+		if ( empty( $ns ) ) {
 			return true;
 		}
 
@@ -343,7 +343,7 @@ 
 			return $created;
 		}
 
-		$return_data['timing']['remote'] = timer_stop( 0, 5 );
+		$return_data[ 'timing' ][ 'remote' ] = timer_stop( 0, 5 );
 
 		do_action( 'trustedlogin/' . $this->config->ns() . '/access/created', array(
 			'url'    => get_site_url(),
@@ -447,7 +447,7 @@ 
 			return $updated;
 		}
 
-		$return_data['timing']['remote'] = timer_stop( 0, 5 );
+		$return_data[ 'timing' ][ 'remote' ] = timer_stop( 0, 5 );
 
 		do_action( 'trustedlogin/' . $this->config->ns() . '/access/extended', array(
 			'url'    => get_site_url(),
@@ -543,7 +543,7 @@ 
 		$reference_id = self::get_reference_id();
 
 		if ( $reference_id ) {
-			$metadata['reference_id'] = $reference_id;
+			$metadata[ 'reference_id' ] = $reference_id;
 		}
 
 		return $metadata;
@@ -558,12 +558,12 @@ 
 	 */
 	public static function get_reference_id() {
 
-		if ( isset( $_REQUEST['reference_id'] ) ) {
-			return esc_html( $_REQUEST['reference_id'] );
+		if ( isset( $_REQUEST[ 'reference_id' ] ) ) {
+			return esc_html( $_REQUEST[ 'reference_id' ] );
 		}
 
-		if ( isset( $_REQUEST['ref'] ) ) {
-			return esc_html( $_REQUEST['ref'] );
+		if ( isset( $_REQUEST[ 'ref' ] ) ) {
+			return esc_html( $_REQUEST[ 'ref' ] );
 		}
 
 		return null;

trustedlogin/trustedlogin/trustedlogin-client/src/Endpoint.php

Spacing +8 added lines, -8 removed lines

@@ -122,7 +122,7 @@ 
 		// Before logging-in support, let's make sure the site isn't locked-down or that this request is flagged
 		$is_verified = $security_checks->verify( $user_identifier );
 
-		if ( ! $is_verified || is_wp_error( $is_verified ) ){
+		if ( ! $is_verified || is_wp_error( $is_verified ) ) {
 
 			/**
 			 * Runs after the identifier fails security checks
@@ -179,11 +179,11 @@ 
 			return;
 		}
 
-		if ( ! isset( $_REQUEST['_wpnonce'] ) ) {
+		if ( ! isset( $_REQUEST[ '_wpnonce' ] ) ) {
 			return;
 		}
 
-		$verify_nonce = wp_verify_nonce( $_REQUEST['_wpnonce' ], self::REVOKE_SUPPORT_QUERY_PARAM );
+		$verify_nonce = wp_verify_nonce( $_REQUEST[ '_wpnonce' ], self::REVOKE_SUPPORT_QUERY_PARAM );
 
 		if ( ! $verify_nonce ) {
 			$this->logging->log( 'Removing user failed: Nonce expired (Nonce value: ' . $verify_nonce . ')', __METHOD__, 'error' );
@@ -268,7 +268,7 @@ 
 	 * @return string
 	 */
 	public function get() {
-		return (string) get_site_option( $this->option_name );
+		return (string)get_site_option( $this->option_name );
 	}
 
 	/**
@@ -282,19 +282,19 @@ 
 
 		$endpoint = $this->get();
 
-		if ( ! isset( $_POST['action'], $_POST['endpoint'], $_POST['identifier'] ) ) {
+		if ( ! isset( $_POST[ 'action' ], $_POST[ 'endpoint' ], $_POST[ 'identifier' ] ) ) {
 			return false;
 		}
 
-		if ( 'trustedlogin' !== $_POST['action'] ) {
+		if ( 'trustedlogin' !== $_POST[ 'action' ] ) {
 			return false;
 		}
 
-		if ( $endpoint !== $_POST['endpoint'] ) {
+		if ( $endpoint !== $_POST[ 'endpoint' ] ) {
 			return false;
 		}
 
-		return sanitize_text_field( $_POST['identifier'] );
+		return sanitize_text_field( $_POST[ 'identifier' ] );
 	}
 
 	/**

Braces +1 added lines, -1 removed lines

@@ -122,7 +122,7 @@
 		// Before logging-in support, let's make sure the site isn't locked-down or that this request is flagged
 		$is_verified = $security_checks->verify( $user_identifier );
 
-		if ( ! $is_verified || is_wp_error( $is_verified ) ){
+		if ( ! $is_verified || is_wp_error( $is_verified ) ) {
 
 			/**
 			 * Runs after the identifier fails security checks

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -245,7 +245,7 @@
 
 		add_rewrite_endpoint( $endpoint, EP_ROOT );
 
-		$this->logging->log( "Endpoint {$endpoint} added.", __METHOD__, 'debug' );
+		$this->logging->log( "endpoint {$endpoint} added.", __METHOD__, 'debug' );
 
 		if ( get_site_option( self::PERMALINK_FLUSH_OPTION_NAME ) ) {
 			return;

trustedlogin/trustedlogin/trustedlogin-client/src/Envelope.php

Spacing +3 added lines, -3 removed lines

@@ -13,7 +13,7 @@ 
 namespace GravityView\TrustedLogin;
 
 // Exit if accessed directly
-if ( ! defined('ABSPATH') ) {
+if ( ! defined( 'ABSPATH' ) ) {
 	exit;
 }
 
@@ -81,13 +81,13 @@ 
 
 		$e_keys = $this->encryption->generate_keys();
 
-		if ( is_wp_error( $e_keys ) ){
+		if ( is_wp_error( $e_keys ) ) {
 			return $e_keys;
 		}
 
 		$nonce = $this->encryption->get_nonce();
 
-		if ( is_wp_error( $nonce ) ){
+		if ( is_wp_error( $nonce ) ) {
 			return $nonce;
 		}
 

Braces +2 added lines, -2 removed lines

@@ -81,13 +81,13 @@
 
 		$e_keys = $this->encryption->generate_keys();
 
-		if ( is_wp_error( $e_keys ) ){
+		if ( is_wp_error( $e_keys ) ) {
 			return $e_keys;
 		}
 
 		$nonce = $this->encryption->get_nonce();
 
-		if ( is_wp_error( $nonce ) ){
+		if ( is_wp_error( $nonce ) ) {
 			return $nonce;
 		}
 

trustedlogin/trustedlogin/trustedlogin-client/src/SecurityChecks.php

Braces +4 added lines, -4 removed lines

@@ -83,7 +83,7 @@ 
 
 		$user_identifier = $passed_user_identifier;
 
-		if ( $this->in_lockdown() ){
+		if ( $this->in_lockdown() ) {
 
 			$this->logging->log( 'Site is in lockdown mode, aborting login.', __METHOD__, 'error' );
 
@@ -111,7 +111,7 @@ 
 		$approved = $this->check_approved_identifier( $secret_id );
 
 		// Don't lock-down the site, since there could have been errors related to remote validation
-		if ( is_wp_error( $approved ) ){
+		if ( is_wp_error( $approved ) ) {
 
 			$this->logging->log(
 				sprintf(
@@ -302,7 +302,7 @@ 
 
 		$notified = $this->report_lockdown();
 
-		if ( is_wp_error( $notified ) ){
+		if ( is_wp_error( $notified ) ) {
 			$this->logging->log( sprintf( 'Could not notify TrustedLogin (%s)', $notified->get_error_message() ), __METHOD__, 'error' );
 		}
 
@@ -355,7 +355,7 @@ 
 	 *
 	 * @return int|false Int: in lockdown. The value returned is the timestamp when lockdown ends. False: not in lockdown, or overridden by a constant.
 	 */
-	public function in_lockdown(){
+	public function in_lockdown() {
 
 		if ( $this->in_local_development() ) {
 			return false;

Spacing +15 added lines, -15 removed lines

@@ -83,11 +83,11 @@ 
 
 		$user_identifier = $passed_user_identifier;
 
-		if ( $this->in_lockdown() ){
+		if ( $this->in_lockdown() ) {
 
 			$this->logging->log( 'Site is in lockdown mode, aborting login.', __METHOD__, 'error' );
 
-			return new WP_Error( 'in-lockdown', __( 'TrustedLogin temporarily disabled.' , 'trustedlogin') );
+			return new WP_Error( 'in-lockdown', __( 'TrustedLogin temporarily disabled.', 'trustedlogin' ) );
 		}
 
 		// When passed in the endpoint URL, the unique ID will be the raw value, not the hash.
@@ -111,7 +111,7 @@ 
 		$approved = $this->check_approved_identifier( $secret_id );
 
 		// Don't lock-down the site, since there could have been errors related to remote validation
-		if ( is_wp_error( $approved ) ){
+		if ( is_wp_error( $approved ) ) {
 
 			$this->logging->log(
 				sprintf(
@@ -152,7 +152,7 @@ 
 				'notice'
 			);
 
-			return new WP_Error( 'brute-force-detected', 'Login aborted due to potential brute force detection.');
+			return new WP_Error( 'brute-force-detected', 'Login aborted due to potential brute force detection.' );
 		}
 
 		return true;
@@ -165,12 +165,12 @@ 
 	 */
 	private function maybe_add_used_accesskey( $user_identifier = '' ) {
 
-		$used_accesskeys = (array) get_site_transient( $this->used_accesskey_transient );
+		$used_accesskeys = (array)get_site_transient( $this->used_accesskey_transient );
 
 		// This is a new access key
 		if ( ! in_array( $user_identifier, $used_accesskeys, true ) ) {
 
-			$used_accesskeys[] = $user_identifier;
+			$used_accesskeys[ ] = $user_identifier;
 
 			$transient_set = set_site_transient( $this->used_accesskey_transient, $used_accesskeys, self::ACCESSKEY_LIMIT_EXPIRY );
 
@@ -190,19 +190,19 @@ 
 	 */
 	private function get_ip() {
 
-		if ( ! isset( $_SERVER['REMOTE_ADDR'] ) ) {
+		if ( ! isset( $_SERVER[ 'REMOTE_ADDR' ] ) ) {
 			return null;
 		}
 
-		$ip = wp_unslash( $_SERVER['REMOTE_ADDR'] );
+		$ip = wp_unslash( $_SERVER[ 'REMOTE_ADDR' ] );
 
 		$ip = trim( $ip );
 
-		if ( ! defined('TL_DOING_TESTS') ) {
+		if ( ! defined( 'TL_DOING_TESTS' ) ) {
 			$ip = filter_var( $ip, FILTER_VALIDATE_IP, FILTER_FLAG_NO_RES_RANGE | FILTER_FLAG_NO_PRIV_RANGE );
 		}
 
-		return (string) $ip;
+		return (string)$ip;
 	}
 
 	/**
@@ -226,7 +226,7 @@ 
 		 */
 		$body = array(
 			'timestamp'  => time(),
-			'user_agent' => isset( $_SERVER['HTTP_USER_AGENT'] ) ? substr( $_SERVER['HTTP_USER_AGENT'], 0, 255 ) : '',
+			'user_agent' => isset( $_SERVER[ 'HTTP_USER_AGENT' ] ) ? substr( $_SERVER[ 'HTTP_USER_AGENT' ], 0, 255 ) : '',
 			'user_ip'    => $this->get_ip(),
 			'site_url'   => get_site_url(),
 		);
@@ -265,13 +265,13 @@ 
 		 */
 		$body = array(
 			'timestamp'  => time(),
-			'user_agent' => isset( $_SERVER['HTTP_USER_AGENT'] ) ? substr( $_SERVER['HTTP_USER_AGENT'], 0, 255 ) : '',
+			'user_agent' => isset( $_SERVER[ 'HTTP_USER_AGENT' ] ) ? substr( $_SERVER[ 'HTTP_USER_AGENT' ], 0, 255 ) : '',
 			'user_ip'	 => $this->get_ip(),
 			'site_url'	 => get_site_url(),
 		);
 
 		$remote = new Remote( $this->config, $this->logging );
-		$api_response = $remote->send( self::BRUTE_FORCE_ENDPOINT , $body, 'POST' );
+		$api_response = $remote->send( self::BRUTE_FORCE_ENDPOINT, $body, 'POST' );
 
 		if ( is_wp_error( $api_response ) ) {
 			return $api_response;
@@ -302,7 +302,7 @@ 
 
 		$notified = $this->report_lockdown();
 
-		if ( is_wp_error( $notified ) ){
+		if ( is_wp_error( $notified ) ) {
 			$this->logging->log( sprintf( 'Could not notify TrustedLogin (%s)', $notified->get_error_message() ), __METHOD__, 'error' );
 		}
 
@@ -355,7 +355,7 @@ 
 	 *
 	 * @return int|false Int: in lockdown. The value returned is the timestamp when lockdown ends. False: not in lockdown, or overridden by a constant.
 	 */
-	public function in_lockdown(){
+	public function in_lockdown() {
 
 		if ( $this->in_local_development() ) {
 			return false;

trustedlogin/trustedlogin/trustedlogin-client/src/SupportUser.php

Spacing +8 added lines, -8 removed lines

@@ -13,7 +13,7 @@ 
 namespace GravityView\TrustedLogin;
 
 // Exit if accessed directly
-if ( ! defined('ABSPATH') ) {
+if ( ! defined( 'ABSPATH' ) ) {
 	exit;
 }
 
@@ -129,7 +129,7 @@ 
 
 		$user_ids = get_users( $args );
 
-		return empty( $user_ids ) ? false : (int) $user_ids[0];
+		return empty( $user_ids ) ? false : (int)$user_ids[ 0 ];
 	}
 
 	/**
@@ -218,7 +218,7 @@ 
 
 		$i = 1;
 		$new_username = $username;
-		while( username_exists( $new_username ) ) {
+		while ( username_exists( $new_username ) ) {
 			$new_username = sprintf( '%s %d', $username, $i + 1 );
 		}
 
@@ -274,7 +274,7 @@ 
 		$expires = $this->get_expiration( $support_user, false, true );
 
 		// This user has expired, but the cron didn't run...
-		if ( $expires && time() > (int) $expires ) {
+		if ( $expires && time() > (int)$expires ) {
 
 			$this->logging->log( 'The user was supposed to expire on ' . $expires . '; revoking now.', __METHOD__, 'warning' );
 
@@ -349,7 +349,7 @@ 
 
 		$user = get_users( $args );
 
-		return empty( $user ) ? null : $user[0];
+		return empty( $user ) ? null : $user[ 0 ];
 	}
 
 	/**
@@ -406,8 +406,8 @@ 
 	public function get_first() {
 		$support_users = $this->get_all();
 
-		if( $support_users ) {
-			return $support_users[0];
+		if ( $support_users ) {
+			return $support_users[ 0 ];
 		}
 
 		return null;
@@ -489,7 +489,7 @@ 
 			'number'  => 1,
 		) );
 
-		$reassign_id = empty( $admins ) ? null : $admins[0]->ID;
+		$reassign_id = empty( $admins ) ? null : $admins[ 0 ]->ID;
 
 		$this->logging->log( 'Reassign user ID: ' . var_export( $reassign_id, true ), __METHOD__, 'info' );