Pink-Crab / Perique_Plugin_Life_Cycle

src/Plugin_Life_Cycle.php

Spacing +20 added lines, -20 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types=1 );
+declare(strict_types=1);
 
 /**
  * Interface for all classes which act on a plugin state change.
@@ -44,10 +44,10 @@ 
 	 *
 	 * @return self
 	 */
-	public function event( string $event ): self {
+	public function event(string $event): self {
 		// Ensure the event is a valid class.
-		if ( ! class_exists( $event ) ) {
-			throw Plugin_State_Exception::invalid_state_change_event_type( $event );
+		if ( ! class_exists($event)) {
+			throw Plugin_State_Exception::invalid_state_change_event_type($event);
 		}
 		$this->events[] = $event;
 
@@ -61,7 +61,7 @@ 
 	 *
 	 * @return self
 	 */
-	public function plugin_base_file( string $plugin_base_file ): self {
+	public function plugin_base_file(string $plugin_base_file): self {
 		$this->plugin_base_file = $plugin_base_file;
 
 		return $this;
@@ -75,13 +75,13 @@ 
 	 * @pram DI_Container $di_container
 	 * @return void
 	 */
-	public function pre_boot( App_Config $config, Hook_Loader $loader, DI_Container $di_container ): void { // phpcs:ignore Generic.CodeAnalysis.UnusedFunctionParameter.FoundInImplementedInterfaceBeforeLastUsed
+	public function pre_boot(App_Config $config, Hook_Loader $loader, DI_Container $di_container): void { // phpcs:ignore Generic.CodeAnalysis.UnusedFunctionParameter.FoundInImplementedInterfaceBeforeLastUsed
 
 		// Create the instance of the state controller.
-		$this->state_controller = new Plugin_State_Controller( $di_container, $this->plugin_base_file );
+		$this->state_controller = new Plugin_State_Controller($di_container, $this->plugin_base_file);
 
 		// Finalise the state controller.
-		add_action( Hooks::APP_INIT_PRE_BOOT, array( $this, 'finalise' ), 999, 3 );
+		add_action(Hooks::APP_INIT_PRE_BOOT, array($this, 'finalise'), 999, 3);
 	}
 
 	/**
@@ -92,26 +92,26 @@ 
 	 * @pram DI_Container $di_container
 	 * @return void
 	 */
-	public function finalise( App_Config $config, Hook_Loader $loader, DI_Container $di_container ): void { // phpcs:ignore Generic.CodeAnalysis.UnusedFunctionParameter.FoundInImplementedInterfaceAfterLastUsed
+	public function finalise(App_Config $config, Hook_Loader $loader, DI_Container $di_container): void { // phpcs:ignore Generic.CodeAnalysis.UnusedFunctionParameter.FoundInImplementedInterfaceAfterLastUsed
 
 		// Throw exception if controller not set.
-		if ( null === $this->state_controller ) {
+		if (null === $this->state_controller) {
 			throw Plugin_State_Exception::missing_controller();
 		}
 
 		// Trigger the pre action.
-		do_action( self::PRE_FINALISE, $this );
+		do_action(self::PRE_FINALISE, $this);
 
 		// Add events to the controller.
-		foreach ( $this->get_events() as $event ) {
-			$this->state_controller->event( $event );
+		foreach ($this->get_events() as $event) {
+			$this->state_controller->event($event);
 		}
 
 		// Register the state controller.
 		$this->state_controller->finalise();
 
 		// Trigger the post action.
-		do_action( self::POST_FINALISE, $this );
+		do_action(self::POST_FINALISE, $this);
 	}
 
 
@@ -121,10 +121,10 @@ 
 	 * @return class-string<Plugin_State_Change>[]
 	 */
 	public function get_events(): array {
-		$events = apply_filters( self::STATE_EVENTS, $this->events );
-		$events = array_unique( $events );
-		$events = array_filter( $events, 'is_string' );
-		$events = array_filter( $events, fn( $event ) => is_subclass_of( $event, Plugin_State_Change::class ) );
+		$events = apply_filters(self::STATE_EVENTS, $this->events);
+		$events = array_unique($events);
+		$events = array_filter($events, 'is_string');
+		$events = array_filter($events, fn($event) => is_subclass_of($event, Plugin_State_Change::class));
 
 		return $events;
 	}
@@ -132,10 +132,10 @@ 
 	## Unused methods
 
 	/** @inheritDoc */
-	public function pre_register( App_Config $config, Hook_Loader $loader, DI_Container $di_container ): void {} // phpcs:ignore Generic.CodeAnalysis.UnusedFunctionParameter.FoundInImplementedInterfaceBeforeLastUsed
+	public function pre_register(App_Config $config, Hook_Loader $loader, DI_Container $di_container): void {} // phpcs:ignore Generic.CodeAnalysis.UnusedFunctionParameter.FoundInImplementedInterfaceBeforeLastUsed
 
 	/** @inheritDoc */
-	public function post_register( App_Config $config, Hook_Loader $loader, DI_Container $di_container ): void {} // phpcs:ignore Generic.CodeAnalysis.UnusedFunctionParameter.FoundInImplementedInterfaceBeforeLastUsed
+	public function post_register(App_Config $config, Hook_Loader $loader, DI_Container $di_container): void {} // phpcs:ignore Generic.CodeAnalysis.UnusedFunctionParameter.FoundInImplementedInterfaceBeforeLastUsed
 
 	/** @inheritDoc */
 	public function get_middleware(): ?string {

src/Plugin_State_Controller.php

Spacing +31 added lines, -31 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare( strict_types=1 );
+declare(strict_types=1);
 
 /**
  * Interface for all classes which act on a plugin state change.
@@ -48,7 +48,7 @@ 
 	 */
 	protected $plugin_base_file;
 
-	public function __construct( DI_Container $container, ?string $plugin_base_file ) {
+	public function __construct(DI_Container $container, ?string $plugin_base_file) {
 		$this->container        = $container;
 		$this->plugin_base_file = $plugin_base_file ?? $this->get_instantiating_file();
 	}
@@ -61,25 +61,25 @@ 
 	 * @return self
 	 * @throws Plugin_State_Exception If none Plugin_State_Change (string or object) passed or fails to create instance from valid class name.
 	 */
-	public function event( string $state_event ): self {
+	public function event(string $state_event): self {
 		// Cache the event class name.
 		$state_event_string = $state_event;
 
 		/* @phpstan-ignore-next-line, as this cannot be type hinted the check exists. */
-		if ( ! is_subclass_of( $state_event, Plugin_State_Change::class ) ) {
-			throw Plugin_State_Exception::invalid_state_change_event_type( $state_event );
+		if ( ! is_subclass_of($state_event, Plugin_State_Change::class)) {
+			throw Plugin_State_Exception::invalid_state_change_event_type($state_event);
 		}
 
 		try {
 			/** @var Plugin_State_Change|null */
-			$state_event = $this->container->create( $state_event );
-		} catch ( \Throwable $th ) {
-			throw Plugin_State_Exception::failed_to_create_state_change_event( $state_event_string );
+			$state_event = $this->container->create($state_event);
+		} catch (\Throwable $th) {
+			throw Plugin_State_Exception::failed_to_create_state_change_event($state_event_string);
 		}
 
 		// Throw exception if failed to create
-		if ( null === $state_event || ! is_a( $state_event, Plugin_State_Change::class ) ) {
-			throw Plugin_State_Exception::failed_to_create_state_change_event( $state_event_string );
+		if (null === $state_event || ! is_a($state_event, Plugin_State_Change::class)) {
+			throw Plugin_State_Exception::failed_to_create_state_change_event($state_event_string);
 		}
 		$this->state_events[] = $state_event;
 
@@ -97,29 +97,29 @@ 
 		$file = $this->plugin_base_file;
 
 		// Activation hooks if need adding.
-		if ( $this->has_events_for_state( Activation::class ) ) {
-			register_activation_hook( $file, $this->activation() );
+		if ($this->has_events_for_state(Activation::class)) {
+			register_activation_hook($file, $this->activation());
 		}
 
 		// Deactivation hooks.
-		if ( $this->has_events_for_state( Deactivation::class ) ) {
-			register_deactivation_hook( $file, $this->deactivation() );
+		if ($this->has_events_for_state(Deactivation::class)) {
+			register_deactivation_hook($file, $this->deactivation());
 		}
 
 		// If we have an uninstall events, add then during activation.
-		if ( $this->has_events_for_state( Uninstall::class ) ) {
+		if ($this->has_events_for_state(Uninstall::class)) {
 			$callback = $this->uninstall();
 
 			// Register the callback so itsits included (but wont run due to serialization issues).
 			register_activation_hook(
 				$file,
-				static function () use ( $file, $callback ): void {
-					register_uninstall_hook( $file, $callback );
+				static function() use ($file, $callback): void {
+					register_uninstall_hook($file, $callback);
 				}
 			);
 
 			// Manually re-add the uninstall hook.
-			add_action( 'uninstall_' . plugin_basename( $file ), $callback );
+			add_action('uninstall_'.plugin_basename($file), $callback);
 		}
 
 		return $this;
@@ -132,12 +132,12 @@ 
 	 *
 	 * @return Plugin_State_Change[]
 	 */
-	private function get_events_for_state( string $state ): array {
+	private function get_events_for_state(string $state): array {
 		return array_filter(
-			apply_filters( Plugin_Life_Cycle::EVENT_LIST, $this->state_events ),
-			function ( $e ) use ( $state ): bool {
+			apply_filters(Plugin_Life_Cycle::EVENT_LIST, $this->state_events),
+			function($e) use ($state): bool {
 				/* @phpstan-ignore-next-line */
-				return is_subclass_of( $e, $state );
+				return is_subclass_of($e, $state);
 			}
 		);
 	}
@@ -149,8 +149,8 @@ 
 	 *
 	 * @return bool
 	 */
-	private function has_events_for_state( string $state ): bool {
-		return count( $this->get_events_for_state( $state ) ) !== 0;
+	private function has_events_for_state(string $state): bool {
+		return count($this->get_events_for_state($state)) !== 0;
 	}
 
 	/**
@@ -159,7 +159,7 @@ 
 	 * @return State_Change_Queue
 	 */
 	public function activation(): State_Change_Queue {
-		return new State_Change_Queue( ...$this->get_events_for_state( Activation::class ) );
+		return new State_Change_Queue(...$this->get_events_for_state(Activation::class));
 	}
 
 	/**
@@ -168,7 +168,7 @@ 
 	 * @return State_Change_Queue
 	 */
 	public function deactivation(): State_Change_Queue {
-		return new State_Change_Queue( ...$this->get_events_for_state( Deactivation::class ) );
+		return new State_Change_Queue(...$this->get_events_for_state(Deactivation::class));
 	}
 
 	/**
@@ -177,7 +177,7 @@ 
 	 * @return State_Change_Queue
 	 */
 	public function uninstall(): State_Change_Queue {
-		return new State_Change_Queue( ...$this->get_events_for_state( Uninstall::class ) );
+		return new State_Change_Queue(...$this->get_events_for_state(Uninstall::class));
 	}
 
 	/**
@@ -193,17 +193,17 @@ 
 		$source_trace = array_values(
 			array_filter(
 				$backtrace,
-				function ( $bt ) {
-					return array_key_exists( 'class', $bt ) && $bt['class'] === App_Factory::class;
+				function($bt) {
+					return array_key_exists('class', $bt) && $bt['class'] === App_Factory::class;
 				}
 			)
 		);
 
-		if ( 1 !== count( $source_trace ) ) {
+		if (1 !== count($source_trace)) {
 			throw Plugin_State_Exception::failed_to_locate_calling_file();
 		}
 
-		if ( ! \array_key_exists( 'file', $source_trace[0] ) ) {
+		if ( ! \array_key_exists('file', $source_trace[0])) {
 			throw Plugin_State_Exception::failed_to_locate_calling_file();
 		}