Pink-Crab / Perique_Plugin_Life_Cycle

src/Plugin_State_Controller.php

Spacing +36 added lines, -36 removed lines

@@ -47,7 +47,7 @@ 
 	 */
 	protected $plugin_base_file = null;
 
-	public function __construct( App $app, ?string $plugin_base_file = null ) {
+	public function __construct(App $app, ?string $plugin_base_file = null) {
 		$this->app              = $app;
 		$this->plugin_base_file = $plugin_base_file;
 	}
@@ -58,8 +58,8 @@ 
 	 * @param App $app
 	 * @return self
 	 */
-	public static function init( App $app, ?string $plugin_base_file = null ): self {
-		$instance = new self( $app, $plugin_base_file );
+	public static function init(App $app, ?string $plugin_base_file = null): self {
+		$instance = new self($app, $plugin_base_file);
 		return $instance;
 	}
 
@@ -69,7 +69,7 @@ 
 	 * @param string $plugin_base_file  Holds the location of the plugin base file.
 	 * @return self
 	 */
-	public function set_plugin_base_file( string $plugin_base_file ): self {
+	public function set_plugin_base_file(string $plugin_base_file): self {
 		$this->plugin_base_file = $plugin_base_file;
 		return $this;
 	}
@@ -81,24 +81,24 @@ 
 	 * @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( $state_event ): self {
-		if ( ! is_subclass_of( $state_event, Plugin_State_Change::class ) ) {
-			throw Plugin_State_Exception::invalid_state_change_event_type( $state_event );
+	public function event($state_event): self {
+		if ( ! is_subclass_of($state_event, Plugin_State_Change::class)) {
+			throw Plugin_State_Exception::invalid_state_change_event_type($state_event);
 		}
 		// If its a string, attempt to create via DI container.
-		if ( is_string( $state_event ) ) {
+		if (is_string($state_event)) {
 			$state_event_string = $state_event;
 
 			try {
 				/** @var Plugin_State_Change|null */
-				$state_event = $this->app->get_container()->create( $state_event );
-			} catch ( \Throwable $th ) {
-				throw Plugin_State_Exception::failed_to_create_state_change_event( $state_event_string );
+				$state_event = $this->app->get_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;
@@ -112,28 +112,28 @@ 
 	 * @return self
 	 * @throws Plugin_State_Exception [103] failed_to_locate_calling_file()
 	 */
-	public function finalise( ?string $file = null ): self {
-		if ( null === $file ) {
+	public function finalise(?string $file = null): self {
+		if (null === $file) {
 			$file = $this->plugin_base_file ?? $this->get_called_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_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);
 				}
 			);
 		}
@@ -147,11 +147,11 @@ 
 	 * @param string $state
 	 * @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(
 			$this->state_events,
-			function( $e ) use ( $state ): bool {
-				return is_subclass_of( $e, $state );
+			function($e) use ($state): bool {
+				return is_subclass_of($e, $state);
 			}
 		);
 	}
@@ -162,8 +162,8 @@ 
 	 * @param string $state
 	 * @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;
 	}
 
 	/**
@@ -172,7 +172,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));
 	}
 
 	/**
@@ -181,7 +181,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));
 	}
 
 	/**
@@ -190,7 +190,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));
 	}
 
 	/**
@@ -202,13 +202,13 @@ 
 	protected function get_called_file(): string {
 		$backtrace = debug_backtrace(); // phpcs:ignore WordPress.PHP.DevelopmentFunctions.error_log_debug_backtrace
 
-		$backtrace_count = count( $backtrace );
-		for ( $i = 0; $i < $backtrace_count; $i++ ) {
-			if ( $backtrace[ $i ]['function'] === __FUNCTION__
-			&& $backtrace[ $i ]['class'] === get_class()
-			&& \array_key_exists( ( $i + 1 ), $backtrace )
+		$backtrace_count = count($backtrace);
+		for ($i = 0; $i < $backtrace_count; $i++) {
+			if ($backtrace[$i]['function'] === __FUNCTION__
+			&& $backtrace[$i]['class'] === get_class()
+			&& \array_key_exists(($i + 1), $backtrace)
 			) {
-				return $backtrace[ $i + 1 ]['file'];
+				return $backtrace[$i + 1]['file'];
 			}
 		}
 		// @codeCoverageIgnoreStart

src/State_Event/Activation.php

Spacing +1 added lines, -1 removed lines

@@ -14,4 +14,4 @@
 
 use PinkCrab\Plugin_Lifecycle\Plugin_State_Change;
 
-interface Activation extends Plugin_State_Change{}
+interface Activation extends Plugin_State_Change {}

src/State_Event/Deactivation.php

Spacing +1 added lines, -1 removed lines

@@ -14,4 +14,4 @@
 
 use PinkCrab\Plugin_Lifecycle\Plugin_State_Change;
 
-interface Deactivation extends Plugin_State_Change{}
+interface Deactivation extends Plugin_State_Change {}

src/Plugin_State_Exception.php

Spacing +10 added lines, -10 removed lines

@@ -24,13 +24,13 @@ 
 	 * @param Throwable|null $exception
 	 * @return Plugin_State_Exception
 	 */
-	public static function failed_to_create_state_change_event( $event, ?Throwable $exception = null ): Plugin_State_Exception {
+	public static function failed_to_create_state_change_event($event, ?Throwable $exception = null): Plugin_State_Exception {
 		$message = \sprintf(
 			'Failed to construct %s using the DI Container. %s',
-			is_string( $event ) ? $event : get_class( $event ),
+			is_string($event) ? $event : get_class($event),
 			$exception ? $exception->getMessage() : ''
 		);
-		return new Plugin_State_Exception( $message, 101 );
+		return new Plugin_State_Exception($message, 101);
 	}
 
 	/**
@@ -39,12 +39,12 @@ 
 	 * @param string|object $event
 	 * @return Plugin_State_Exception
 	 */
-	public static function invalid_state_change_event_type( $event ): Plugin_State_Exception {
+	public static function invalid_state_change_event_type($event): Plugin_State_Exception {
 		$message = \sprintf(
 			'%s is not a valid Plugin State Change Event class',
-			is_string( $event ) ? $event : get_class( $event )
+			is_string($event) ? $event : get_class($event)
 		);
-		return new Plugin_State_Exception( $message, 102 );
+		return new Plugin_State_Exception($message, 102);
 	}
 
 	/**
@@ -53,7 +53,7 @@ 
 	 * @return Plugin_State_Exception
 	 */
 	public static function failed_to_locate_calling_file(): Plugin_State_Exception {
-		return new Plugin_State_Exception( 'Could not locate the file which created the Plugin State Controller. Please define this value manually using $controller->finalise( string $file )', 103 );
+		return new Plugin_State_Exception('Could not locate the file which created the Plugin State Controller. Please define this value manually using $controller->finalise( string $file )', 103);
 	}
 
 	/**
@@ -63,12 +63,12 @@ 
 	 * @param Throwable $exception
 	 * @return Plugin_State_Exception
 	 */
-	public static function error_running_state_change_event( Plugin_State_Change $event, ?Throwable $exception = null ): Plugin_State_Exception {
+	public static function error_running_state_change_event(Plugin_State_Change $event, ?Throwable $exception = null): Plugin_State_Exception {
 		$message = \sprintf(
 			'Failed to run %s->run(), error thrown::%s',
-			get_class( $event ),
+			get_class($event),
 			$exception === null ? 'NO ERROR PASSED' : $exception->getMessage()
 		);
-		return new Plugin_State_Exception( $message, 104 );
+		return new Plugin_State_Exception($message, 104);
 	}
 }

src/State_Change_Queue.php

Spacing +5 added lines, -5 removed lines

@@ -25,7 +25,7 @@ 
 	protected $events;
 
 	/** @param Plugin_State_Change ...$event */
-	public function __construct( Plugin_State_Change ...$event ) {
+	public function __construct(Plugin_State_Change ...$event) {
 		$this->events = $event;
 	}
 
@@ -35,13 +35,13 @@ 
 	 * @return void
 	 */
 	public function __invoke() {
-		foreach ( $this->events as $event ) {
+		foreach ($this->events as $event) {
 			try {
 				$event->run();
-			} catch ( \Throwable $th ) {
+			} catch (\Throwable $th) {
 				// If caught on Activation, throw Plugin_State_Exception
-				if ( is_a( $event, Activation::class ) ) {
-					throw Plugin_State_Exception::error_running_state_change_event( $event, $th );
+				if (is_a($event, Activation::class)) {
+					throw Plugin_State_Exception::error_running_state_change_event($event, $th);
 				}
 
 				continue;