Pink-Crab / Perique_Migrations

src/Migrations.php

Spacing +27 added lines, -27 removed lines

@@ -72,7 +72,7 @@ 
 	 *
 	 * @param Plugin_State_Controller $plugin_state_controller
 	 */
-	public function __construct( Plugin_State_Controller $plugin_state_controller, ?string $migration_log_key = null ) {
+	public function __construct(Plugin_State_Controller $plugin_state_controller, ?string $migration_log_key = null) {
 		$this->plugin_state_controller = $plugin_state_controller;
 		$this->migration_log_key       = $migration_log_key;
 		$this->di_container            = $plugin_state_controller->get_app()->get_container();
@@ -84,7 +84,7 @@ 
 	 * @param string $log_key
 	 * @return self
 	 */
-	public function set_migration_log_key( string $log_key ): self {
+	public function set_migration_log_key(string $log_key): self {
 		$this->migration_log_key = $log_key;
 		return $this;
 	}
@@ -95,7 +95,7 @@ 
 	 * @param Migration_Manager $migration_manager  The migration manager instance.
 	 * @return self
 	 */
-	public function set_migration_manager( Migration_Manager $migration_manager ):self {
+	public function set_migration_manager(Migration_Manager $migration_manager):self {
 		$this->migration_manager = $migration_manager;
 		return $this;
 	}
@@ -106,14 +106,14 @@ 
 	 * @param class-string<Migration>|Migration $migration
 	 * @return self
 	 */
-	public function add_migration( $migration ): self {
-		if ( ! is_subclass_of( $migration, Migration::class ) ) {
-			throw Migration_Exception::none_migration_type( \serialize( $migration ) ); //phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.serialize_serialize, used for exception messages
+	public function add_migration($migration): self {
+		if ( ! is_subclass_of($migration, Migration::class)) {
+			throw Migration_Exception::none_migration_type(\serialize($migration)); //phpcs:ignore WordPress.PHP.DiscouragedPHPFunctions.serialize_serialize, used for exception messages
 		}
 
-		$migration = $this->maybe_construct_migration( $migration );
-		if ( null === $migration ) {
-			throw Migration_Exception::failed_to_construct_migration( 'Invalid after construction' );
+		$migration = $this->maybe_construct_migration($migration);
+		if (null === $migration) {
+			throw Migration_Exception::failed_to_construct_migration('Invalid after construction');
 		}
 
 		$this->migrations[] = $migration;
@@ -126,17 +126,17 @@ 
 	 * @param class-string<Migration>|Migration $migration
 	 * @return Migration|null
 	 */
-	protected function maybe_construct_migration( $migration ): ?Migration {
-		if ( is_string( $migration ) ) {
+	protected function maybe_construct_migration($migration): ?Migration {
+		if (is_string($migration)) {
 			$migration_string = $migration;
 			try {
-				$migration = $this->di_container->create( $migration_string );
-			} catch ( \Throwable $th ) {
-				throw Migration_Exception::failed_to_construct_migration( $migration_string );
+				$migration = $this->di_container->create($migration_string);
+			} catch (\Throwable $th) {
+				throw Migration_Exception::failed_to_construct_migration($migration_string);
 			}
 		}
 
-		return is_object( $migration ) && is_a( $migration, Migration::class )
+		return is_object($migration) && is_a($migration, Migration::class)
 		? $migration
 		: null;
 	}
@@ -148,13 +148,13 @@ 
 	 */
 	public function done(): self {
 		// Bail if no migrations.
-		if ( 0 === count( $this->get_migrations() ) ) {
+		if (0 === count($this->get_migrations())) {
 			return $this;
 		}
 
 		// Set with a fallback Migration Manager if not set.
-		if ( null === $this->migration_manager ) {
-			$this->migration_manager = Factory::manager_with_db_delta( $this->migration_log_key );
+		if (null === $this->migration_manager) {
+			$this->migration_manager = Factory::manager_with_db_delta($this->migration_log_key);
 		}
 
 		// Register all migrations and hooks.
@@ -172,8 +172,8 @@ 
 	 * @return void
 	 */
 	private function populate_migration_manager(): void {
-		foreach ( $this->migrations as $migration ) {
-			$this->migration_manager->add_migration( $migration );
+		foreach ($this->migrations as $migration) {
+			$this->migration_manager->add_migration($migration);
 		}
 	}
 
@@ -192,7 +192,7 @@ 
 	 * @return void
 	 */
 	private function set_activation_calls(): void {
-		$this->plugin_state_controller->event( new Activation( $this->migration_manager ) );
+		$this->plugin_state_controller->event(new Activation($this->migration_manager));
 	}
 
 	/**
@@ -205,12 +205,12 @@ 
 		// Check we have valid deactivation calls.
 		$drop_on_deactivation_migrations = array_filter(
 			$this->migrations,
-			function( Migration $migration ): bool {
+			function(Migration $migration): bool {
 				return $migration->drop_on_deactivation() === true;
 			}
 		);
-		if ( count( $drop_on_deactivation_migrations ) >= 1 ) {
-			$this->plugin_state_controller->event( new Deactivation( $this->migration_manager ) );
+		if (count($drop_on_deactivation_migrations) >= 1) {
+			$this->plugin_state_controller->event(new Deactivation($this->migration_manager));
 		}
 	}
 
@@ -224,14 +224,14 @@ 
 		// Check we have valid uninstall calls.
 		$drop_on_uninstall_migrations = array_filter(
 			$this->migrations,
-			function( Migration $migration ): bool {
+			function(Migration $migration): bool {
 				return $migration->drop_on_uninstall() === true;
 			}
 		);
-		if ( count( $drop_on_uninstall_migrations ) >= 1 ) {
+		if (count($drop_on_uninstall_migrations) >= 1) {
 			// Get all table names to be dropped.
 			$table_names = \array_map(
-				function( Migration $migration ): string {
+				function(Migration $migration): string {
 					return $migration->get_table_name();
 				},
 				$drop_on_uninstall_migrations