pryley / site-reviews

plugin/Handlers/RegisterShortcodeButtons.php

Spacing +3 added lines, -3 removed lines

@@ -14,14 +14,14 @@
 	{
 		foreach( $command->shortcodes as $slug => $label ) {
 			$buttonClass = glsr( Helper::class )->buildClassName( $slug.'-button', 'Shortcodes' );
-			if( !class_exists( $buttonClass )) {
-				glsr_log()->error( sprintf( 'Class missing (%s)', $buttonClass ));
+			if( !class_exists( $buttonClass ) ) {
+				glsr_log()->error( sprintf( 'Class missing (%s)', $buttonClass ) );
 				continue;
 			}
 			$shortcode = glsr( $buttonClass )->register( $slug, [
 				'label' => $label,
 				'title' => $label,
-			]);
+			] );
 			glsr()->mceShortcodes[$slug] = $shortcode->properties;
 		}
 	}

plugin/Handlers/RegisterShortcodes.php

Spacing +2 added lines, -2 removed lines

@@ -14,8 +14,8 @@
 	{
 		foreach( $command->shortcodes as $shortcode ) {
 			$shortcodeClass = glsr( Helper::class )->buildClassName( $shortcode.'-shortcode', 'Shortcodes' );
-			if( !class_exists( $shortcodeClass )) {
-				glsr_log()->error( sprintf( 'Class missing (%s)', $shortcodeClass ));
+			if( !class_exists( $shortcodeClass ) ) {
+				glsr_log()->error( sprintf( 'Class missing (%s)', $shortcodeClass ) );
 				continue;
 			}
 			add_shortcode( $shortcode, [glsr( $shortcodeClass ), 'buildShortcode'] );

plugin/Handlers/RegisterWidgets.php

Spacing +2 added lines, -2 removed lines

@@ -16,8 +16,8 @@
 		global $wp_widget_factory;
 		foreach( $command->widgets as $key => $values ) {
 			$widgetClass = glsr( Helper::class )->buildClassName( $key.'-widget', 'Widgets' );
-			if( !class_exists( $widgetClass )) {
-				glsr_log()->error( sprintf( 'Class missing (%s)', $widgetClass ));
+			if( !class_exists( $widgetClass ) ) {
+				glsr_log()->error( sprintf( 'Class missing (%s)', $widgetClass ) );
 				continue;
 			}
 			// Here we bypass register_widget() in order to pass our custom values to the widget

plugin/Handlers/TogglePinned.php

Spacing +2 added lines, -2 removed lines

@@ -12,10 +12,10 @@
 	 */
 	public function handle( Command $command )
 	{
-		if( !get_post( $command->id )) {
+		if( !get_post( $command->id ) ) {
 			return false;
 		}
-		if( is_null( $command->pinned )) {
+		if( is_null( $command->pinned ) ) {
 			$meta = get_post_meta( $command->id, 'pinned', true );
 			$command->pinned = !wp_validate_boolean( $meta );
 		}

plugin/Controllers/Controller.php

Braces +3 added lines, -1 removed lines

@@ -15,7 +15,9 @@
 	 */
 	public function download( $filename, $content )
 	{
-		if( !current_user_can( Application::CAPABILITY ))return;
+		if( !current_user_can( Application::CAPABILITY )) {
+			return;
+		}
 		nocache_headers();
 		header( 'Content-Type: text/plain' );
 		header( 'Content-Disposition: attachment; filename="'.$filename.'"' );

Spacing +5 added lines, -5 removed lines

@@ -15,7 +15,7 @@ 
 	 */
 	public function download( $filename, $content )
 	{
-		if( !current_user_can( Application::CAPABILITY ))return;
+		if( !current_user_can( Application::CAPABILITY ) )return;
 		nocache_headers();
 		header( 'Content-Type: text/plain' );
 		header( 'Content-Disposition: attachment; filename="'.$filename.'"' );
@@ -30,8 +30,8 @@ 
 	 */
 	public function execute( $command )
 	{
-		$handlerClass = str_replace( 'Commands', 'Handlers', get_class( $command ));
-		if( !class_exists( $handlerClass )) {
+		$handlerClass = str_replace( 'Commands', 'Handlers', get_class( $command ) );
+		if( !class_exists( $handlerClass ) ) {
 			throw new InvalidArgumentException( 'Handler '.$handlerClass.' not found.' );
 		}
 		try {
@@ -39,7 +39,7 @@ 
 		}
 		catch( Exception $e ) {
 			status_header( 400 );
-			glsr( Notice::class )->addError( new WP_Error( 'site_reviews_error', $e->getMessage() ));
+			glsr( Notice::class )->addError( new WP_Error( 'site_reviews_error', $e->getMessage() ) );
 			glsr_log()->error( $e->getMessage() );
 		}
 	}
@@ -49,6 +49,6 @@ 
 	 */
 	protected function getPostId()
 	{
-		return intval( filter_input( INPUT_GET, 'post' ));
+		return intval( filter_input( INPUT_GET, 'post' ) );
 	}
 }

plugin/Modules/Html.php

Spacing +1 added lines, -1 removed lines

@@ -82,7 +82,7 @@
 	{
 		$this->render()->div( glsr( Notice::class )->get(), [
 			'id' => 'glsr-notices',
-		]);
+		] );
 	}
 
 	/**

plugin/Modules/Validator/ValidationRules.php

Spacing +2 added lines, -2 removed lines

@@ -130,8 +130,8 @@
 	public function validateRequired( $attribute, $value )
 	{
 		return is_null( $value )
-			|| ( is_string( $value ) && trim( $value ) === '' )
-			|| ( is_array( $value ) && count( $value ) < 1 )
+			|| (is_string( $value ) && trim( $value ) === '')
+			|| (is_array( $value ) && count( $value ) < 1)
 			? false
 			: true;
 	}

plugin/Modules/Email.php

Braces +6 added lines, -2 removed lines

@@ -64,7 +64,9 @@ 
 	 */
 	public function send()
 	{
-		if( !$this->message || !$this->subject || !$this->to )return;
+		if( !$this->message || !$this->subject || !$this->to ) {
+			return;
+		}
 		$sent = wp_mail(
 			$this->to,
 			$this->subject,
@@ -83,7 +85,9 @@ 
 	 */
 	public function buildPlainTextMessage( $phpmailer )
 	{
-		if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody ))return;
+		if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody )) {
+			return;
+		}
 		$message = $this->stripHtmlTags( $phpmailer->Body );
 		$phpmailer->AltBody = apply_filters( 'site-reviews/email/message', $message, 'text', $this );
 	}

Spacing +10 added lines, -10 removed lines

@@ -43,7 +43,7 @@ 
 		$this->message = $this->buildHtmlMessage( $email );
 		$this->subject = $email['subject'];
 		$this->to = $email['to'];
-		add_action( 'phpmailer_init', [ $this, 'buildPlainTextMessage'] );
+		add_action( 'phpmailer_init', [$this, 'buildPlainTextMessage'] );
 		return $this;
 	}
 
@@ -83,7 +83,7 @@ 
 	 */
 	public function buildPlainTextMessage( $phpmailer )
 	{
-		if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody ))return;
+		if( $phpmailer->ContentType === 'text/plain' || !empty($phpmailer->AltBody) )return;
 		$message = $this->stripHtmlTags( $phpmailer->Body );
 		$phpmailer->AltBody = apply_filters( 'site-reviews/email/message', $message, 'text', $this );
 	}
@@ -99,10 +99,10 @@ 
 			'from',
 			'reply-to',
 		];
-		$headers = array_intersect_key( $email, array_flip( $allowed ));
+		$headers = array_intersect_key( $email, array_flip( $allowed ) );
 		$headers = array_filter( $headers );
 		foreach( $headers as $key => $value ) {
-			unset( $headers[ $key ] );
+			unset($headers[$key]);
 			$headers[] = "{$key}: {$value}";
 		}
 		$headers[] = 'Content-Type: text/html';
@@ -114,16 +114,16 @@ 
 	 */
 	protected function buildHtmlMessage( $email )
 	{
-		$template = trim( glsr( OptionManager::class )->get( 'settings.general.notification_message' ));
-		if( !empty( $template )) {
+		$template = trim( glsr( OptionManager::class )->get( 'settings.general.notification_message' ) );
+		if( !empty($template) ) {
 			$message = glsr( Template::class )->interpolate( $template, $email['template-tags'] );
 		}
 		else if( $email['template'] ) {
 			$message = glsr( Template::class )->build( 'templates/'.$email['template'], [
 				'context' => $email['template-tags'],
-			]);
+			] );
 		}
-		if( !isset( $message )) {
+		if( !isset($message) ) {
 			$message = $email['message'];
 		}
 		$message = $email['before'].$message.$email['after'];
@@ -134,7 +134,7 @@ 
 		$message = str_replace( ']]>', ']]>', $message );
 		$message = glsr( Template::class )->build( 'partials/email/index', [
 			'context' => ['message' => $message],
-		]);
+		] );
 		return apply_filters( 'site-reviews/email/message', stripslashes( $message ), 'html', $this );
 	}
 
@@ -160,7 +160,7 @@ 
 			'to' => '',
 		];
 		$email = shortcode_atts( $defaults, $email );
-		if( empty( $email['reply-to'] )) {
+		if( empty($email['reply-to']) ) {
 			$email['reply-to'] = $email['from'];
 		}
 		return apply_filters( 'site-reviews/email/compose', $email, $this );

plugin/Modules/Validator.php

Braces +12 added lines, -4 removed lines

@@ -74,7 +74,9 @@ 
 		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;
@@ -90,7 +92,9 @@ 
 	public function validateAttribute( $attribute, $rule )
 	{
 		list( $rule, $parameters ) = $this->parseRule( $rule );
-		if( $rule == '' )return;
+		if( $rule == '' ) {
+			return;
+		}
 		$value = $this->getValue( $attribute );
 		$this->validateRequired( $attribute, $value ) || in_array( $rule, $this->implicitRules );
 		if( !method_exists( $this, $method = 'validate'.$rule )) {
@@ -163,7 +167,9 @@ 
 	 */
 	protected function getRule( $attribute, $rules )
 	{
-		if( !array_key_exists( $attribute, $this->rules ))return;
+		if( !array_key_exists( $attribute, $this->rules )) {
+			return;
+		}
 		$rules = (array) $rules;
 		foreach( $this->rules[$attribute] as $rule ) {
 			list( $rule, $parameters ) = $this->parseRule( $rule );
@@ -314,7 +320,9 @@ 
 		$message = isset( $strings[$key] )
 			? $strings[$key]
 			: false;
-		if( !$message )return;
+		if( !$message ) {
+			return;
+		}
 		$message = str_replace( ':attribute', $attribute, $message );
 		if( method_exists( $this, $replacer = 'replace'.$rule )) {
 			$message = $this->$replacer( $message, $parameters );

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;