Pink-Crab / Perique-Framework

src/Interfaces/Renderable.php

Spacing +4 added lines, -4 removed lines

@@ -35,7 +35,7 @@ 
 	 * @param bool $print
 	 * @return void|string
 	 */
-	public function render( string $view, iterable $data, bool $print = true );
+	public function render(string $view, iterable $data, bool $print = true);
 
 	/**
 	 * Renders a component.
@@ -43,7 +43,7 @@ 
 	 * @param Component $component
 	 * @return string|void
 	 */
-	public function component( Component $component, bool $print = true );
+	public function component(Component $component, bool $print = true);
 
 	/**
 	 * Renders a view Model
@@ -51,7 +51,7 @@ 
 	 * @param View_Model $view_model
 	 * @return string|void
 	 */
-	public function view_model( View_Model $view_model, bool $print = true );
+	public function view_model(View_Model $view_model, bool $print = true);
 
 	/**
 	 * Sets the component compiler.
@@ -59,5 +59,5 @@ 
 	 * @param Component_Compiler $compiler
 	 * @return void
 	 */
-	public function set_component_compiler( Component_Compiler $compiler ): void;
+	public function set_component_compiler(Component_Compiler $compiler): void;
 }

src/Services/View/Component/Component.php

Spacing +4 added lines, -4 removed lines

@@ -34,14 +34,14 @@
 	 */
 	public function get_variables(): array {
 		// Get all Private, public and protected properties.
-		$reflect = new \ReflectionClass( get_class( $this ) );
+		$reflect = new \ReflectionClass(get_class($this));
 		$vars    = array();
 
-		foreach ( $reflect->getProperties( \ReflectionProperty::IS_PUBLIC | \ReflectionProperty::IS_PRIVATE | \ReflectionProperty::IS_PROTECTED )
+		foreach ($reflect->getProperties(\ReflectionProperty::IS_PUBLIC | \ReflectionProperty::IS_PRIVATE | \ReflectionProperty::IS_PROTECTED)
 			as $var
 		) {
-			$var->setAccessible( true );
-			$vars[ $var->getName() ] = $var->getValue( $this );
+			$var->setAccessible(true);
+			$vars[$var->getName()] = $var->getValue($this);
 		}
 
 		return $vars;

src/Services/View/Component/Component_Compiler.php

Spacing +22 added lines, -22 removed lines

@@ -49,7 +49,7 @@ 
 	private $component_aliases = array();
 
 	/** @param array<string, string> $component_aliases */
-	public function __construct( string $component_base_path = '', array $component_aliases = array() ) {
+	public function __construct(string $component_base_path = '', array $component_aliases = array()) {
 		$this->component_base_path = $component_base_path;
 		$this->component_aliases   = $component_aliases;
 	}
@@ -60,8 +60,8 @@ 
 	 * @param Component $component
 	 * @return View_Model
 	 */
-	public function compile( Component $component ): View_Model {
-		return new View_Model( $this->get_component_path( $component ), $component->get_variables() );
+	public function compile(Component $component): View_Model {
+		return new View_Model($this->get_component_path($component), $component->get_variables());
 	}
 
 	/**
@@ -70,32 +70,32 @@ 
 	 * @param Component $component
 	 * @return string
 	 */
-	private function get_component_path( Component $component ): string {
+	private function get_component_path(Component $component): string {
 
 		// Check aliases.
-		if ( isset( $this->component_aliases[ get_class( $component ) ] ) ) {
-			return esc_html( $this->component_aliases[ get_class( $component ) ] );
+		if (isset($this->component_aliases[get_class($component)])) {
+			return esc_html($this->component_aliases[get_class($component)]);
 		}
 
-		$from_annotation = $this->get_annotation( 'view', $component );
+		$from_annotation = $this->get_annotation('view', $component);
 		// If it does have a path defined, use that.
-		if ( ! empty( $from_annotation ) ) {
-			return \trailingslashit( $this->component_base_path ) . $from_annotation;
+		if ( ! empty($from_annotation)) {
+			return \trailingslashit($this->component_base_path) . $from_annotation;
 		}
 
 		// If the component has a defined path
-		if ( $component->template() ) {
-			return \trailingslashit( $this->component_base_path ) . $component->template();
+		if ($component->template()) {
+			return \trailingslashit($this->component_base_path) . $component->template();
 		}
 
 		// Get path based on class name.
-		$reflect    = new \ReflectionClass( $component );
+		$reflect    = new \ReflectionClass($component);
 		$short_name = $reflect->getShortName();
 		// Add space between capitals, make lowercase and replace underscores with dashes.
-		$short_name = strtolower( preg_replace( '/(?<!^)[A-Z]/', '-$0', $short_name ) ?? '' );
-		$short_name = str_replace( '_', '-', $short_name );
+		$short_name = strtolower(preg_replace('/(?<!^)[A-Z]/', '-$0', $short_name) ?? '');
+		$short_name = str_replace('_', '-', $short_name);
 
-		return \trailingslashit( $this->component_base_path ) . $short_name;
+		return \trailingslashit($this->component_base_path) . $short_name;
 	}
 
 	/**
@@ -105,18 +105,18 @@ 
 	 * @param Component $component
 	 * @return string|null
 	 */
-	private function get_annotation( string $annotation, Component $component ): ?string {
-		$reflect = new \ReflectionClass( $component );
+	private function get_annotation(string $annotation, Component $component): ?string {
+		$reflect = new \ReflectionClass($component);
 		$comment = $reflect->getDocComment();
 
 		// if no comment, return null.
-		if ( empty( $comment ) ) {
+		if (empty($comment)) {
 			return null;
 		}
 
 		// If comment contains the annotation, return the value.
-		return strpos( $comment, "@{$annotation}" ) !== false
-			? $this->extract_annotation_value( $comment, $annotation )
+		return strpos($comment, "@{$annotation}") !== false
+			? $this->extract_annotation_value($comment, $annotation)
 			: null;
 	}
 
@@ -127,9 +127,9 @@ 
 	 * @param string $annotation
 	 * @return string|null
 	 */
-	private function extract_annotation_value( string $comment, string $annotation ): ?string {
+	private function extract_annotation_value(string $comment, string $annotation): ?string {
 		$pattern = "/@{$annotation}\s+(.*)/";
-		preg_match( $pattern, $comment, $matches );
+		preg_match($pattern, $comment, $matches);
 		return $matches[1] ?? null;
 	}
 

src/Services/View/PHP_Engine.php

Spacing +36 added lines, -36 removed lines

@@ -51,8 +51,8 @@ 
 	 *
 	 * @param string $base_view_path
 	 */
-	public function __construct( string $base_view_path ) {
-		$this->base_view_path = $this->verify_view_path( $base_view_path );
+	public function __construct(string $base_view_path) {
+		$this->base_view_path = $this->verify_view_path($base_view_path);
 	}
 
 	/**
@@ -61,7 +61,7 @@ 
 	 * @param Component_Compiler $compiler
 	 * @return void
 	 */
-	public function set_component_compiler( Component_Compiler $compiler ): void {
+	public function set_component_compiler(Component_Compiler $compiler): void {
 		$this->component_compiler = $compiler;
 	}
 
@@ -73,12 +73,12 @@ 
 	 * @param bool $print
 	 * @return string|void
 	 */
-	public function render( string $view, iterable $data, bool $print = true ) {
+	public function render(string $view, iterable $data, bool $print = true) {
 
-		if ( $print ) {
-			print( $this->render_buffer( $view, $data ) ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
+		if ($print) {
+			print($this->render_buffer($view, $data)); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
 		} else {
-			return $this->render_buffer( $view, $data );
+			return $this->render_buffer($view, $data);
 		}
 	}
 
@@ -88,16 +88,16 @@ 
 	 * @param Component $component
 	 * @return string|void
 	 */
-	public function component( Component $component, bool $print = true ) {
+	public function component(Component $component, bool $print = true) {
 
 		// Throw exception of no compiler passed.
-		if ( ! is_a( $this->component_compiler, Component_Compiler::class ) ) {
-			throw new Exception( 'No component compiler passed to PHP_Engine' );
+		if ( ! is_a($this->component_compiler, Component_Compiler::class)) {
+			throw new Exception('No component compiler passed to PHP_Engine');
 		}
 
 		// Compile the component.
-		$compiled = $this->component_compiler->compile( $component );
-		return $this->render( $compiled->template(), $compiled->data(), $print );
+		$compiled = $this->component_compiler->compile($component);
+		return $this->render($compiled->template(), $compiled->data(), $print);
 	}
 
 
@@ -107,8 +107,8 @@ 
 	 * @param View_Model $view_model
 	 * @return string|void
 	 */
-	public function view_model( View_Model $view_model, bool $print = true ) {
-		return $this->render( $view_model->template(), $view_model->data(), $print );
+	public function view_model(View_Model $view_model, bool $print = true) {
+		return $this->render($view_model->template(), $view_model->data(), $print);
 	}
 
 	/**
@@ -119,11 +119,11 @@ 
 	 * @param bool $print
 	 * @return string|void
 	 */
-	public function partial( string $view, iterable $data = array(), bool $print = true ) {
-		if ( $print ) {
-			$this->render( $view, $data, $print );
+	public function partial(string $view, iterable $data = array(), bool $print = true) {
+		if ($print) {
+			$this->render($view, $data, $print);
 		} else {
-			return $this->render( $view, $data, $print );
+			return $this->render($view, $data, $print);
 		}
 	}
 
@@ -135,26 +135,26 @@ 
 	 * @return string
 	 * @throws Exception
 	 */
-	protected function render_buffer( string $view, iterable $__data ): string {
+	protected function render_buffer(string $view, iterable $__data): string {
 
-		if ( ! file_exists( $this->resolve_file_path( $view ) ) ) {
-			throw new Exception( "{$view} doesn't exist" );
+		if ( ! file_exists($this->resolve_file_path($view))) {
+			throw new Exception("{$view} doesn't exist");
 		}
 
 		$output = '';
 		ob_start();
 
 		// Set all the data values a parameters.
-		foreach ( $__data as $__key => $__value ) {
-			if ( is_string( $__key ) ) {
-				${\wp_strip_all_tags( $__key )} = $__value;
+		foreach ($__data as $__key => $__value) {
+			if (is_string($__key)) {
+				${\wp_strip_all_tags($__key)} = $__value;
 			}
 
 			// Unset the key and value.
-			unset( $__key, $__value, $__data );
+			unset($__key, $__value, $__data);
 		}
 
-		include $this->resolve_file_path( $view );
+		include $this->resolve_file_path($view);
 		$output = ob_get_contents();
 		ob_end_clean();
 		return $output ?: '';
@@ -166,10 +166,10 @@ 
 	 * @param string $file
 	 * @return string
 	 */
-	protected function clean_filename( string $file ): string {
-			$file = ltrim( $file, '/' );
-			return substr( $file, -4 ) === '.php'
-			? substr( $file, 0, -4 )
+	protected function clean_filename(string $file): string {
+			$file = ltrim($file, '/');
+			return substr($file, -4) === '.php'
+			? substr($file, 0, -4)
 			: $file;
 
 	}
@@ -180,11 +180,11 @@ 
 	 * @param string $filename
 	 * @return string
 	 */
-	protected function resolve_file_path( string $filename ): string {
+	protected function resolve_file_path(string $filename): string {
 		return sprintf(
 			'%s%s.php',
 			$this->base_view_path,
-			$this->clean_filename( $filename )
+			$this->clean_filename($filename)
 		);
 	}
 
@@ -196,12 +196,12 @@ 
 	 * @return string
 	 * @throws Exception
 	 */
-	protected function verify_view_path( string $path ): string {
+	protected function verify_view_path(string $path): string {
 
-		$path = rtrim( $path, '/' ) . '/';
+		$path = rtrim($path, '/') . '/';
 
-		if ( ! \is_dir( $path ) ) {
-			throw new Exception( "{$path} doesn't exist and cant be used as the base view path." );
+		if ( ! \is_dir($path)) {
+			throw new Exception("{$path} doesn't exist and cant be used as the base view path.");
 		}
 
 		return $path;

src/Services/View/View_Model.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
 	private $data = array();
 
 	/** @param array<string, mixed> $data */
-	public function __construct( string $template, array $data = array() ) {
+	public function __construct(string $template, array $data = array()) {
 		$this->template = $template;
 		$this->data     = $data;
 	}

src/Services/View/View.php

Spacing +9 added lines, -9 removed lines

@@ -61,12 +61,12 @@ 
 	 *
 	 * @param Renderable $engine
 	 */
-	public function __construct( Renderable $engine, Component_Compiler $component_compiler ) {
+	public function __construct(Renderable $engine, Component_Compiler $component_compiler) {
 		$this->engine             = $engine;
 		$this->component_compiler = $component_compiler;
 
 		// Populate engine with compiler.
-		$this->engine->set_component_compiler( $component_compiler );
+		$this->engine->set_component_compiler($component_compiler);
 	}
 
 	/**
@@ -77,8 +77,8 @@ 
 	 * @param bool $print Print or Return the HTML
 	 * @return string|void
 	 */
-	public function render( string $view, iterable $view_data = array(), bool $print = self::PRINT_VIEW ) {
-		return $this->engine->render( $view, $view_data, $print );
+	public function render(string $view, iterable $view_data = array(), bool $print = self::PRINT_VIEW) {
+		return $this->engine->render($view, $view_data, $print);
 	}
 
 	/**
@@ -88,8 +88,8 @@ 
 	 * @param bool $print Print or Return the HTML
 	 * @return string|void
 	 */
-	public function component( Component $component, bool $print = self::PRINT_VIEW ) {
-		return $this->engine->component( $component, $print );
+	public function component(Component $component, bool $print = self::PRINT_VIEW) {
+		return $this->engine->component($component, $print);
 	}
 
 	/**
@@ -99,8 +99,8 @@ 
 	 * @param bool $print Print or Return the HTML
 	 * @return string|void
 	 */
-	public function view_model( View_Model $view_model, bool $print = self::PRINT_VIEW ) {
-		return $this->engine->view_model( $view_model, $print );
+	public function view_model(View_Model $view_model, bool $print = self::PRINT_VIEW) {
+		return $this->engine->view_model($view_model, $print);
 	}
 
 	/**
@@ -109,7 +109,7 @@ 
 	 * @param callable $to_buffer
 	 * @return string
 	 */
-	public static function print_buffer( callable $to_buffer ): string {
+	public static function print_buffer(callable $to_buffer): string {
 		ob_start();
 		$to_buffer();
 		$output = ob_get_contents();

src/Services/Registration/Registration_Service.php

Spacing +16 added lines, -16 removed lines

@@ -65,7 +65,7 @@ 
 	 * @param DI_Container $di_container
 	 * @return self
 	 */
-	public function set_container( DI_Container $di_container ): self {
+	public function set_container(DI_Container $di_container): self {
 		$this->di_container = $di_container;
 		return $this;
 	}
@@ -76,7 +76,7 @@ 
 	 * @param Hook_Loader $loader
 	 * @return self
 	 */
-	public function set_loader( Hook_Loader $loader ): self {
+	public function set_loader(Hook_Loader $loader): self {
 		$this->loader = $loader;
 		return $this;
 	}
@@ -87,8 +87,8 @@ 
 	 * @param Registration_Middleware $middleware
 	 * @return self
 	 */
-	public function push_middleware( Registration_Middleware $middleware ): self {
-		$this->middleware[ \get_class( $middleware ) ] = $middleware;
+	public function push_middleware(Registration_Middleware $middleware): self {
+		$this->middleware[\get_class($middleware)] = $middleware;
 		return $this;
 	}
 
@@ -98,7 +98,7 @@ 
 	 * @param array<string> $class_list
 	 * @return self
 	 */
-	public function set_classes( array $class_list ): self {
+	public function set_classes(array $class_list): self {
 		$this->class_list = $class_list;
 		return $this;
 	}
@@ -109,7 +109,7 @@ 
 	 * @param string $class
 	 * @return self
 	 */
-	public function push_class( string $class ): self {
+	public function push_class(string $class): self {
 		$this->class_list[] = $class;
 		return $this;
 	}
@@ -122,32 +122,32 @@ 
 	public function process(): void {
 
 		// Filter all classes, before processing.
-		$class_list = apply_filters( Hooks::APP_INIT_REGISTRATION_CLASS_LIST, $this->class_list );
+		$class_list = apply_filters(Hooks::APP_INIT_REGISTRATION_CLASS_LIST, $this->class_list);
 
-		foreach ( $this->middleware as $middleware ) {
+		foreach ($this->middleware as $middleware) {
 
 			// Set the container if requested.
-			if ( is_object( $middleware ) && \method_exists( $middleware, 'set_di_container' ) ) {
-				$middleware->set_di_container( $this->di_container );
+			if (is_object($middleware) && \method_exists($middleware, 'set_di_container')) {
+				$middleware->set_di_container($this->di_container);
 			}
 
 			// Set the hook loader if requested.
-			if ( is_object( $middleware ) && \method_exists( $middleware, 'set_hook_loader' ) && ! is_null( $this->loader ) ) {
-				$middleware->set_hook_loader( $this->loader );
+			if (is_object($middleware) && \method_exists($middleware, 'set_hook_loader') && ! is_null($this->loader)) {
+				$middleware->set_hook_loader($this->loader);
 			}
 
 			// Run middleware setup
 			$middleware->setup();
 
 			// Pass each class to the middleware.
-			foreach ( $class_list as $class ) {
+			foreach ($class_list as $class) {
 
 				// Construct class using container,
-				$class_instance = $this->di_container->create( $class );
+				$class_instance = $this->di_container->create($class);
 
 				// if valid object process via current middleware
-				if ( is_object( $class_instance ) ) {
-					$middleware->process( $class_instance );
+				if (is_object($class_instance)) {
+					$middleware->process($class_instance);
 				}
 			}