Pink-Crab / Perique_Migrations

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 string $var
 	 * @return Migration_Exception
 	 */
-	public static function none_migration_type( string $var ): Migration_Exception {
+	public static function none_migration_type(string $var): Migration_Exception {
 		$message = \sprintf(
 			'Only valid Migration Class names can be passed as migrations, "%s" was passed to Perique_Migrations::add_module()',
-			esc_attr( $var )
+			esc_attr($var)
 		);
-		return new Migration_Exception( $message, 102 );
+		return new Migration_Exception($message, 102);
 	}
 }

src/Module/Perique_Migrations.php

Spacing +28 added lines, -28 removed lines

@@ -38,9 +38,9 @@ 
 	 * @param class-string<Migration> $migration
 	 * @return self
 	 */
-	public function add_migration( string $migration ): self {
-		if ( ! is_subclass_of( $migration, Migration::class ) ) {
-			throw Migration_Exception::none_migration_type( $migration );
+	public function add_migration(string $migration): self {
+		if ( ! is_subclass_of($migration, Migration::class)) {
+			throw Migration_Exception::none_migration_type($migration);
 		}
 
 		$this->migrations[] = $migration;
@@ -53,8 +53,8 @@ 
 	 * @param string $migration_log_key
 	 * @return self
 	 */
-	public function set_migration_log_key( string $migration_log_key ): self {
-		$this->migration_log_key = \sanitize_title( $migration_log_key );
+	public function set_migration_log_key(string $migration_log_key): self {
+		$this->migration_log_key = \sanitize_title($migration_log_key);
 		return $this;
 	}
 
@@ -76,18 +76,18 @@ 
 	 * @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
 		// Set the migration manager.
-		$migration_manager = Factory::manager_with_db_delta( $this->migration_log_key );
+		$migration_manager = Factory::manager_with_db_delta($this->migration_log_key);
 
 		// Add the migrations to the manager.
-		foreach ( $this->generate_migrations( $di_container ) as $migration ) {
-			$migration_manager->add_migration( $migration );
+		foreach ($this->generate_migrations($di_container) as $migration) {
+			$migration_manager->add_migration($migration);
 		}
 
 		// Add to the events.
-		if ( \class_exists( Plugin_Life_Cycle::class ) ) {
-			add_filter( Plugin_Life_Cycle::EVENT_LIST, $this->generate_migration_events_callback( $migration_manager ) );
+		if (\class_exists(Plugin_Life_Cycle::class)) {
+			add_filter(Plugin_Life_Cycle::EVENT_LIST, $this->generate_migration_events_callback($migration_manager));
 		}
 
 	}
@@ -98,21 +98,21 @@ 
 	 * @param Migration_Manager $migration_manager
 	 * @return \Closure(array<Plugin_State_Change>):array<Plugin_State_Change>
 	 */
-	protected function generate_migration_events_callback( Migration_Manager $migration_manager ): \Closure {
+	protected function generate_migration_events_callback(Migration_Manager $migration_manager): \Closure {
 		/**
 		 * @param array<Plugin_State_Change> $events
 		 * @return array<Plugin_State_Change>
 		 */
-		return function( array $events ) use ( $migration_manager ): array {
+		return function(array $events) use ($migration_manager): array {
 			static $cached = null;
-			if ( is_null( $cached ) ) {
+			if (is_null($cached)) {
 				$cached = array(
-					new Activation( $migration_manager ),
-					new Deactivation( $migration_manager ),
-					new Uninstall( $migration_manager ),
+					new Activation($migration_manager),
+					new Deactivation($migration_manager),
+					new Uninstall($migration_manager),
 				);
 			}
-			return array_merge( $events, $cached );
+			return array_merge($events, $cached);
 		};
 	}
 
@@ -123,19 +123,19 @@ 
 	 * @return array<Migration>
 	 * @throws Migration_Exception
 	 */
-	protected function generate_migrations( DI_Container $container ): array {
+	protected function generate_migrations(DI_Container $container): array {
 		$migrations = array();
-		foreach ( $this->migrations as $migration ) {
-			if ( is_string( $migration ) ) {
+		foreach ($this->migrations as $migration) {
+			if (is_string($migration)) {
 				try {
-					$instance = $container->create( $migration );
-				} catch ( \Throwable $th ) {
-					throw Migration_Exception::failed_to_construct_migration( $migration );
+					$instance = $container->create($migration);
+				} catch (\Throwable $th) {
+					throw Migration_Exception::failed_to_construct_migration($migration);
 				}
 
 				// Add to the list if valid.
-				if ( ! is_object( $instance ) || ! is_a( $instance, Migration::class ) ) {
-					throw Migration_Exception::failed_to_construct_migration( $migration );
+				if ( ! is_object($instance) || ! is_a($instance, Migration::class)) {
+					throw Migration_Exception::failed_to_construct_migration($migration);
 				}
 
 				$migrations[] = $instance;
@@ -147,10 +147,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/Event/Uninstall.php

Spacing +10 added lines, -10 removed lines

@@ -33,8 +33,8 @@ 
 	 */
 	protected string $migration_log_key;
 
-	public function __construct( Migration_Manager $migration_manager ) {
-		$this->set_tables( $migration_manager->get_migrations() );
+	public function __construct(Migration_Manager $migration_manager) {
+		$this->set_tables($migration_manager->get_migrations());
 		$this->migration_log_key = $migration_manager->migration_log()->get_log_key();
 	}
 
@@ -45,10 +45,10 @@ 
 	 * @param Database_Migration[] $migrations
 	 * @return void
 	 */
-	public function set_tables( array $migrations ): void {
+	public function set_tables(array $migrations): void {
 		$this->tables = \array_filter(
 			$migrations,
-			fn( Database_Migration $migration ) => is_a( $migration, Migration::class ) && $migration->drop_on_uninstall()
+			fn(Database_Migration $migration) => is_a($migration, Migration::class) && $migration->drop_on_uninstall()
 		);
 	}
 
@@ -71,7 +71,7 @@ 
 	 */
 	public function run(): void {
 		// If no tables, return.
-		if ( empty( $this->tables ) ) {
+		if (empty($this->tables)) {
 			return;
 		}
 
@@ -91,15 +91,15 @@ 
 
 		// Temp disable warnings.
 		$original_state = (bool) $wpdb->suppress_errors;
-		$wpdb->suppress_errors( true );
+		$wpdb->suppress_errors(true);
 
-		foreach ( $this->tables as $table ) {
+		foreach ($this->tables as $table) {
 			$table_name = $table->get_table_name();
-			$wpdb->get_results( "DROP TABLE IF EXISTS {$table_name}" ); // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
+			$wpdb->get_results("DROP TABLE IF EXISTS {$table_name}"); // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared
 		}
 
 		// Reset warnings to initial state.
-		$wpdb->suppress_errors( $original_state );
+		$wpdb->suppress_errors($original_state);
 	}
 
 	/**
@@ -108,6 +108,6 @@ 
 	 * @return void
 	 */
 	protected function remove_migration_log(): void {
-		\delete_option( $this->migration_log_key );
+		\delete_option($this->migration_log_key);
 	}
 }

src/Event/Activation.php

Spacing +4 added lines, -4 removed lines

@@ -25,7 +25,7 @@ 
 	 */
 	protected Migration_Manager $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/Deactivation.php

Spacing +3 added lines, -3 removed lines

@@ -25,7 +25,7 @@ 
 	 */
 	protected Migration_Manager $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();
 					}
 				)