Pink-Crab / Perique-Framework

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/Application/App_Factory.php

Indentation +5 added lines, -5 removed lines

@@ -61,11 +61,11 @@
 	}
 
 		/**
-	 * Gets the defined base path.
-	 *
-	 * @return string
-	 * @since 1.4.0
-	 */
+		 * Gets the defined base path.
+		 *
+		 * @return string
+		 * @since 1.4.0
+		 */
 	public function get_base_path(): string {
 		return $this->base_path;
 	}

Spacing +42 added lines, -42 removed lines

@@ -62,17 +62,17 @@ 
 	 */
 	protected array $modules = array();
 
-	public function __construct( ?string $base_path = null ) {
+	public function __construct(?string $base_path = null) {
 
-		if ( null === $base_path ) {
+		if (null === $base_path) {
 			$file_index      = 0;
 			$trace           = debug_backtrace(); //phpcs:ignore WordPress.PHP.DevelopmentFunctions.error_log_debug_backtrace
-			$this->base_path = isset( $trace[ $file_index ]['file'] ) ? \trailingslashit( dirname( $trace[ $file_index ]['file'] ) ) : __DIR__;
+			$this->base_path = isset($trace[$file_index]['file']) ? \trailingslashit(dirname($trace[$file_index]['file'])) : __DIR__;
 		} else {
-			$this->base_path = \trailingslashit( $base_path );
+			$this->base_path = \trailingslashit($base_path);
 		}
 
-		$this->app = new App( $this->base_path );
+		$this->app = new App($this->base_path);
 
 	}
 
@@ -93,11 +93,11 @@ 
 	 * @param string $base_view_path
 	 * @return self
 	 */
-	public function set_base_view_path( string $base_view_path ): self {
-		$this->base_view_path = \trailingslashit( $base_view_path );
+	public function set_base_view_path(string $base_view_path): self {
+		$this->base_view_path = \trailingslashit($base_view_path);
 
 		// Set the view base path on the app.
-		$this->app->set_view_path( $this->base_view_path );
+		$this->app->set_view_path($this->base_view_path);
 
 		return $this;
 	}
@@ -111,7 +111,7 @@ 
 	public function get_base_view_path(): string {
 		return null !== $this->base_view_path
 			? $this->base_view_path
-			: \trailingslashit( $this->default_config_paths()['path']['view'] );
+			: \trailingslashit($this->default_config_paths()['path']['view']);
 	}
 
 	/**
@@ -123,12 +123,12 @@ 
 	 * @infection-ignore-all
 	 * @return self
 	 */
-	public function with_wp_dice( bool $include_default_rules = false ): self {
+	public function with_wp_dice(bool $include_default_rules = false): self {
 		// If the view path is not set, set it to the same as base path.
-		if ( null === $this->base_view_path ) {
+		if (null === $this->base_view_path) {
 			$this->base_view_path = $this->base_path;
 		}
-		return $this->default_setup( $include_default_rules );
+		return $this->default_setup($include_default_rules);
 	}
 
 	/**
@@ -143,29 +143,29 @@ 
 	 * @param bool $include_default_rules
 	 * @return self
 	 */
-	public function default_setup( bool $include_default_rules = true ): self {
+	public function default_setup(bool $include_default_rules = true): self {
 		$loader = new Hook_Loader();
 
 		// Setup DI Container
-		$container = PinkCrab_Dice::withDice( new Dice() );
+		$container = PinkCrab_Dice::withDice(new Dice());
 
-		if ( $include_default_rules === true ) {
-			$container->addRules( $this->default_di_rules() );
+		if ($include_default_rules === true) {
+			$container->addRules($this->default_di_rules());
 		}
 
-		$this->app->set_container( $container );
-		$this->app->set_loader( $loader );
+		$this->app->set_container($container);
+		$this->app->set_loader($loader);
 
 		// Set registration middleware
-		$module_manager = new Module_Manager( $container, new Registration_Service( $container ) );
-		$module_manager->push_module( Hookable_Module::class );
+		$module_manager = new Module_Manager($container, new Registration_Service($container));
+		$module_manager->push_module(Hookable_Module::class);
 
 		// Push any modules that have been added before the module manager was set.
-		foreach ( $this->modules as $module ) {
-			$module_manager->push_module( $module[0], $module[1] );
+		foreach ($this->modules as $module) {
+			$module_manager->push_module($module[0], $module[1]);
 		}
 
-		$this->app->set_module_manager( $module_manager );
+		$this->app->set_module_manager($module_manager);
 
 		return $this;
 	}
@@ -178,15 +178,15 @@ 
 	 * @param ?callable(Module, ?Registration_Middleware):Module $callback
 	 * @return self
 	 */
-	public function module( string $module, ?callable $callback = null ): self {
+	public function module(string $module, ?callable $callback = null): self {
 
 		// If the Module_Manager has been set in app, add the module to app.
-		if ( $this->app->has_module_manager() ) {
-			$this->app->module( $module, $callback );
+		if ($this->app->has_module_manager()) {
+			$this->app->module($module, $callback);
 			return $this;
 		}
 
-		$this->modules[] = array( $module, $callback );
+		$this->modules[] = array($module, $callback);
 		return $this;
 	}
 
@@ -208,7 +208,7 @@ 
 				'shared'     => true,
 			),
 			Component_Compiler::class => array(
-				'constructParams' => array( 'components' ),
+				'constructParams' => array('components'),
 			),
 		);
 	}
@@ -219,10 +219,10 @@ 
 	 * @param array<string,array<string,string|object|callable|null|false|\Closure>> $rules
 	 * @return self
 	 */
-	public function di_rules( array $rules ): self {
+	public function di_rules(array $rules): self {
 		$this->app->container_config(
-			function( DI_Container $container ) use ( $rules ): void {
-				$container->addRules( $rules );
+			function(DI_Container $container) use ($rules): void {
+				$container->addRules($rules);
 			}
 		);
 		return $this;
@@ -234,8 +234,8 @@ 
 	 * @param array<class-string> $class_list Array of fully namespaced class names.
 	 * @return self
 	 */
-	public function registration_classes( array $class_list ): self {
-		$this->app->registration_classes( $class_list );
+	public function registration_classes(array $class_list): self {
+		$this->app->registration_classes($class_list);
 		return $this;
 	}
 
@@ -245,8 +245,8 @@ 
 	 * @param array<string, mixed> $app_config
 	 * @return self
 	 */
-	public function app_config( array $app_config ): self {
-		$this->app->set_app_config( \array_replace_recursive( $this->default_config_paths(), $app_config ) );
+	public function app_config(array $app_config): self {
+		$this->app->set_app_config(\array_replace_recursive($this->default_config_paths(), $app_config));
 		return $this;
 	}
 
@@ -266,8 +266,8 @@ 
 	 */
 	public function boot(): App {
 		// Sets default settings if not already set.
-		if ( ! $this->app->has_app_config() ) {
-			$this->app_config( $this->default_config_paths() );
+		if ( ! $this->app->has_app_config()) {
+			$this->app_config($this->default_config_paths());
 		}
 
 		return $this->app->boot();
@@ -296,8 +296,8 @@ 
 	private function default_config_paths(): array {
 		$wp_uploads = \wp_upload_dir();
 
-		$base_path = App_Config_Path_Helper::normalise_path( $this->base_path );
-		$view_path = $this->base_view_path ?? App_Config_Path_Helper::assume_view_path( $base_path );
+		$base_path = App_Config_Path_Helper::normalise_path($this->base_path);
+		$view_path = $this->base_view_path ?? App_Config_Path_Helper::assume_view_path($base_path);
 
 		return array(
 			'path' => array(
@@ -308,9 +308,9 @@ 
 				'upload_current' => $wp_uploads['path'],
 			),
 			'url'  => array(
-				'plugin'         => App_Config_Path_Helper::assume_base_url( $base_path ),
-				'view'           => App_Config_Path_Helper::assume_view_url( $base_path, $view_path ),
-				'assets'         => App_Config_Path_Helper::assume_base_url( $base_path ) . '/assets',
+				'plugin'         => App_Config_Path_Helper::assume_base_url($base_path),
+				'view'           => App_Config_Path_Helper::assume_view_url($base_path, $view_path),
+				'assets'         => App_Config_Path_Helper::assume_base_url($base_path) . '/assets',
 				'upload_root'    => $wp_uploads['baseurl'],
 				'upload_current' => $wp_uploads['url'],
 			),

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,7 +59,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;
 
 	/**
 	 * Returns the base view path.

src/Utils/App_Config_Path_Helper.php

Spacing +12 added lines, -12 removed lines

@@ -34,8 +34,8 @@ 
 	 * @param string $path
 	 * @return string
 	 */
-	public static function normalise_path( string $path ): string {
-		return rtrim( $path, '/\\' );
+	public static function normalise_path(string $path): string {
+		return rtrim($path, '/\\');
 	}
 
 	/**
@@ -44,9 +44,9 @@ 
 	 * @param string $base_path
 	 * @return string
 	 */
-	public static function assume_view_path( string $base_path ): string {
+	public static function assume_view_path(string $base_path): string {
 		// Remove any trailing slash.
-		$base_path = self::normalise_path( $base_path );
+		$base_path = self::normalise_path($base_path);
 		return $base_path . \DIRECTORY_SEPARATOR . 'views';
 	}
 
@@ -56,10 +56,10 @@ 
 	 * @param string $base_path
 	 * @return string
 	 */
-	public static function assume_base_url( string $base_path ): string {
+	public static function assume_base_url(string $base_path): string {
 		// Remove any trailing slash.
-		$base_path = self::normalise_path( $base_path );
-		return plugins_url( basename( $base_path ) );
+		$base_path = self::normalise_path($base_path);
+		return plugins_url(basename($base_path));
 	}
 
 	/**
@@ -69,15 +69,15 @@ 
 	 * @param string $view_path
 	 * @return string
 	 */
-	public static function assume_view_url( string $base_path, string $view_path ): string {
+	public static function assume_view_url(string $base_path, string $view_path): string {
 
-		$base_path = self::normalise_path( $base_path );
-		$view_path = self::normalise_path( $view_path );
+		$base_path = self::normalise_path($base_path);
+		$view_path = self::normalise_path($view_path);
 
 		// Remove any trailing slash.
-		$diff = ltrim( str_replace( $base_path, '', $view_path ), '/\\' );
+		$diff = ltrim(str_replace($base_path, '', $view_path), '/\\');
 
 		// Return the base url with the diff.
-		return self::assume_base_url( $base_path ) . '/' . $diff;
+		return self::assume_base_url($base_path) . '/' . $diff;
 	}
 }

src/Utils/Object_Helper.php

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@
 	 * @param string $class
 	 * @return bool
 	 */
-	public static function is_a( $value, string $class ): bool {
-		return is_object( $value ) && is_a( $value, $class );
+	public static function is_a($value, string $class): bool {
+		return is_object($value) && is_a($value, $class);
 	}
 }

src/Application/App.php

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)),
 				),
 			)
 		);
@@ -362,15 +362,15 @@ 
 		$this->module_manager->register_modules(); // @phpstan-ignore-line, already verified if not null
 
 		/** @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
@@ -389,11 +389,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
 	}
 
 	/**
@@ -404,11 +404,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);
 	}
 
 	/**
@@ -418,11 +418,11 @@ 
 	 * @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
 	}
 
 	/** @return array{
@@ -450,7 +450,7 @@ 
 	 * @return bool
 	 */
 	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);
 	}
 
 	/**
@@ -460,7 +460,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/Application/App_Validation.php

Spacing +8 added lines, -8 removed lines

@@ -43,7 +43,7 @@ 
 	/** @var App */
 	private App $app;
 
-	public function __construct( App $app ) {
+	public function __construct(App $app) {
 		$this->app = $app;
 	}
 
@@ -55,7 +55,7 @@ 
 	public function validate(): bool {
 		$this->already_booted();
 		$this->validate_properties_set();
-		return count( $this->errors ) === 0;
+		return count($this->errors) === 0;
 	}
 
 	/**
@@ -65,11 +65,11 @@ 
 	 * @return void
 	 */
 	private function validate_properties_set(): void {
-		foreach ( $this->required_properties as $property => $is_static ) {
-			$property_reflection = new ReflectionProperty( $this->app, $property );
-			$property_reflection->setAccessible( true );
-			if ( empty( $property_reflection->getValue( $this->app ) ) ) {
-				$this->errors[] = \sprintf( self::ERROR_MESSAGE_TEMPLATE, $property );
+		foreach ($this->required_properties as $property => $is_static) {
+			$property_reflection = new ReflectionProperty($this->app, $property);
+			$property_reflection->setAccessible(true);
+			if (empty($property_reflection->getValue($this->app))) {
+				$this->errors[] = \sprintf(self::ERROR_MESSAGE_TEMPLATE, $property);
 			}
 		}
 	}
@@ -80,7 +80,7 @@ 
 	 * @return void
 	 */
 	private function already_booted(): void {
-		if ( $this->app->is_booted() === true ) {
+		if ($this->app->is_booted() === true) {
 			$this->errors[] = self::ERROR_MESSAGE_APP_BOOTED;
 		}
 	}

src/Application/App_Config.php

Spacing +69 added lines, -69 removed lines

@@ -89,9 +89,9 @@ 
 	/**
 	 * @param array<string, mixed> $settings
 	 */
-	public function __construct( array $settings = array() ) {
-		$settings = $this->set_defaults( $settings );
-		$this->set_props( $settings );
+	public function __construct(array $settings = array()) {
+		$settings = $this->set_defaults($settings);
+		$this->set_props($settings);
 	}
 
 	/**
@@ -100,8 +100,8 @@ 
 	 * @param array<string, mixed> $settings
 	 * @return array<string, mixed>
 	 */
-	private function set_defaults( array $settings ): array {
-		return array_replace_recursive( $this->settings_defaults(), $settings );
+	private function set_defaults(array $settings): array {
+		return array_replace_recursive($this->settings_defaults(), $settings);
 	}
 
 	/**
@@ -110,17 +110,17 @@ 
 	 * @param array<string, mixed> $settings
 	 * @return void
 	 */
-	private function set_props( array $settings ): void {
+	private function set_props(array $settings): void {
 		$this->paths['url']  = $settings['url'];
 		$this->paths['path'] = $settings['path'];
-		$this->namespaces    = $this->filter_key_value_pair( $settings['namespaces'] );
+		$this->namespaces    = $this->filter_key_value_pair($settings['namespaces']);
 		$this->plugin        = $settings['plugin'];
 		$this->additional    = $settings['additional'];
-		$this->db_tables     = $this->filter_key_value_pair( $settings['db_tables'] );
-		$this->post_types    = $this->filter_key_value_pair( $settings['post_types'] );
-		$this->taxonomies    = $this->filter_key_value_pair( $settings['taxonomies'] );
+		$this->db_tables     = $this->filter_key_value_pair($settings['db_tables']);
+		$this->post_types    = $this->filter_key_value_pair($settings['post_types']);
+		$this->taxonomies    = $this->filter_key_value_pair($settings['taxonomies']);
 
-		$this->set_meta( $settings['meta'] );
+		$this->set_meta($settings['meta']);
 	}
 
 	/**
@@ -130,14 +130,14 @@ 
 	 * @param string|null $default The default value to return if not set.
 	 * @return array<string, mixed>|string|null
 	 */
-	public function path( ?string $path = null, ?string $default = null ) {
+	public function path(?string $path = null, ?string $default = null) {
 
-		if ( is_null( $path ) ) {
+		if (is_null($path)) {
 			return $this->paths['path'];
 		}
 
-		return \array_key_exists( $path, $this->paths['path'] )
-			? trailingslashit( $this->paths['path'][ $path ] )
+		return \array_key_exists($path, $this->paths['path'])
+			? trailingslashit($this->paths['path'][$path])
 			: $default;
 	}
 
@@ -148,14 +148,14 @@ 
 	 * @param string|null $default The default value to return if not set.
 	 * @return array<string, mixed>|string|null
 	 */
-	public function url( ?string $url = null, ?string $default = null ) {
+	public function url(?string $url = null, ?string $default = null) {
 
-		if ( is_null( $url ) ) {
+		if (is_null($url)) {
 			return $this->paths['url'];
 		}
 
-		return \array_key_exists( $url, $this->paths['url'] )
-			? trailingslashit( $this->paths['url'][ $url ] )
+		return \array_key_exists($url, $this->paths['url'])
+			? trailingslashit($this->paths['url'][$url])
 			: $default;
 	}
 
@@ -184,9 +184,9 @@ 
 	 * @param string|null $default The default value to return if not set.
 	 * @return string|null
 	 */
-	public function namespace( string $key, ?string $default = null ): ?string {
-		return array_key_exists( $key, $this->namespaces )
-			? $this->namespaces[ $key ] : $default;
+	public function namespace(string $key, ?string $default = null) : ?string {
+		return array_key_exists($key, $this->namespaces)
+			? $this->namespaces[$key] : $default;
 	}
 
 	/**
@@ -196,9 +196,9 @@ 
 	 * @param string|null $default The default value to return if not set.
 	 * @return mixed
 	 */
-	public function additional( string $key, ?string $default = null ) {
-		return array_key_exists( $key, $this->additional )
-			? $this->additional[ $key ] : $default;
+	public function additional(string $key, ?string $default = null) {
+		return array_key_exists($key, $this->additional)
+			? $this->additional[$key] : $default;
 	}
 
 	/**
@@ -226,12 +226,12 @@ 
 	 * @return string
 	 * @throws OutOfBoundsException
 	 */
-	public function post_types( string $key ) {
-		if ( ! array_key_exists( $key, $this->post_types ) ) {
-			throw new OutOfBoundsException( "App Config :: \"{$key}\" is not a defined post type" );
+	public function post_types(string $key) {
+		if ( ! array_key_exists($key, $this->post_types)) {
+			throw new OutOfBoundsException("App Config :: \"{$key}\" is not a defined post type");
 		}
 
-		return $this->post_types[ $key ];
+		return $this->post_types[$key];
 	}
 
 	/**
@@ -242,17 +242,17 @@ 
 	 * @return string
 	 * @throws OutOfBoundsException
 	 */
-	public function meta( string $key, string $type = self::POST_META ): string {
+	public function meta(string $key, string $type = self::POST_META): string {
 		// Check meta type.
-		if ( ! array_key_exists( $type, $this->meta ) ) {
-			throw new OutOfBoundsException( "App Config :: \"{$type}\" is not a valid meta type and cant be fetched" );
+		if ( ! array_key_exists($type, $this->meta)) {
+			throw new OutOfBoundsException("App Config :: \"{$type}\" is not a valid meta type and cant be fetched");
 		}
 		// Check key.
-		if ( ! array_key_exists( $key, $this->meta[ $type ] ) ) {
-			throw new OutOfBoundsException( "App Config :: \"{$key}\" is not a defined {$type} meta key" );
+		if ( ! array_key_exists($key, $this->meta[$type])) {
+			throw new OutOfBoundsException("App Config :: \"{$key}\" is not a defined {$type} meta key");
 		}
 
-		return $this->meta[ $type ][ $key ];
+		return $this->meta[$type][$key];
 	}
 
 	/**
@@ -262,8 +262,8 @@ 
 	 * @param string $key
 	 * @return string
 	 */
-	public function post_meta( string $key ): string {
-		return $this->meta( $key, self::POST_META );
+	public function post_meta(string $key): string {
+		return $this->meta($key, self::POST_META);
 	}
 
 	/**
@@ -273,8 +273,8 @@ 
 	 * @param string $key
 	 * @return string
 	 */
-	public function user_meta( string $key ): string {
-		return $this->meta( $key, self::USER_META );
+	public function user_meta(string $key): string {
+		return $this->meta($key, self::USER_META);
 	}
 
 	/**
@@ -284,8 +284,8 @@ 
 	 * @param string $key
 	 * @return string
 	 */
-	public function term_meta( string $key ): string {
-		return $this->meta( $key, self::TERM_META );
+	public function term_meta(string $key): string {
+		return $this->meta($key, self::TERM_META);
 	}
 
 	/**
@@ -294,15 +294,15 @@ 
 	 * @param array<string, array<string,string>> $meta
 	 * @return void
 	 */
-	public function set_meta( array $meta ): void {
-		$valid_meta_types = array( self::POST_META, self::USER_META, self::TERM_META );
-		foreach ( $meta as $meta_type => $pairs ) {
-			if ( ! in_array( $meta_type, $valid_meta_types, true ) ) {
-				throw new OutOfBoundsException( "App Config :: \"{$meta_type}\" is not a valid meta type and cant be defined" );
+	public function set_meta(array $meta): void {
+		$valid_meta_types = array(self::POST_META, self::USER_META, self::TERM_META);
+		foreach ($meta as $meta_type => $pairs) {
+			if ( ! in_array($meta_type, $valid_meta_types, true)) {
+				throw new OutOfBoundsException("App Config :: \"{$meta_type}\" is not a valid meta type and cant be defined");
 			}
 
 			// Set all pairs which have both valid key and values.
-			$this->meta[ $meta_type ] = $this->filter_key_value_pair( $pairs );
+			$this->meta[$meta_type] = $this->filter_key_value_pair($pairs);
 		}
 	}
 
@@ -313,12 +313,12 @@ 
 	 * @return string
 	 * @throws OutOfBoundsException
 	 */
-	public function taxonomies( string $key ): string {
-		if ( ! array_key_exists( $key, $this->taxonomies ) ) {
-			throw new OutOfBoundsException( "App Config :: \"{$key}\" is not a defined taxonomy" );
+	public function taxonomies(string $key): string {
+		if ( ! array_key_exists($key, $this->taxonomies)) {
+			throw new OutOfBoundsException("App Config :: \"{$key}\" is not a defined taxonomy");
 		}
 
-		return $this->taxonomies[ $key ];
+		return $this->taxonomies[$key];
 	}
 
 
@@ -329,11 +329,11 @@ 
 	 * @return string
 	 * @throws OutOfBoundsException
 	 */
-	public function db_tables( string $name ): string {
-		if ( ! array_key_exists( $name, $this->db_tables ) ) {
-			throw new OutOfBoundsException( "App Config :: \"{$name}\" is not a defined DB table" );
+	public function db_tables(string $name): string {
+		if ( ! array_key_exists($name, $this->db_tables)) {
+			throw new OutOfBoundsException("App Config :: \"{$name}\" is not a defined DB table");
 		}
-		return $this->db_tables[ $name ];
+		return $this->db_tables[$name];
 	}
 
 	/**
@@ -342,8 +342,8 @@ 
 	 * @param string $name
 	 * @return mixed
 	 */
-	public function __get( $name ) {
-		return $this->additional( $name );
+	public function __get($name) {
+		return $this->additional($name);
 	}
 
 	/**
@@ -352,11 +352,11 @@ 
 	 * @return array<string, mixed>
 	 */
 	private function settings_defaults(): array {
-		$base_path  = \dirname( __DIR__, 2 );
+		$base_path  = \dirname(__DIR__, 2);
 		$wp_uploads = \wp_upload_dir();
 
-		$base_path = App_Config_Path_Helper::normalise_path( $base_path );
-		$view_path = App_Config_Path_Helper::assume_view_path( $base_path );
+		$base_path = App_Config_Path_Helper::normalise_path($base_path);
+		$view_path = App_Config_Path_Helper::assume_view_path($base_path);
 
 		global $wpdb;
 
@@ -373,9 +373,9 @@ 
 				'upload_current' => $wp_uploads['path'],
 			),
 			'url'        => array(
-				'plugin'         => App_Config_Path_Helper::assume_base_url( $base_path ),
-				'view'           => App_Config_Path_Helper::assume_view_url( $base_path, $view_path ),
-				'assets'         => App_Config_Path_Helper::assume_base_url( $base_path ) . '/assets',
+				'plugin'         => App_Config_Path_Helper::assume_base_url($base_path),
+				'view'           => App_Config_Path_Helper::assume_view_url($base_path, $view_path),
+				'assets'         => App_Config_Path_Helper::assume_base_url($base_path) . '/assets',
 				'upload_root'    => $wp_uploads['baseurl'],
 				'upload_current' => $wp_uploads['url'],
 			),
@@ -397,15 +397,15 @@ 
 	 * @param array<int|string, mixed> $pairs
 	 * @return array<string, string>
 	 */
-	private function filter_key_value_pair( array $pairs ): array {
+	private function filter_key_value_pair(array $pairs): array {
 		/** @var array<string, string> (as per filter function)*/
 		return array_filter(
 			$pairs,
-			function ( $value, $key ): bool {
-				return is_string( $value )
-				&& \strlen( $value ) > 0
-				&& is_string( $key )
-				&& \strlen( $key ) > 0;
+			function($value, $key): bool {
+				return is_string($value)
+				&& \strlen($value) > 0
+				&& is_string($key)
+				&& \strlen($key) > 0;
 			},
 			ARRAY_FILTER_USE_BOTH
 		);

src/Exceptions/App_Initialization_Exception.php

Spacing +12 added lines, -12 removed lines

@@ -36,7 +36,7 @@ 
 	 */
 	public static function requires_di_container(): App_Initialization_Exception {
 		$message = 'The Application must be populated with a DI_Container before booting.';
-		return new App_Initialization_Exception( $message, 1 );
+		return new App_Initialization_Exception($message, 1);
 	}
 
 	/**
@@ -46,7 +46,7 @@ 
 	 */
 	public static function di_container_exists(): App_Initialization_Exception {
 		$message = 'App already contains a DI Container, can not redeclare.';
-		return new App_Initialization_Exception( $message, 2 );
+		return new App_Initialization_Exception($message, 2);
 	}
 
 	/**
@@ -56,7 +56,7 @@ 
 	 */
 	public static function requires_module_manager(): App_Initialization_Exception {
 		$message = 'App has not defined Registration Service, this must be set before use.';
-		return new App_Initialization_Exception( $message, 3 );
+		return new App_Initialization_Exception($message, 3);
 	}
 
 	/**
@@ -65,9 +65,9 @@ 
 	 * @param string $service The service which has been called without initialising the app.
 	 * @return App_Initialization_Exception
 	 */
-	public static function app_not_initialized( string $service ): App_Initialization_Exception {
+	public static function app_not_initialized(string $service): App_Initialization_Exception {
 		$message = "App must be initialised before calling {$service}";
-		return new App_Initialization_Exception( $message, 4 );
+		return new App_Initialization_Exception($message, 4);
 	}
 
 	/**
@@ -77,7 +77,7 @@ 
 	 */
 	public static function app_config_exists(): App_Initialization_Exception {
 		$message = 'Can not redeclare App_Config as its already set to the application';
-		return new App_Initialization_Exception( $message, 5 );
+		return new App_Initialization_Exception($message, 5);
 	}
 
 	/**
@@ -87,7 +87,7 @@ 
 	 */
 	public static function registration_exists(): App_Initialization_Exception {
 		$message = 'Can not redeclare Registration_Service as its already set to the application';
-		return new App_Initialization_Exception( $message, 7 );
+		return new App_Initialization_Exception($message, 7);
 	}
 
 	/**
@@ -96,12 +96,12 @@ 
 	 * @param array<int,string> $errors
 	 * @return App_Initialization_Exception
 	 */
-	public static function failed_boot_validation( array $errors ): App_Initialization_Exception {
+	public static function failed_boot_validation(array $errors): App_Initialization_Exception {
 		$message = sprintf(
 			'App failed boot validation : %s',
-			join( ', ', $errors )
+			join(', ', $errors)
 		);
-		return new App_Initialization_Exception( $message, 6 );
+		return new App_Initialization_Exception($message, 6);
 	}
 
 	/**
@@ -111,7 +111,7 @@ 
 	 */
 	public static function loader_exists(): App_Initialization_Exception {
 		$message = 'Can not redeclare Loader as its already set to the application';
-		return new App_Initialization_Exception( $message, 8 );
+		return new App_Initialization_Exception($message, 8);
 	}
 
 	/**
@@ -121,7 +121,7 @@ 
 	 */
 	public static function module_manager_exists(): App_Initialization_Exception {
 		$message = 'Can not redeclare Module_Manager as its already set to the application';
-		return new App_Initialization_Exception( $message, 10 );
+		return new App_Initialization_Exception($message, 10);
 	}
 
 }