Pink-Crab / Perique-Framework

src/Application/App.php

Indentation +3 added lines, -3 removed lines

@@ -362,7 +362,7 @@ 
 		$this->module_manager->register_modules(); // @phpstan-ignore-line, already verified if not null
 
 		/**
- * @hook{string, App_Config, Loader, DI_Container}
+		 * @hook{string, App_Config, Loader, DI_Container}
 */
 		do_action( Hooks::APP_INIT_PRE_BOOT, self::$app_config, $this->loader, self::$container ); // phpcs:disable WordPress.NamingConventions.ValidHookName.*
 
@@ -424,7 +424,7 @@ 
 			throw App_Initialization_Exception::app_not_initialized( View::class );
 		}
 		/**
- * @var ?View
+		 * @var ?View
 */
 		return self::$container->create( View::class ); // @phpstan-ignore-line, already verified if not null
 	}
@@ -440,7 +440,7 @@ 
 	 *  base_path:string,
 	 *  view_path:?string
 	 * }
-	*/
+	 */
 	public function __debugInfo(): array {
 		return array(
 			'container'      => self::$container,

Spacing +49 added lines, -49 removed lines

@@ -106,11 +106,11 @@ 
 	 *
 	 * @param string $base_path
 	 */
-	public function __construct( string $base_path ) {
+	public function __construct(string $base_path) {
 		$this->base_path = $base_path;
 
 		// Assume the view path.
-		$this->view_path = rtrim( $this->base_path, '/\\' ) . \DIRECTORY_SEPARATOR . 'views';
+		$this->view_path = rtrim($this->base_path, '/\\') . \DIRECTORY_SEPARATOR . 'views';
 	}
 
 	/**
@@ -119,7 +119,7 @@ 
 	 * @param string $view_path
 	 * @return self
 	 */
-	public function set_view_path( string $view_path ): self {
+	public function set_view_path(string $view_path): self {
 		$this->view_path = $view_path;
 		return $this;
 	}
@@ -131,8 +131,8 @@ 
 	 * @return self
 	 * @throws App_Initialization_Exception Code 2
 	 */
-	public function set_container( DI_Container $container ): self {
-		if ( self::$container !== null ) {
+	public function set_container(DI_Container $container): self {
+		if (self::$container !== null) {
 			throw App_Initialization_Exception::di_container_exists();
 		}
 
@@ -157,28 +157,28 @@ 
 	 * @return self
 	 * @throws App_Initialization_Exception Code 5
 	 */
-	public function set_app_config( array $settings ): self {
-		if ( self::$app_config !== null ) {
+	public function set_app_config(array $settings): self {
+		if (self::$app_config !== null) {
 			throw App_Initialization_Exception::app_config_exists();
 		}
 
 		// Run through the filter to allow for config changes.
-		$settings = apply_filters( Hooks::APP_INIT_CONFIG_VALUES, $settings );
+		$settings = apply_filters(Hooks::APP_INIT_CONFIG_VALUES, $settings);
 
 		// Ensure the base path and url are defined from app.
 		$settings['path']           = $settings['path'] ?? array();
 		$settings['path']['plugin'] = $this->base_path;
-		$settings['path']['view']   = $this->view_path ?? App_Config_Path_Helper::assume_view_path( $this->base_path );
+		$settings['path']['view']   = $this->view_path ?? App_Config_Path_Helper::assume_view_path($this->base_path);
 
 		// Get the url from the base path.
 		$settings['url']           = $settings['url'] ?? array();
-		$settings['url']['plugin'] = App_Config_Path_Helper::assume_base_url( $this->base_path );
+		$settings['url']['plugin'] = App_Config_Path_Helper::assume_base_url($this->base_path);
 		$settings['url']['view']   = App_Config_Path_Helper::assume_view_url(
 			$this->base_path,
-			$this->view_path ?? App_Config_Path_Helper::assume_view_path( $this->base_path )
+			$this->view_path ?? App_Config_Path_Helper::assume_view_path($this->base_path)
 		);
 
-		self::$app_config = new App_Config( $settings );
+		self::$app_config = new App_Config($settings);
 		return $this;
 	}
 
@@ -189,8 +189,8 @@ 
 	 * @return self
 	 * @throws App_Initialization_Exception Code 10
 	 */
-	public function set_module_manager( Module_Manager $module_manager ): self {
-		if ( $this->module_manager !== null ) {
+	public function set_module_manager(Module_Manager $module_manager): self {
+		if ($this->module_manager !== null) {
 			throw App_Initialization_Exception::module_manager_exists();
 		}
 
@@ -204,8 +204,8 @@ 
 	 * @param \PinkCrab\Loader\Hook_Loader $loader
 	 * @return self
 	 */
-	public function set_loader( Hook_Loader $loader ): self {
-		if ( $this->loader !== null ) {
+	public function set_loader(Hook_Loader $loader): self {
+		if ($this->loader !== null) {
 			throw App_Initialization_Exception::loader_exists();
 		}
 		$this->loader = $loader;
@@ -220,11 +220,11 @@ 
 	 * @return self
 	 * @throws App_Initialization_Exception Code 1
 	 */
-	public function container_config( callable $callback ): self {
-		if ( self::$container === null ) {
+	public function container_config(callable $callback): self {
+		if (self::$container === null) {
 			throw App_Initialization_Exception::requires_di_container();
 		}
-		$callback( self::$container );
+		$callback(self::$container);
 		return $this;
 	}
 
@@ -235,13 +235,13 @@ 
 	 * @return self
 	 * @throws App_Initialization_Exception Code 3
 	 */
-	public function registration_classes( array $class_list ): self {
-		if ( $this->module_manager === null ) {
+	public function registration_classes(array $class_list): self {
+		if ($this->module_manager === null) {
 			throw App_Initialization_Exception::requires_module_manager();
 		}
 
-		foreach ( $class_list as $class ) {
-			$this->module_manager->register_class( $class );
+		foreach ($class_list as $class) {
+			$this->module_manager->register_class($class);
 		}
 		return $this;
 	}
@@ -256,17 +256,17 @@ 
 	 * @throws App_Initialization_Exception Code 1 If DI container not registered
 	 * @throws App_Initialization_Exception Code 3 If module manager not defined.
 	 */
-	public function module( string $module, ?callable $callback = null ): self {
+	public function module(string $module, ?callable $callback = null): self {
 		// Check if module manager exists.
-		if ( $this->module_manager === null ) {
+		if ($this->module_manager === null) {
 			throw App_Initialization_Exception::requires_module_manager();
 		}
 
-		if ( self::$container === null ) {
+		if (self::$container === null) {
 			throw App_Initialization_Exception::requires_di_container();
 		}
 
-		$this->module_manager->push_module( $module, $callback );
+		$this->module_manager->push_module($module, $callback);
 
 		return $this;
 	}
@@ -280,8 +280,8 @@ 
 	public function boot(): self {
 
 		// Validate.
-		$validate = new App_Validation( $this );
-		if ( $validate->validate() === false ) {
+		$validate = new App_Validation($this);
+		if ($validate->validate() === false) {
 			throw App_Initialization_Exception::failed_boot_validation(
 				$validate->errors
 			);
@@ -333,7 +333,7 @@ 
 			Inject_Hook_Loader::class,
 			array(
 				'call' => array(
-					array( 'set_hook_loader', array( $this->loader ) ),
+					array('set_hook_loader', array($this->loader)),
 				),
 			)
 		);
@@ -343,7 +343,7 @@ 
 			Inject_App_Config::class,
 			array(
 				'call' => array(
-					array( 'set_app_config', array( self::$app_config ) ),
+					array('set_app_config', array(self::$app_config)),
 				),
 			)
 		);
@@ -353,7 +353,7 @@ 
 			Inject_DI_Container::class,
 			array(
 				'call' => array(
-					array( 'set_di_container', array( self::$container ) ),
+					array('set_di_container', array(self::$container)),
 				),
 			)
 		);
@@ -364,15 +364,15 @@ 
 		/**
  * @hook{string, App_Config, Loader, DI_Container}
 */
-		do_action( Hooks::APP_INIT_PRE_BOOT, self::$app_config, $this->loader, self::$container ); // phpcs:disable WordPress.NamingConventions.ValidHookName.*
+		do_action(Hooks::APP_INIT_PRE_BOOT, self::$app_config, $this->loader, self::$container); // phpcs:disable WordPress.NamingConventions.ValidHookName.*
 
 		// Initialise on init
 		add_action(
 			'init',
-			function () {
-				do_action( Hooks::APP_INIT_PRE_REGISTRATION, self::$app_config, $this->loader, self::$container );
+			function() {
+				do_action(Hooks::APP_INIT_PRE_REGISTRATION, self::$app_config, $this->loader, self::$container);
 				$this->module_manager->process_middleware(); // @phpstan-ignore-line, already verified if not null
-				do_action( Hooks::APP_INIT_POST_REGISTRATION, self::$app_config, $this->loader, self::$container );
+				do_action(Hooks::APP_INIT_POST_REGISTRATION, self::$app_config, $this->loader, self::$container);
 				$this->loader->register_hooks(); // @phpstan-ignore-line, if loader is not defined, exception will be thrown above
 			},
 			1
@@ -391,11 +391,11 @@ 
 	 * @return object|null
 	 * @throws App_Initialization_Exception Code 4
 	 */
-	public static function make( string $class, array $args = array() ): ?object {
-		if ( self::$booted === false ) {
-			throw App_Initialization_Exception::app_not_initialized( DI_Container::class );
+	public static function make(string $class, array $args = array()): ?object {
+		if (self::$booted === false) {
+			throw App_Initialization_Exception::app_not_initialized(DI_Container::class);
 		}
-		return self::$container->create( $class, $args ); // @phpstan-ignore-line, already verified if not null
+		return self::$container->create($class, $args); // @phpstan-ignore-line, already verified if not null
 	}
 
 	/**
@@ -406,11 +406,11 @@ 
 	 * @return mixed
 	 * @throws App_Initialization_Exception Code 4
 	 */
-	public static function config( string $key, string ...$child ) {
-		if ( self::$booted === false ) {
-			throw App_Initialization_Exception::app_not_initialized( App_Config::class );
+	public static function config(string $key, string ...$child) {
+		if (self::$booted === false) {
+			throw App_Initialization_Exception::app_not_initialized(App_Config::class);
 		}
-		return self::$app_config->{$key}( ...$child );
+		return self::$app_config->{$key}(...$child);
 	}
 
 	/**
@@ -420,13 +420,13 @@ 
 	 * @throws App_Initialization_Exception Code 4
 	 */
 	public static function view(): ?View {
-		if ( self::$booted === false ) {
-			throw App_Initialization_Exception::app_not_initialized( View::class );
+		if (self::$booted === false) {
+			throw App_Initialization_Exception::app_not_initialized(View::class);
 		}
 		/**
  * @var ?View
 */
-		return self::$container->create( View::class ); // @phpstan-ignore-line, already verified if not null
+		return self::$container->create(View::class); // @phpstan-ignore-line, already verified if not null
 	}
 
 	/**
@@ -458,7 +458,7 @@ 
 	 * @return boolean
 	 */
 	public function has_app_config(): bool {
-		return Object_Helper::is_a( self::$app_config, App_Config::class );
+		return Object_Helper::is_a(self::$app_config, App_Config::class);
 	}
 
 	/**
@@ -468,7 +468,7 @@ 
 	 * @throws App_Initialization_Exception (Code 1)
 	 */
 	public function get_container(): DI_Container {
-		if ( self::$container === null ) {
+		if (self::$container === null) {
 			// Throw container not set.
 			throw App_Initialization_Exception::requires_di_container();
 		}

src/Services/View/PHP_Engine.php

Spacing +44 added lines, -44 removed lines

@@ -53,8 +53,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);
 	}
 
 	/**
@@ -63,7 +63,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;
 	}
 
@@ -75,13 +75,13 @@ 
 	 * @param boolean                 $print
 	 * @return string|null
 	 */
-	public function render( string $view, iterable $data, bool $print = true ): ?string {
-		$view = $this->resolve_file_path( $view );
-		if ( $print ) {
-			print( $this->render_buffer( $view, $data ) ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
+	public function render(string $view, iterable $data, bool $print = true): ?string {
+		$view = $this->resolve_file_path($view);
+		if ($print) {
+			print($this->render_buffer($view, $data)); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
 			return null;
 		} else {
-			return $this->render_buffer( $view, $data );
+			return $this->render_buffer($view, $data);
 		}
 	}
 
@@ -91,22 +91,22 @@ 
 	 * @param Component $component
 	 * @return string
 	 */
-	public function component( Component $component, bool $print = true ): ?string {
+	public function component(Component $component, bool $print = true): ?string {
 
 		// Throw exception of no compiler passed.
-		if ( ! Object_Helper::is_a( $this->component_compiler, Component_Compiler::class ) ) {
-			throw new Exception( 'No component compiler passed to PHP_Engine' );
+		if ( ! Object_Helper::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 ); // @phpstan-ignore-line, checked above.
-		$template = $this->maybe_resolve_dot_notation( $compiled->template() );
-		$view     = sprintf( '%s%s%s.php', $this->base_view_path, \DIRECTORY_SEPARATOR, trim( $template ) );
-		if ( $print ) {
-			print( $this->render_buffer( $view, $compiled->data() ) ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
+		$compiled = $this->component_compiler->compile($component); // @phpstan-ignore-line, checked above.
+		$template = $this->maybe_resolve_dot_notation($compiled->template());
+		$view     = sprintf('%s%s%s.php', $this->base_view_path, \DIRECTORY_SEPARATOR, trim($template));
+		if ($print) {
+			print($this->render_buffer($view, $compiled->data())); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
 			return null;
 		} else {
-			return $this->render_buffer( $view, $compiled->data() );
+			return $this->render_buffer($view, $compiled->data());
 		}
 	}
 
@@ -117,8 +117,8 @@ 
 	 * @param View_Model $view_model
 	 * @return string|null
 	 */
-	public function view_model( View_Model $view_model, bool $print = true ): ?string {
-		return $this->render( $view_model->template(), $view_model->data(), $print );
+	public function view_model(View_Model $view_model, bool $print = true): ?string {
+		return $this->render($view_model->template(), $view_model->data(), $print);
 	}
 
 	/**
@@ -129,12 +129,12 @@ 
 	 * @param boolean                 $print
 	 * @return string|null
 	 */
-	public function partial( string $view, iterable $data = array(), bool $print = true ): ?string {
-		if ( $print ) {
-			$this->render( $view, $data, $print );
+	public function partial(string $view, iterable $data = array(), bool $print = true): ?string {
+		if ($print) {
+			$this->render($view, $data, $print);
 			return null;
 		} else {
-			return $this->render( $view, $data, $print );
+			return $this->render($view, $data, $print);
 		}
 	}
 
@@ -146,27 +146,27 @@ 
 	 * @return string
 	 * @throws Exception
 	 */
-	private function render_buffer( string $view, iterable $__data ): string {
+	private function render_buffer(string $view, iterable $__data): string {
 
-		if ( ! file_exists( $view ) ) {
-			throw new Exception( "{$view} doesn't exist" );
+		if ( ! file_exists($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 );
+			unset($__key, $__value);
 		}
 
 		// Unset the data.
-		unset( $__data );
+		unset($__data);
 
 		include $view;
 		$output = ob_get_contents();
@@ -180,12 +180,12 @@ 
 	 * @param string $filename
 	 * @return string
 	 */
-	private function resolve_file_path( string $filename ): string {
-		$filename = $this->maybe_resolve_dot_notation( $filename );
+	private function resolve_file_path(string $filename): string {
+		$filename = $this->maybe_resolve_dot_notation($filename);
 		return sprintf(
 			'%s%s.php',
 			$this->base_view_path,
-			trim( $filename )
+			trim($filename)
 		);
 	}
 
@@ -195,13 +195,13 @@ 
 	 * @param string $filename
 	 * @return string
 	 */
-	private function maybe_resolve_dot_notation( string $filename ): string {
-		if ( endsWith( '.php' )( $filename ) ) {
-			$filename = substr( $filename, 0, -4 );
+	private function maybe_resolve_dot_notation(string $filename): string {
+		if (endsWith('.php')($filename)) {
+			$filename = substr($filename, 0, -4);
 		}
 
-		$parts    = explode( '.', $filename );
-		$filename = implode( DIRECTORY_SEPARATOR, $parts );
+		$parts    = explode('.', $filename);
+		$filename = implode(DIRECTORY_SEPARATOR, $parts);
 
 		return $filename;
 	}
@@ -213,12 +213,12 @@ 
 	 * @return string
 	 * @throws Exception
 	 */
-	private function verify_view_path( string $path ): string {
-		$path = $this->maybe_resolve_dot_notation( $path );
-		$path = rtrim( $path, '/' ) . '/';
+	private function verify_view_path(string $path): string {
+		$path = $this->maybe_resolve_dot_notation($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;