pryley / site-reviews

activate.php

Spacing +10 added lines, -10 removed lines

@@ -32,8 +32,8 @@ 
 	 */
 	public static function isPhpValid( $version = '' )
 	{
-		if( !empty( $version )) {
-			static::normalize( array( 'php' => $version ));
+		if( !empty($version) ) {
+			static::normalize( array( 'php' => $version ) );
 		}
 		return !version_compare( PHP_VERSION, static::$versions->php, '<' );
 	}
@@ -43,7 +43,7 @@ 
 	 */
 	public static function isValid( array $args = array() )
 	{
-		if( !empty( $args )) {
+		if( !empty($args) ) {
 			static::normalize( $args );
 		}
 		return static::isPhpValid() && static::isWpValid();
@@ -56,8 +56,8 @@ 
 	public static function isWpValid( $version = '' )
 	{
 		global $wp_version;
-		if( !empty( $version )) {
-			static::normalize( array( 'wordpress' => $version ));
+		if( !empty($version) ) {
+			static::normalize( array( 'wordpress' => $version ) );
 		}
 		return !version_compare( $wp_version, static::$versions->wordpress, '<' );
 	}
@@ -68,14 +68,14 @@ 
 	 */
 	public static function shouldDeactivate( $file, array $args = array() )
 	{
-		if( empty( static::$instance )) {
+		if( empty(static::$instance) ) {
 			static::$file = realpath( $file );
 			static::$instance = new static;
 			static::$versions = static::normalize( $args );
 		}
 		if( !static::isValid() ) {
-			add_action( 'activated_plugin', array( static::$instance, 'deactivate' ));
-			add_action( 'admin_notices', array( static::$instance, 'deactivate' ));
+			add_action( 'activated_plugin', array( static::$instance, 'deactivate' ) );
+			add_action( 'admin_notices', array( static::$instance, 'deactivate' ) );
 			return true;
 		}
 		return false;
@@ -105,7 +105,7 @@ 
 		return (object)wp_parse_args( $args, array(
 			'php' => static::MIN_PHP_VERSION,
 			'wordpress' => static::MIN_WORDPRESS_VERSION,
-		));
+		) );
 	}
 
 	/**
@@ -117,7 +117,7 @@ 
 			filter_input( INPUT_GET, 'plugin_status' ),
 			filter_input( INPUT_GET, 'paged' ),
 			filter_input( INPUT_GET, 's' )
-		)));
+		) ) );
 		exit;
 	}
 

plugin/Commands/CreateReview.php

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
 
 	public function __construct( $input )
 	{
-		$this->ajaxRequest = isset( $input['ajax_request'] );
+		$this->ajaxRequest = isset($input['ajax_request']);
 		$this->assignedTo = is_numeric( $input['assign_to'] )
 			? $input['assign_to']
 			: '';
@@ -37,7 +37,7 @@ 
 		$this->rating = intval( $input['rating'] );
 		$this->referrer = $input['_wp_http_referer'];
 		$this->request = $input;
-		$this->terms = isset( $input['terms'] );
+		$this->terms = isset($input['terms']);
 		$this->title = sanitize_text_field( $input['title'] );
 	}
 }

plugin/Shortcodes/ButtonGenerator.php

Spacing +24 added lines, -24 removed lines

@@ -47,7 +47,7 @@ 
 			'label' => '['.$tag.']',
 			'required' => $this->required,
 			'title' => esc_html__( 'Shortcode', 'site-reviews' ),
-		]);
+		] );
 		return $this;
 	}
 
@@ -57,12 +57,12 @@ 
 	protected function generateFields( array $fields )
 	{
 		$generatedFields = array_map( function( $field ) {
-			if( empty( $field ))return;
+			if( empty($field) )return;
 			$field = $this->normalize( $field );
-			if( !method_exists( $this, $method = 'normalize'.ucfirst( $field['type'] )))return;
+			if( !method_exists( $this, $method = 'normalize'.ucfirst( $field['type'] ) ) )return;
 			return $this->$method( $field );
 		}, $fields );
-		return array_values( array_filter( $generatedFields ));
+		return array_values( array_filter( $generatedFields ) );
 	}
 
 	/**
@@ -71,15 +71,15 @@ 
 	protected function getFields()
 	{
 		$fields = $this->generateFields( $this->fields() );
-		if( !empty( $this->errors )) {
+		if( !empty($this->errors) ) {
 			$errors = [];
 			foreach( $this->required as $name => $alert ) {
-				if( false !== array_search( $name, array_column( $fields, 'name' )))continue;
+				if( false !== array_search( $name, array_column( $fields, 'name' ) ) )continue;
 				$errors[] = $this->errors[$name];
 			}
 			$this->errors = $errors;
 		}
-		return empty( $this->errors )
+		return empty($this->errors)
 			? $fields
 			: $this->errors;
 	}
@@ -92,7 +92,7 @@ 
 		return wp_parse_args( $field, [
 			'items' => [],
 			'type' => '',
-		]);
+		] );
 	}
 
 	/**
@@ -110,7 +110,7 @@ 
 			'tooltip' => '',
 			'type' => '',
 			'value' => '',
-		]);
+		] );
 	}
 
 	/**
@@ -118,7 +118,7 @@ 
 	 */
 	protected function normalizeContainer( array $field )
 	{
-		if( !array_key_exists( 'html', $field ) && !array_key_exists( 'items', $field ))return;
+		if( !array_key_exists( 'html', $field ) && !array_key_exists( 'items', $field ) )return;
 		$field['items'] = $this->generateFields( $field['items'] );
 		return $field;
 	}
@@ -128,7 +128,7 @@ 
 	 */
 	protected function normalizeField( array $field, array $defaults )
 	{
-		if( !$this->validate( $field ))return;
+		if( !$this->validate( $field ) )return;
 		return array_filter( shortcode_atts( $defaults, $field ), function( $value ) {
 			return $value !== '';
 		});
@@ -148,9 +148,9 @@ 
 			'tooltip' => '',
 			'type' => '',
 			'value' => '',
-		]);
-		if( !is_array( $listbox ))return;
-		if( !array_key_exists( '', $listbox['options'] )) {
+		] );
+		if( !is_array( $listbox ) )return;
+		if( !array_key_exists( '', $listbox['options'] ) ) {
 			$listbox['options'] = ['' => $listbox['placeholder']] + $listbox['options'];
 		}
 		foreach( $listbox['options'] as $value => $text ) {
@@ -167,7 +167,7 @@ 
 	 */
 	protected function normalizePost( array $field )
 	{
-		if( !is_array( $field['query_args'] )) {
+		if( !is_array( $field['query_args'] ) ) {
 			$field['query_args'] = [];
 		}
 		$posts = get_posts( wp_parse_args( $field['query_args'], [
@@ -175,8 +175,8 @@ 
 			'orderby' => 'title',
 			'post_type' => 'post',
 			'posts_per_page' => 30,
-		]));
-		if( !empty( $posts )) {
+		] ) );
+		if( !empty($posts) ) {
 			$options = [];
 			foreach( $posts as $post ) {
 				$options[$post->ID] = esc_html( $post->post_title );
@@ -206,7 +206,7 @@ 
 			'tooltip' => '',
 			'type' => '',
 			'value' => '',
-		]);
+		] );
 	}
 
 	/**
@@ -214,7 +214,7 @@ 
 	 */
 	protected function validate( array $field )
 	{
-		$args = shortcode_atts([
+		$args = shortcode_atts( [
 			'label' => '',
 			'name' => false,
 			'required' => false,
@@ -230,13 +230,13 @@ 
 	 */
 	protected function validateErrors( array $args )
 	{
-		if( !isset( $args['required']['error'] )) {
+		if( !isset($args['required']['error']) ) {
 			return true;
 		}
-		$this->errors[$args['name']] = $this->normalizeContainer([
+		$this->errors[$args['name']] = $this->normalizeContainer( [
 			'html' => $args['required']['error'],
 			'type' => 'container',
-		]);
+		] );
 		return false;
 	}
 
@@ -249,10 +249,10 @@ 
 			return true;
 		}
 		$alert = esc_html__( 'Some of the shortcode options are required.', 'site-reviews' );
-		if( isset( $args['required']['alert'] )) {
+		if( isset($args['required']['alert']) ) {
 			$alert = $args['required']['alert'];
 		}
-		else if( !empty( $args['label'] )) {
+		else if( !empty($args['label']) ) {
 			$alert = sprintf(
 				esc_html_x( 'The "%s" option is required.', 'the option label', 'site-reviews' ),
 				str_replace( ':', '', $args['label'] )

plugin/Modules/Validator.php

Spacing +18 added lines, -18 removed lines

@@ -73,7 +73,7 @@ 
 		foreach( $this->rules as $attribute => $rules ) {
 			foreach( $rules as $rule ) {
 				$this->validateAttribute( $attribute, $rule );
-				if( $this->shouldStopValidating( $attribute ))break;
+				if( $this->shouldStopValidating( $attribute ) )break;
 			}
 		}
 		return $this->errors;
@@ -88,14 +88,14 @@ 
 	 */
 	public function validateAttribute( $attribute, $rule )
 	{
-		list( $rule, $parameters ) = $this->parseRule( $rule );
+		list($rule, $parameters) = $this->parseRule( $rule );
 		if( $rule == '' )return;
 		$value = $this->getValue( $attribute );
 		$this->validateRequired( $attribute, $value ) || in_array( $rule, $this->implicitRules );
-		if( !method_exists( $this, $method = 'validate'.$rule )) {
+		if( !method_exists( $this, $method = 'validate'.$rule ) ) {
 			throw new BadMethodCallException( "Method [$method] does not exist." );
 		}
-		if( !$this->$method( $attribute, $value, $parameters )) {
+		if( !$this->$method( $attribute, $value, $parameters ) ) {
 			$this->addFailure( $attribute, $rule, $parameters );
 		}
 	}
@@ -110,7 +110,7 @@ 
 	{
 		$message = $this->getMessage( $attribute, $rule, $parameters );
 		$this->errors[$attribute]['errors'][] = $message;
-		if( !isset( $this->errors[$attribute]['value'] )) {
+		if( !isset($this->errors[$attribute]['value']) ) {
 			$this->errors[$attribute]['value'] = $this->getValue( $attribute );
 		}
 	}
@@ -147,7 +147,7 @@ 
 	 */
 	protected function getMessage( $attribute, $rule, array $parameters )
 	{
-		if( in_array( $rule, $this->sizeRules )) {
+		if( in_array( $rule, $this->sizeRules ) ) {
 			return $this->getSizeMessage( $attribute, $rule, $parameters );
 		}
 		$lowerRule = glsr( Helper::class )->snakeCase( $rule );
@@ -162,11 +162,11 @@ 
 	 */
 	protected function getRule( $attribute, $rules )
 	{
-		if( !array_key_exists( $attribute, $this->rules ))return;
-		$rules = (array) $rules;
+		if( !array_key_exists( $attribute, $this->rules ) )return;
+		$rules = (array)$rules;
 		foreach( $this->rules[$attribute] as $rule ) {
-			list( $rule, $parameters ) = $this->parseRule( $rule );
-			if( in_array( $rule, $rules )) {
+			list($rule, $parameters) = $this->parseRule( $rule );
+			if( in_array( $rule, $rules ) ) {
 				return [$rule, $parameters];
 			}
 		}
@@ -184,7 +184,7 @@ 
 		if( is_numeric( $value ) && $hasNumeric ) {
 			return $value;
 		}
-		elseif( is_array( $value )) {
+		elseif( is_array( $value ) ) {
 			return count( $value );
 		}
 		return mb_strlen( $value );
@@ -211,7 +211,7 @@ 
 	 */
 	protected function getValue( $attribute )
 	{
-		if( isset( $this->data[$attribute] )) {
+		if( isset($this->data[$attribute]) ) {
 			return $this->data[$attribute];
 		}
 	}
@@ -224,7 +224,7 @@ 
 	 */
 	protected function hasRule( $attribute, $rules )
 	{
-		return !is_null( $this->getRule( $attribute, $rules ));
+		return !is_null( $this->getRule( $attribute, $rules ) );
 	}
 
 	/**
@@ -264,11 +264,11 @@ 
 		$parameters = [];
 		// example: {rule}:{parameters}
 		if( strpos( $rule, ':' ) !== false ) {
-			list( $rule, $parameter ) = explode( ':', $rule, 2 );
+			list($rule, $parameter) = explode( ':', $rule, 2 );
 			// example: {parameter1,parameter2,...}
 			$parameters = $this->parseParameters( $rule, $parameter );
 		}
-		$rule = ucwords( str_replace( ['-', '_'], ' ', trim( $rule )));
+		$rule = ucwords( str_replace( ['-', '_'], ' ', trim( $rule ) ) );
 		$rule = str_replace( ' ', '', $rule );
 		return [$rule, $parameters];
 	}
@@ -296,7 +296,7 @@ 
 	protected function shouldStopValidating( $attribute )
 	{
 		return $this->hasRule( $attribute, $this->implicitRules )
-			&& isset( $this->failedRules[$attribute] )
+			&& isset($this->failedRules[$attribute])
 			&& array_intersect( array_keys( $this->failedRules[$attribute] ), $this->implicitRules );
 	}
 
@@ -321,12 +321,12 @@ 
 			'regex' => _x( 'The :attribute format is invalid.', ':attribute is a placeholder and should not be translated.', 'site-reviews' ),
 			'required' => _x( 'The :attribute field is required.', ':attribute is a placeholder and should not be translated.', 'site-reviews' ),
 		];
-		$message = isset( $strings[$key] )
+		$message = isset($strings[$key])
 			? $strings[$key]
 			: false;
 		if( !$message )return;
 		$message = str_replace( ':attribute', $attribute, $message );
-		if( method_exists( $this, $replacer = 'replace'.$rule )) {
+		if( method_exists( $this, $replacer = 'replace'.$rule ) ) {
 			$message = $this->$replacer( $message, $parameters );
 		}
 		return $message;

plugin/Modules/Date.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
 		$diff = time() - strtotime( $date );
 		foreach( static::$TIME_PERIODS as $i => $timePeriod ) {
 			if( $diff > $timePeriod[0] )continue;
-			$unit = intval( floor( $diff / $timePeriod[1] ));
+			$unit = intval( floor( $diff / $timePeriod[1] ) );
 			$relativeDates = [
 				_n( '%s second ago', '%s seconds ago', $unit, 'site-reviews' ),
 				_n( '%s minute ago', '%s minutes ago', $unit, 'site-reviews' ),

plugin/Modules/Session.php

Spacing +16 added lines, -16 removed lines

@@ -33,8 +33,8 @@ 
 
 	public function __construct()
 	{
-		if( $cookieId = filter_input( INPUT_COOKIE, static::SESSION_COOKIE )) {
-			$cookie = explode( '||', stripslashes( $cookieId ));
+		if( $cookieId = filter_input( INPUT_COOKIE, static::SESSION_COOKIE ) ) {
+			$cookie = explode( '||', stripslashes( $cookieId ) );
 			$this->sessionId = preg_replace( '/[^A-Za-z0-9_]/', '', $cookie[0] );
 			$this->expiryTimestamp = absint( $cookie[1] );
 			$this->expiryTimestampReset = absint( $cookie[2] );
@@ -78,7 +78,7 @@ 
 	public function deleteExpiredSessions( $limit = 1000 )
 	{
 		global $wpdb;
-		if( $expiredSessions = implode( "','", $this->getExpiredSessions( $limit ))) {
+		if( $expiredSessions = implode( "','", $this->getExpiredSessions( $limit ) ) ) {
 			$wpdb->query(
 				"DELETE FROM {$wpdb->options} WHERE option_name IN ('{$expiredSessions}')"
 			);
@@ -94,11 +94,11 @@ 
 	public function get( $key, $fallback = '', $unset = false )
 	{
 		$key = sanitize_key( $key );
-		$value = isset( $this->sessionData[$key] )
+		$value = isset($this->sessionData[$key])
 			? maybe_unserialize( $this->sessionData[$key] )
 			: $fallback;
-		if( isset( $this->sessionData[$key] ) && $unset ) {
-			unset( $this->sessionData[$key] );
+		if( isset($this->sessionData[$key]) && $unset ) {
+			unset($this->sessionData[$key]);
 			$this->updateSession();
 		}
 		return $value;
@@ -132,7 +132,7 @@ 
 	protected function deleteSession()
 	{
 		delete_option( $this->getSessionId() );
-		delete_option( $this->getSessionId( 'expires' ));
+		delete_option( $this->getSessionId( 'expires' ) );
 	}
 
 	/**
@@ -140,7 +140,7 @@ 
 	 */
 	protected function generateSessionId()
 	{
-		return md5(( new PasswordHash( 8, false ))->get_random_bytes( 32 ));
+		return md5( (new PasswordHash( 8, false ))->get_random_bytes( 32 ) );
 	}
 
 	/**
@@ -152,13 +152,13 @@ 
 		global $wpdb;
 		$expiredSessions = [];
 		$sessions = $wpdb->get_results(
-			"SELECT option_name AS name, option_value AS expiration " .
-			"FROM {$wpdb->options} " .
-			"WHERE option_name LIKE '".static::SESSION_COOKIE."_expires_%' " .
-			"ORDER BY option_value ASC " .
+			"SELECT option_name AS name, option_value AS expiration ".
+			"FROM {$wpdb->options} ".
+			"WHERE option_name LIKE '".static::SESSION_COOKIE."_expires_%' ".
+			"ORDER BY option_value ASC ".
 			"LIMIT 0, ".absint( $limit )
 		);
-		if( !empty( $sessions )) {
+		if( !empty($sessions) ) {
 			$now = time();
 			foreach( $sessions as $session ) {
 				if( $now <= $session->expiration )continue;
@@ -175,7 +175,7 @@ 
 	 */
 	protected function getSessionId( $separator = '' )
 	{
-		return implode( '_', array_filter( [static::SESSION_COOKIE, $separator, $this->sessionId] ));
+		return implode( '_', array_filter( [static::SESSION_COOKIE, $separator, $this->sessionId] ) );
 	}
 
 	/**
@@ -206,7 +206,7 @@ 
 	{
 		if( headers_sent() )return;
 		$cookie = $this->sessionId.'||'.$this->expiryTimestamp.'||'.$this->expiryTimestampReset;
-		$cookiePath = preg_replace( '|https?://[^/]+|i', '', trailingslashit( (string)get_option( 'home' )));
+		$cookiePath = preg_replace( '|https?://[^/]+|i', '', trailingslashit( (string)get_option( 'home' ) ) );
 		setcookie( static::SESSION_COOKIE, $cookie, $this->expiryTimestamp, $cookiePath );
 	}
 
@@ -224,7 +224,7 @@ 
 	 */
 	protected function updateSession()
 	{
-		if( false === get_option( $this->getSessionId() )) {
+		if( false === get_option( $this->getSessionId() ) ) {
 			$this->createSession();
 		}
 		else {

plugin/Modules/Upgrader.php

Spacing +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
 		natsort( $routines );
 		array_walk( $routines, function( $routine ) {
 			$version = str_replace( strtolower( __CLASS__ ).'_', '', $routine );
-			if( version_compare( glsr()->version, $version, '>=' ))return;
+			if( version_compare( glsr()->version, $version, '>=' ) )return;
 			call_user_func( [$this, $routine] );
 		});
 		$this->updateVersion();
@@ -28,7 +28,7 @@ 
 	public function updateVersion()
 	{
 		$currentVersion = glsr( OptionManager::class )->get( 'version' );
-		if( version_compare( $currentVersion, glsr()->version, '<' )) {
+		if( version_compare( $currentVersion, glsr()->version, '<' ) ) {
 			glsr( OptionManager::class )->set( 'version', glsr()->version );
 		}
 		if( $currentVersion != glsr()->version ) {

plugin/Modules/Editor/Customization.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
 	 */
 	protected function isReviewEditable()
 	{
-		$postId = intval( filter_input( INPUT_GET, 'post' ));
+		$postId = intval( filter_input( INPUT_GET, 'post' ) );
 		return $postId > 0
 			&& get_post_meta( $postId, 'review_type', true ) == 'local'
 			&& $this->isReviewEditor();

plugin/Controllers/PublicController.php

Spacing +4 added lines, -4 removed lines

@@ -44,10 +44,10 @@ 
 		$vars[] = Application::PAGED_QUERY_VAR;
 		// dirty hack to fix a form submission with a field that has "name" as name
 		if( filter_input( INPUT_POST, 'action' ) == 'submit-review'
-			&& !is_null( filter_input( INPUT_POST, 'gotcha' ))) {
+			&& !is_null( filter_input( INPUT_POST, 'gotcha' ) ) ) {
 			$index = array_search( 'name', $vars, true );
 			if( false !== $index ) {
-				unset( $vars[$index] );
+				unset($vars[$index]);
 			}
 		}
 		return $vars;
@@ -59,10 +59,10 @@ 
 	public function routerCreateReview( array $request )
 	{
 		$validated = glsr( ValidateReview::class )->validate( $request );
-		if( !empty( $validated->error )) {
+		if( !empty($validated->error) ) {
 			return $validated->request;
 		}
 		if( $validated->recaptchaIsUnset )return;
-		return $this->execute( new CreateReview( $validated->request ));
+		return $this->execute( new CreateReview( $validated->request ) );
 	}
 }