Pink-Crab / Perique_Migrations

src/Migrations.php

Spacing +27 added lines, -27 removed lines

@@ -72,13 +72,13 @@ 
 	 *
 	 * @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();
 	}
 
-	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;
 	}
@@ -89,7 +89,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;
 	}
@@ -100,14 +100,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;
@@ -120,17 +120,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;
 	}
@@ -142,13 +142,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.
@@ -166,8 +166,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);
 		}
 	}
 
@@ -186,7 +186,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));
 	}
 
 	/**
@@ -199,12 +199,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));
 		}
 	}
 
@@ -218,14 +218,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

src/Event/Activation.php

Spacing +4 added lines, -4 removed lines

@@ -25,7 +25,7 @@ 
 	 */
 	protected $migration_manager;
 
-	public function __construct( Migration_Manager $migration_manager ) {
+	public function __construct(Migration_Manager $migration_manager) {
 		$this->migration_manager = $migration_manager;
 	}
 
@@ -70,13 +70,13 @@ 
 
 		return array_values(
 			array_map(
-				function( Migration $migration ): string {
+				function(Migration $migration): string {
 					return $migration->get_table_name();
 				},
 				array_filter(
 					$migrations,
-					function( Migration $migration ):bool {
-						return count( $migration->get_seeds() ) === 0
+					function(Migration $migration):bool {
+						return count($migration->get_seeds()) === 0
 						|| false === $migration->seed_on_inital_activation();
 					}
 				)

src/Event/Uninstall.php

Spacing +6 added lines, -6 removed lines

@@ -31,7 +31,7 @@ 
 	protected $migration_log_key;
 
 	/** @param string[] $tables */
-	public function __construct( array $tables, string $migration_log_key ) {
+	public function __construct(array $tables, string $migration_log_key) {
 		$this->tables            = $tables;
 		$this->migration_log_key = $migration_log_key;
 	}
@@ -68,14 +68,14 @@ 
 
 		// Temp disable warnings.
 		$original_state = (bool) $wpdb->suppress_errors;
-		$wpdb->suppress_errors( true );
+		$wpdb->suppress_errors(true);
 
-		foreach ( $this->tables as $table ) {
-			$wpdb->get_results( "DROP TABLE IF EXISTS {$table}" ); // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
+		foreach ($this->tables as $table) {
+			$wpdb->get_results("DROP TABLE IF EXISTS {$table}"); // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
 		}
 
 		// Reset warnings to initial state.
-		$wpdb->suppress_errors( $original_state );
+		$wpdb->suppress_errors($original_state);
 	}
 
 	/**
@@ -84,6 +84,6 @@ 
 	 * @return void
 	 */
 	protected function remove_migration_log(): void {
-		\delete_option( $this->migration_log_key );
+		\delete_option($this->migration_log_key);
 	}
 }

src/Event/Deactivation.php

Spacing +3 added lines, -3 removed lines

@@ -25,7 +25,7 @@ 
 	 */
 	protected $migration_manager;
 
-	public function __construct( Migration_Manager $migration_manager ) {
+	public function __construct(Migration_Manager $migration_manager) {
 		$this->migration_manager = $migration_manager;
 	}
 
@@ -60,12 +60,12 @@ 
 
 		return array_values(
 			array_map(
-				function( Migration $migration ): string {
+				function(Migration $migration): string {
 					return $migration->get_table_name();
 				},
 				array_filter(
 					$migrations,
-					function( Migration $migration ):bool {
+					function(Migration $migration):bool {
 						return false === $migration->drop_on_deactivation();
 					}
 				)

src/Migration_Exception.php

Spacing +6 added lines, -6 removed lines

@@ -22,9 +22,9 @@ 
 	 * @code 101
 	 * @return Migration_Exception
 	 */
-	public static function failed_to_construct_migration( string $migration_class_name ): Migration_Exception {
-		$message = \sprintf( 'Failed to construct %s using the DI Container', $migration_class_name );
-		return new Migration_Exception( $message, 101 );
+	public static function failed_to_construct_migration(string $migration_class_name): Migration_Exception {
+		$message = \sprintf('Failed to construct %s using the DI Container', $migration_class_name);
+		return new Migration_Exception($message, 101);
 	}
 
 	/**
@@ -33,11 +33,11 @@ 
 	 * @param mixed $var
 	 * @return Migration_Exception
 	 */
-	public static function none_migration_type( $var ): Migration_Exception {
+	public static function none_migration_type($var): Migration_Exception {
 		$message = \sprintf(
 			'Migration::class instance or class name expected, got %s',
-			\is_string( $var ) ? $var : \gettype( $var )
+			\is_string($var) ? $var : \gettype($var)
 		);
-		return new Migration_Exception( $message, 102 );
+		return new Migration_Exception($message, 102);
 	}
 }

src/Migration.php

Spacing +2 added lines, -2 removed lines

@@ -20,8 +20,8 @@
 
 	public function __construct() {
 		$this->table_name = $this->table_name();
-		$this->schema     = new Schema( $this->table_name, array( $this, 'schema' ) );
-		$this->seed_data  = $this->seed( array() );
+		$this->schema     = new Schema($this->table_name, array($this, 'schema'));
+		$this->seed_data  = $this->seed(array());
 	}
 
 	abstract protected function table_name(): string;