Pink-Crab / Perique-Framework

src/Interfaces/DI_Container.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
 	 * @param array<string, mixed> $rule
 	 * @return DI_Container
 	 */
-	public function addRule( string $name, array $rule ): DI_Container;
+	public function addRule(string $name, array $rule): DI_Container;
 
 	/**
 	 * Add multiple rules
@@ -44,7 +44,7 @@ 
 	 * @param array<string, array<mixed>> $rules
 	 * @return DI_Container
 	 */
-	public function addRules( array $rules ): DI_Container;
+	public function addRules(array $rules): DI_Container;
 
 	/**
 	 * Create an instance of a class, with optional parameters.
@@ -53,5 +53,5 @@ 
 	 * @param array<mixed> $args
 	 * @return object|null
 	 */
-	public function create( string $name, array $args = array() ): ?object;
+	public function create(string $name, array $args = array()): ?object;
 }

src/Interfaces/Registration_Middleware.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
 	 * @param T $class
 	 * @return T
 	 */
-	public function process( object $class ): object;
+	public function process(object $class): object;
 
 	/**
 	 * Used to for any middleware setup before process is called

src/Services/View/Component/Component_Compiler.php

Spacing +21 added lines, -21 removed lines

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

src/Services/View/View_Model.php

Spacing +1 added lines, -1 removed lines

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

src/Services/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( "{$id} not defined in container", 1 );
+	public function get(string $id): ?object {
+		if ( ! $this->has($id)) {
+			throw new DI_Container_Exception("{$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 bool
 	 */
-	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/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;
 	}
 
@@ -73,15 +73,15 @@ 
 	 * @template Class_Name of object
 	 * @param class-string<Class_Name> $class
 	 */
-	public function push_class( string $class ): self {
+	public function push_class(string $class): self {
 		// If the class is already in the list, skip.
-		if ( \in_array( $class, $this->class_list, true ) ) {
+		if (\in_array($class, $this->class_list, true)) {
 			return $this;
 		}
 
 		// If $class is not a class, throw exception.
-		if ( ! \class_exists( $class ) ) {
-			throw Module_Manager_Exception::none_class_string_passed_to_registration( $class );
+		if ( ! \class_exists($class)) {
+			throw Module_Manager_Exception::none_class_string_passed_to_registration($class);
 		}
 
 		$this->class_list[] = $class;
@@ -95,25 +95,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/Services/Registration/Modules/Hookable_Module.php

Spacing +3 added lines, -3 removed lines

@@ -51,7 +51,7 @@ 
 	 * @pram DI_Container $di_container
 	 * @return void
 	 */
-	public function pre_boot( App_Config $config, Hook_Loader $loader, DI_Container $di_container ): void {
+	public function pre_boot(App_Config $config, Hook_Loader $loader, DI_Container $di_container): void {
 	}
 
 
@@ -63,7 +63,7 @@ 
 	 * @pram DI_Container $di_container
 	 * @return void
 	 */
-	public function pre_register( App_Config $config, Hook_Loader $loader, DI_Container $di_container ): void {
+	public function pre_register(App_Config $config, Hook_Loader $loader, DI_Container $di_container): void {
 	}
 
 	/**
@@ -74,6 +74,6 @@ 
 	 * @pram DI_Container $di_container
 	 * @return void
 	 */
-	public function post_register( App_Config $config, Hook_Loader $loader, DI_Container $di_container ): void {
+	public function post_register(App_Config $config, Hook_Loader $loader, DI_Container $di_container): void {
 	}
 }

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 {
-		if ( in_array( Hookable::class, class_implements( $class ) ?: array(), true ) ) {
+	public function process(object $class): object {
+		if (in_array(Hookable::class, class_implements($class) ?: array(), true)) {
 			/** @phpstan-ignore-next-line class must implement register for interface*/
-			$class->register( $this->loader );
+			$class->register($this->loader);
 		}
 		return $class;
 	}

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
 	 */
-	public function register_class( string $class ): void {
-		$this->registration_service->push_class( $class );
+	public function register_class(string $class): void {
+		$this->registration_service->push_class($class);
 	}
 
 	/**
@@ -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( $module );
+			throw Module_Manager_Exception::invalid_module_class_name($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 );
+		if ( ! is_a($middleware, Registration_Middleware::class, true)) {
+			throw Module_Manager_Exception::invalid_registration_middleware($middleware);
 		}
 
 		// 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 );
+			throw Module_Manager_Exception::failed_to_create_registration_middleware($middleware);
 		}
 
 		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);
 	}
 
 	/**