Pink-Crab / Perique-Framework

src/Application/App.php

Spacing +50 added lines, -50 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,9 +280,9 @@ 
 	public function boot(): self {
 
 		// Validate.
-		$validate = new App_Validation( $this );
-		if ( $validate->validate() === false ) {
-			throw App_Initialization_Exception::failed_boot_validation( array_map( 'esc_attr', $validate->errors ) );
+		$validate = new App_Validation($this);
+		if ($validate->validate() === false) {
+			throw App_Initialization_Exception::failed_boot_validation(array_map('esc_attr', $validate->errors));
 		}
 
 		// Run the final process, where all are loaded in via
@@ -331,7 +331,7 @@ 
 			Inject_Hook_Loader::class,
 			array(
 				'call' => array(
-					array( 'set_hook_loader', array( $this->loader ) ),
+					array('set_hook_loader', array($this->loader)),
 				),
 			)
 		);
@@ -341,7 +341,7 @@ 
 			Inject_App_Config::class,
 			array(
 				'call' => array(
-					array( 'set_app_config', array( self::$app_config ) ),
+					array('set_app_config', array(self::$app_config)),
 				),
 			)
 		);
@@ -351,7 +351,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 @@ 
 		/**
  * @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
@@ -390,11 +390,11 @@ 
 	 * @return object|null
 	 * @throws App_Initialization_Exception Code 4
 	 */
-	public static function make( string $class_string, array $args = array() ): ?object {
-		if ( self::$booted === false ) {
-			throw App_Initialization_Exception::app_not_initialized( DI_Container::class );
+	public static function make(string $class_string, array $args = array()): ?object {
+		if (self::$booted === false) {
+			throw App_Initialization_Exception::app_not_initialized(DI_Container::class);
 		}
-		return self::$container->create( $class_string, $args ); // @phpstan-ignore-line, already verified if not null
+		return self::$container->create($class_string, $args); // @phpstan-ignore-line, already verified if not null
 	}
 
 	/**
@@ -405,11 +405,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);
 	}
 
 	/**
@@ -419,13 +419,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
 	}
 
 	/**
@@ -457,7 +457,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);
 	}
 
 	/**
@@ -467,7 +467,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_Config.php

Spacing +81 added lines, -81 removed lines

@@ -95,9 +95,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);
 	}
 
 	/**
@@ -106,8 +106,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);
 	}
 
 	/**
@@ -117,17 +117,17 @@ 
 	 *
 	 * @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']);
 	}
 
 	/**
@@ -138,14 +138,14 @@ 
 	 *
 	 * @return array<string, mixed>|string|null
 	 */
-	public function path( ?string $path = null, ?string $default_value = null ) {
+	public function path(?string $path = null, ?string $default_value = 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_value;
 	}
 
@@ -157,14 +157,14 @@ 
 	 *
 	 * @return array<string, mixed>|string|null
 	 */
-	public function url( ?string $url = null, ?string $default_value = null ) {
+	public function url(?string $url = null, ?string $default_value = 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_value;
 	}
 
@@ -194,9 +194,9 @@ 
 	 *
 	 * @return string|null
 	 */
-	public function namespace( string $key, ?string $default_value = null ): ?string {
-		return array_key_exists( $key, $this->namespaces )
-			? $this->namespaces[ $key ] : $default_value;
+	public function namespace(string $key, ?string $default_value = null) : ?string {
+		return array_key_exists($key, $this->namespaces)
+			? $this->namespaces[$key] : $default_value;
 	}
 
 	/**
@@ -207,9 +207,9 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function additional( string $key, ?string $default_value = null ) {
-		return array_key_exists( $key, $this->additional )
-			? $this->additional[ $key ] : $default_value;
+	public function additional(string $key, ?string $default_value = null) {
+		return array_key_exists($key, $this->additional)
+			? $this->additional[$key] : $default_value;
 	}
 
 	/**
@@ -237,12 +237,12 @@ 
 	 * @return string
 	 * @throws OutOfBoundsException
 	 */
-	public function post_types( string $key ) {
-		if ( ! array_key_exists( $key, $this->post_types ) ) {
-			throw new OutOfBoundsException( 'App Config :: "' . esc_html( $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 :: "' . esc_html($key) . '" is not a defined post type');
 		}
 
-		return $this->post_types[ $key ];
+		return $this->post_types[$key];
 	}
 
 	/**
@@ -253,17 +253,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 :: "' . esc_html( $type ) . '" is not a valid meta type and cant be fetched' );
+		if ( ! array_key_exists($type, $this->meta)) {
+			throw new OutOfBoundsException('App Config :: "' . esc_html($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 :: "' . esc_html( $key ) . '" is not a defined ' . esc_html( $type ) . 'meta key' );
+		if ( ! array_key_exists($key, $this->meta[$type])) {
+			throw new OutOfBoundsException('App Config :: "' . esc_html($key) . '" is not a defined ' . esc_html($type) . 'meta key');
 		}
 
-		return $this->meta[ $type ][ $key ];
+		return $this->meta[$type][$key];
 	}
 
 	/**
@@ -273,8 +273,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);
 	}
 
 	/**
@@ -284,8 +284,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);
 	}
 
 	/**
@@ -295,8 +295,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);
 	}
 
 	/**
@@ -305,15 +305,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 :: "' . esc_html( $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 :: "' . esc_html($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);
 		}
 	}
 
@@ -324,12 +324,12 @@ 
 	 * @return string
 	 * @throws OutOfBoundsException
 	 */
-	public function taxonomies( string $key ): string {
-		if ( ! array_key_exists( $key, $this->taxonomies ) ) {
-			throw new OutOfBoundsException( 'App Config :: "' . esc_html( $key ) . '" is not a defined taxonomy' );
+	public function taxonomies(string $key): string {
+		if ( ! array_key_exists($key, $this->taxonomies)) {
+			throw new OutOfBoundsException('App Config :: "' . esc_html($key) . '" is not a defined taxonomy');
 		}
 
-		return $this->taxonomies[ $key ];
+		return $this->taxonomies[$key];
 	}
 
 
@@ -340,11 +340,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 :: "' . esc_html( $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 :: "' . esc_html($name) . '" is not a defined DB table');
 		}
-		return $this->db_tables[ $name ];
+		return $this->db_tables[$name];
 	}
 
 	/**
@@ -353,8 +353,8 @@ 
 	 * @param string $name
 	 * @return mixed
 	 */
-	public function __get( $name ) {
-		return $this->additional( $name );
+	public function __get($name) {
+		return $this->additional($name);
 	}
 
 	/**
@@ -363,11 +363,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;
 
@@ -384,9 +384,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'],
 			),
@@ -408,17 +408,17 @@ 
 	 * @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
 		);
@@ -449,8 +449,8 @@ 
 	 * @return string
 	 */
 	public function asset_path(): string {
-		$paths = $this->path( 'assets' );
-		return is_string( $paths ) ? $paths : '';
+		$paths = $this->path('assets');
+		return is_string($paths) ? $paths : '';
 	}
 
 	/**
@@ -459,8 +459,8 @@ 
 	 * @return string
 	 */
 	public function view_path(): string {
-		$paths = $this->path( 'view' );
-		return is_string( $paths ) ? $paths : '';
+		$paths = $this->path('view');
+		return is_string($paths) ? $paths : '';
 	}
 
 	/**
@@ -469,8 +469,8 @@ 
 	 * @return string
 	 */
 	public function plugin_path(): string {
-		$paths = $this->path( 'plugin' );
-		return is_string( $paths ) ? $paths : '';
+		$paths = $this->path('plugin');
+		return is_string($paths) ? $paths : '';
 	}
 
 	/**
@@ -479,8 +479,8 @@ 
 	 * @return string
 	 */
 	public function asset_url(): string {
-		$url = $this->url( 'assets' );
-		return is_string( $url ) ? $url : '';
+		$url = $this->url('assets');
+		return is_string($url) ? $url : '';
 	}
 
 	/**
@@ -489,8 +489,8 @@ 
 	 * @return string
 	 */
 	public function view_url(): string {
-		$url = $this->url( 'view' );
-		return is_string( $url ) ? $url : '';
+		$url = $this->url('view');
+		return is_string($url) ? $url : '';
 	}
 
 	/**
@@ -499,7 +499,7 @@ 
 	 * @return string
 	 */
 	public function plugin_url(): string {
-		$url = $this->url( 'plugin' );
-		return is_string( $url ) ? $url : '';
+		$url = $this->url('plugin');
+		return is_string($url) ? $url : '';
 	}
 }

src/Application/App_Factory.php

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);
 	}
 
 		/**
@@ -92,11 +92,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;
 	}
@@ -110,7 +110,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']);
 	}
 
 	/**
@@ -122,12 +122,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);
 	}
 
 	/**
@@ -142,29 +142,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;
 	}
@@ -177,15 +177,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;
 	}
 
@@ -207,7 +207,7 @@ 
 				'shared'     => true,
 			),
 			Component_Compiler::class => array(
-				'constructParams' => array( 'components' ),
+				'constructParams' => array('components'),
 			),
 		);
 	}
@@ -218,10 +218,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;
@@ -233,8 +233,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;
 	}
 
@@ -244,8 +244,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;
 	}
 
@@ -265,8 +265,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();
@@ -295,8 +295,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(
@@ -307,9 +307,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/Services/Dice/PinkCrab_Dice.php

Spacing +21 added lines, -21 removed lines

@@ -43,7 +43,7 @@ 
 	 *
 	 * @param Dice $dice
 	 */
-	public function __construct( Dice $dice ) {
+	public function __construct(Dice $dice) {
 		$this->dice = $dice;
 	}
 
@@ -53,8 +53,8 @@ 
 	 * @param Dice $dice
 	 * @return self
 	 */
-	public static function withDice( Dice $dice ): self { // phpcs:disable WordPress.NamingConventions.ValidFunctionName.MethodNameInvalid
-		return new PinkCrab_Dice( $dice );
+	public static function withDice(Dice $dice): self { // phpcs:disable WordPress.NamingConventions.ValidFunctionName.MethodNameInvalid
+		return new PinkCrab_Dice($dice);
 	}
 
 	/**
@@ -64,11 +64,11 @@ 
 	 * @param string $id Class name (fully namespaced.)
 	 * @return object|null
 	 */
-	public function get( string $id ): ?object {
-		if ( ! $this->has( $id ) ) {
-			throw new DI_Container_Exception( esc_html( "{$id} not defined in container" ), 1 );
+	public function get(string $id): ?object {
+		if ( ! $this->has($id)) {
+			throw new DI_Container_Exception(esc_html("{$id} not defined in container"), 1);
 		}
-		return $this->create( $id );
+		return $this->create($id);
 	}
 
 	/**
@@ -78,21 +78,21 @@ 
 	 * @param string $id Class name (fully namespaced.)
 	 * @return boolean
 	 */
-	public function has( string $id ): bool {
-		$from_dice = $this->dice->getRule( $id );
+	public function has(string $id): bool {
+		$from_dice = $this->dice->getRule($id);
 		// If set in global rules.
-		if ( array_key_exists( 'substitutions', $from_dice )
-		&& array_key_exists( $id, $from_dice['substitutions'] ) ) {
+		if (array_key_exists('substitutions', $from_dice)
+		&& array_key_exists($id, $from_dice['substitutions'])) {
 			return true;
 		}
 
 		// If set with a replacement instance.
-		if ( array_key_exists( 'instanceOf', $from_dice ) ) {
+		if (array_key_exists('instanceOf', $from_dice)) {
 			return true;
 		}
 
 		// Checks if the class exists
-		return class_exists( $id );
+		return class_exists($id);
 	}
 
 	/**
@@ -102,8 +102,8 @@ 
 	 * @param array<string, string|object|mixed[]> $rule
 	 * @return PinkCrab_Dice
 	 */
-	public function addRule( string $name, array $rule ): DI_Container { // phpcs:disable WordPress.NamingConventions.ValidFunctionName.MethodNameInvalid
-		$this->dice = $this->dice->addRule( $name, $rule );
+	public function addRule(string $name, array $rule): DI_Container { // phpcs:disable WordPress.NamingConventions.ValidFunctionName.MethodNameInvalid
+		$this->dice = $this->dice->addRule($name, $rule);
 		return $this;
 	}
 
@@ -113,8 +113,8 @@ 
 	 * @param array<string, mixed[]> $rules
 	 * @return PinkCrab_Dice
 	 */
-	public function addRules( array $rules ): DI_Container { // phpcs:disable WordPress.NamingConventions.ValidFunctionName.MethodNameInvalid
-		$this->dice = $this->dice->addRules( apply_filters( Hooks::APP_INIT_SET_DI_RULES, $rules ) );
+	public function addRules(array $rules): DI_Container { // phpcs:disable WordPress.NamingConventions.ValidFunctionName.MethodNameInvalid
+		$this->dice = $this->dice->addRules(apply_filters(Hooks::APP_INIT_SET_DI_RULES, $rules));
 		return $this;
 	}
 
@@ -125,8 +125,8 @@ 
 	 * @param array<mixed> $args
 	 * @return object|null
 	 */
-	public function create( string $name, array $args = array() ): ?object {
-		return $this->dice->create( $name, $args );
+	public function create(string $name, array $args = array()): ?object {
+		return $this->dice->create($name, $args);
 	}
 
 	/**
@@ -135,7 +135,7 @@ 
 	 * @param string $name
 	 * @return array<mixed>
 	 */
-	public function getRule( string $name ): ?array {
-		return $this->dice->getRule( $name );
+	public function getRule(string $name): ?array {
+		return $this->dice->getRule($name);
 	}
 }

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,26 +91,26 @@ 
 	 * @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.
+		$compiled = $this->component_compiler->compile($component); // @phpstan-ignore-line, checked above.
 		$template = $compiled->template();
 
-		$view = file_exists( $template )
+		$view = file_exists($template)
 			? $template
-			: sprintf( '%s%s%s.php', $this->base_view_path, \DIRECTORY_SEPARATOR, trim( $this->maybe_resolve_dot_notation( $template ) ) );
+			: sprintf('%s%s%s.php', $this->base_view_path, \DIRECTORY_SEPARATOR, trim($this->maybe_resolve_dot_notation($template)));
 
-		if ( $print ) {
-			print( $this->render_buffer( $view, $compiled->data() ) ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
+		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());
 		}
 	}
 
@@ -120,8 +120,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);
 	}
 
 	/**
@@ -132,12 +132,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);
 		}
 	}
 
@@ -149,27 +149,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( esc_html( "{$view} doesn't exist" ) );
+		if ( ! file_exists($view)) {
+			throw new Exception(esc_html("{$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();
@@ -183,12 +183,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)
 		);
 	}
 
@@ -198,13 +198,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;
 	}
@@ -216,12 +216,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( esc_html( "{$path} doesn't exist and cant be used as the base view path." ) );
+		if ( ! \is_dir($path)) {
+			throw new Exception(esc_html("{$path} doesn't exist and cant be used as the base view path."));
 		}
 
 		return $path;

src/Services/Registration/Module_Manager.php

Spacing +31 added lines, -31 removed lines

@@ -67,7 +67,7 @@ 
 	 *
 	 * @param DI_Container $di_container
 	 */
-	public function __construct( DI_Container $di_container, Registration_Service $registration_service ) {
+	public function __construct(DI_Container $di_container, Registration_Service $registration_service) {
 		$this->di_container = $di_container;
 
 		// Create the registration service.
@@ -82,8 +82,8 @@ 
 	 * @param callable(Module, ?Registration_Middleware):Module $config
 	 * @return void
 	 */
-	public function push_module( string $module_name, ?callable $config = null ): void {
-		$this->modules[] = array( $module_name, $config );
+	public function push_module(string $module_name, ?callable $config = null): void {
+		$this->modules[] = array($module_name, $config);
 	}
 
 	/**
@@ -91,8 +91,8 @@ 
 	 *
 	 * @param class-string $class_string The class to register.
 	 */
-	public function register_class( string $class_string ): void {
-		$this->registration_service->push_class( $class_string );
+	public function register_class(string $class_string): void {
+		$this->registration_service->push_class($class_string);
 	}
 
 	/**
@@ -104,26 +104,26 @@ 
 	 */
 	public function register_modules(): void {
 		// Allow for additional apps to hook into the Module Manager.
-		do_action( Hooks::MODULE_MANAGER, $this );
+		do_action(Hooks::MODULE_MANAGER, $this);
 
-		foreach ( $this->modules as list($module_name, $config) ) {
+		foreach ($this->modules as list($module_name, $config)) {
 			// Create the instance.
-			$module = $this->create_module( $module_name );
+			$module = $this->create_module($module_name);
 
 			// Create the middleware.
-			$middleware = $this->create_middleware( $module );
+			$middleware = $this->create_middleware($module);
 
 			// If a config is provided, call it.
-			if ( ! is_null( $config ) ) {
-				$module = $config( $module, $middleware );
+			if ( ! is_null($config)) {
+				$module = $config($module, $middleware);
 			}
 
 			// Add to the modules and register all hooks.
-			$this->register_hooks( $module );
+			$this->register_hooks($module);
 
 			// Add to the middleware, if provided.
-			if ( ! is_null( $middleware ) ) {
-				$this->registration_service->push_middleware( $middleware );
+			if ( ! is_null($middleware)) {
+				$this->registration_service->push_middleware($middleware);
 			}
 		}
 	}
@@ -134,14 +134,14 @@ 
 	 * @param class-string<Module> $module
 	 * @return Module
 	 */
-	private function create_module( string $module ): Module {
-		$instance = $this->di_container->create( $module );
+	private function create_module(string $module): Module {
+		$instance = $this->di_container->create($module);
 
 		// If not an object or not an instance of the module interface, throw.
-		if ( ! is_object( $instance )
-		|| ! is_a( $instance, Module::class, true )
+		if ( ! is_object($instance)
+		|| ! is_a($instance, Module::class, true)
 		) {
-			throw Module_Manager_Exception::invalid_module_class_name( esc_html( $module ) );
+			throw Module_Manager_Exception::invalid_module_class_name(esc_html($module));
 		}
 
 		return $instance;
@@ -153,27 +153,27 @@ 
 	 * @param Module $module
 	 * @return Registration_Middleware|null
 	 */
-	private function create_middleware( Module $module ): ?Registration_Middleware {
+	private function create_middleware(Module $module): ?Registration_Middleware {
 		$middleware = $module->get_middleware();
 
 		// If no middleware is provided, return null.
-		if ( is_null( $middleware ) ) {
+		if (is_null($middleware)) {
 			return null;
 		}
 
 		// If not an object or not an instance of the module interface, throw.
-		if ( ! is_a( $middleware, Registration_Middleware::class, true ) ) {
-			throw Module_Manager_Exception::invalid_registration_middleware( $middleware ); // phpcs:ignore WordPress.Security.EscapeOutput.ExceptionNotEscaped, escaped in exception.
+		if ( ! is_a($middleware, Registration_Middleware::class, true)) {
+			throw Module_Manager_Exception::invalid_registration_middleware($middleware); // phpcs:ignore WordPress.Security.EscapeOutput.ExceptionNotEscaped, escaped in exception.
 		}
 
 		// Create the middleware.
-		$middleware = $this->di_container->create( $middleware );
+		$middleware = $this->di_container->create($middleware);
 
 		// If the middleware is not an object, throw.
-		if ( ! is_object( $middleware )
-		|| ! is_a( $middleware, Registration_Middleware::class, true )
+		if ( ! is_object($middleware)
+		|| ! is_a($middleware, Registration_Middleware::class, true)
 		) {
-			throw Module_Manager_Exception::failed_to_create_registration_middleware( $middleware ); // phpcs:ignore WordPress.Security.EscapeOutput.ExceptionNotEscaped, escaped in exception.
+			throw Module_Manager_Exception::failed_to_create_registration_middleware($middleware); // phpcs:ignore WordPress.Security.EscapeOutput.ExceptionNotEscaped, escaped in exception.
 		}
 
 		return $middleware;
@@ -185,10 +185,10 @@ 
 	 * @param Module $module
 	 * @return void
 	 */
-	private function register_hooks( Module $module ): void {
-		add_action( Hooks::APP_INIT_PRE_BOOT, array( $module, 'pre_boot' ), 10, 3 );
-		add_action( Hooks::APP_INIT_PRE_REGISTRATION, array( $module, 'pre_register' ), 10, 3 );
-		add_action( Hooks::APP_INIT_POST_REGISTRATION, array( $module, 'post_register' ), 10, 3 );
+	private function register_hooks(Module $module): void {
+		add_action(Hooks::APP_INIT_PRE_BOOT, array($module, 'pre_boot'), 10, 3);
+		add_action(Hooks::APP_INIT_PRE_REGISTRATION, array($module, 'pre_register'), 10, 3);
+		add_action(Hooks::APP_INIT_POST_REGISTRATION, array($module, 'post_register'), 10, 3);
 	}
 
 	/**

src/Services/Registration/Modules/Hookable_Middleware.php

Spacing +3 added lines, -3 removed lines

@@ -41,10 +41,10 @@
 	 * @param object $class
 	 * @return object
 	 */
-	public function process( object $class ): object {  // phpcs:ignore Universal.NamingConventions.NoReservedKeywordParameterNames.classFound
-		if ( in_array( Hookable::class, class_implements( $class ) ?: array(), true ) ) { // phpcs:ignore Universal.Operators.DisallowShortTernary.Found
+	public function process(object $class): object {  // phpcs:ignore Universal.NamingConventions.NoReservedKeywordParameterNames.classFound
+		if (in_array(Hookable::class, class_implements($class) ?: array(), true)) { // phpcs:ignore Universal.Operators.DisallowShortTernary.Found
 			/** @phpstan-ignore-next-line class must implement register for interface*/
-			$class->register( $this->loader );
+			$class->register($this->loader);
 		}
 		return $class;
 	}

src/Services/Registration/Registration_Service.php

Spacing +14 added lines, -14 removed lines

@@ -53,7 +53,7 @@ 
 	 */
 	protected DI_Container $di_container;
 
-	public function __construct( DI_Container $di_container ) {
+	public function __construct(DI_Container $di_container) {
 		$this->di_container = $di_container;
 	}
 
@@ -63,8 +63,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;
 	}
 
@@ -74,15 +74,15 @@ 
 	 * @template Class_Name of object
 	 * @param class-string<Class_Name> $class_string
 	 */
-	public function push_class( string $class_string ): self {
+	public function push_class(string $class_string): self {
 		// If the class is already in the list, skip.
-		if ( \in_array( $class_string, $this->class_list, true ) ) {
+		if (\in_array($class_string, $this->class_list, true)) {
 			return $this;
 		}
 
 		// If $class_string is not a class, throw exception.
-		if ( ! \class_exists( $class_string ) ) {
-			throw Module_Manager_Exception::none_class_string_passed_to_registration( esc_html( $class_string ) );
+		if ( ! \class_exists($class_string)) {
+			throw Module_Manager_Exception::none_class_string_passed_to_registration(esc_html($class_string));
 		}
 
 		$this->class_list[] = $class_string;
@@ -96,25 +96,25 @@ 
 	 */
 	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);
 
 		// If class list is empty, skip.
-		if ( empty( $class_list ) ) {
+		if (empty($class_list)) {
 			return;
 		}
 
-		foreach ( $this->middleware as $middleware ) {
+		foreach ($this->middleware as $middleware) {
 			// 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);
 				}
 			}
 

src/Exceptions/Module_Manager_Exception.php

Spacing +18 added lines, -18 removed lines

@@ -35,20 +35,20 @@ 
 	 * @param mixed $value
 	 * @return string
 	 */
-	private static function cast_to_string( $value ): string {
-		if ( is_object( $value ) ) {
-			return get_class( $value );
+	private static function cast_to_string($value): string {
+		if (is_object($value)) {
+			return get_class($value);
 		}
 
-		if ( is_array( $value ) ) {
-			return \wp_json_encode( $value ) ?: 'FAILED_TO_CAST ARRAY'; // phpcs:ignore Universal.Operators.DisallowShortTernary.Found
+		if (is_array($value)) {
+			return \wp_json_encode($value) ?: 'FAILED_TO_CAST ARRAY'; // phpcs:ignore Universal.Operators.DisallowShortTernary.Found
 		}
 
-		if ( is_null( $value ) ) {
+		if (is_null($value)) {
 			return 'NULL';
 		}
 
-		if ( is_bool( $value ) ) {
+		if (is_bool($value)) {
 			return $value ? 'BOOL::true' : 'BOOL::false';
 		}
 
@@ -61,9 +61,9 @@ 
 	 * @param string $module
 	 * @return Module_Manager_Exception
 	 */
-	public static function invalid_module_class_name( string $module ): Module_Manager_Exception {
+	public static function invalid_module_class_name(string $module): Module_Manager_Exception {
 		$message = "{$module} must be an instance of the Module interface";
-		return new Module_Manager_Exception( $message, 20 );
+		return new Module_Manager_Exception($message, 20);
 	}
 
 	/**
@@ -73,12 +73,12 @@ 
 	 * @param mixed $created
 	 * @return Module_Manager_Exception
 	 */
-	public static function failed_to_create_registration_middleware( $created ): Module_Manager_Exception {
+	public static function failed_to_create_registration_middleware($created): Module_Manager_Exception {
 
-		$created = self::cast_to_string( $created );
+		$created = self::cast_to_string($created);
 
 		$message = "Failed to create Registration_Middleware, invalid instance created. Created: {$created}";
-		return new Module_Manager_Exception( $message, 21 );
+		return new Module_Manager_Exception($message, 21);
 	}
 
 	/**
@@ -88,11 +88,11 @@ 
 	 * @param mixed $created
 	 * @return Module_Manager_Exception
 	 */
-	public static function invalid_registration_middleware( $created ): Module_Manager_Exception {
-		$created = self::cast_to_string( $created );
+	public static function invalid_registration_middleware($created): Module_Manager_Exception {
+		$created = self::cast_to_string($created);
 
 		$message = "{$created} was returned as the modules Middleware, but this does not implement Registration_Middleware interface";
-		return new Module_Manager_Exception( $message, 22 );
+		return new Module_Manager_Exception($message, 22);
 	}
 
 	/**
@@ -102,10 +102,10 @@ 
 	 * @param mixed $passed
 	 * @return Module_Manager_Exception
 	 */
-	public static function none_class_string_passed_to_registration( $passed ): Module_Manager_Exception {
-		$passed = self::cast_to_string( $passed );
+	public static function none_class_string_passed_to_registration($passed): Module_Manager_Exception {
+		$passed = self::cast_to_string($passed);
 
 		$message = "None class-string \"{$passed}\" passed to the registration class list";
-		return new Module_Manager_Exception( $message, 23 );
+		return new Module_Manager_Exception($message, 23);
 	}
 }