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);
 	}
 
 		/**
@@ -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/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/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/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);
 	}
 
 }

src/Interfaces/Module.php

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,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;
 
 
 	/**
@@ -57,7 +57,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;
 
 	/**
 	 * Callback fired after registration is completed.
@@ -67,5 +67,5 @@ 
 	 * @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/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/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/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 {
 	}
 }