Pink-Crab / WPDB_Migrations

src/Migration_Manager.php

Spacing +28 added lines, -28 removed lines

@@ -59,10 +59,10 @@ 
 	 */
 	protected $migration_log;
 
-	public function __construct( Builder $builder, wpdb $wpdb, ?string $migration_log_key = null ) {
+	public function __construct(Builder $builder, wpdb $wpdb, ?string $migration_log_key = null) {
 		$this->builder       = $builder;
 		$this->wpdb          = $wpdb;
-		$this->migration_log = new Migration_Log_Manager( $migration_log_key );
+		$this->migration_log = new Migration_Log_Manager($migration_log_key);
 
 	}
 
@@ -94,7 +94,7 @@ 
 	 * @param  \PinkCrab\DB_Migration\Database_Migration $migration
 	 * @return self
 	 */
-	public function add_migration( Database_Migration $migration ): self {
+	public function add_migration(Database_Migration $migration): self {
 		$this->migrations[ $migration->get_table_name() ] = $migration;
 		return $this;
 	}
@@ -114,22 +114,22 @@ 
 	 * @param string ...$excluded_table Table names to exclude.
 	 * @return self
 	 */
-	public function create_tables( string ...$excluded_table ): self {
+	public function create_tables(string ...$excluded_table): self {
 
 		// Remove excluded tables.
 		$to_create = array_filter(
 			$this->migrations,
-			function( Database_Migration $migration ) use ( $excluded_table ): bool {
-				return ! in_array( $migration->get_table_name(), $excluded_table, true )
-				&& $this->migration_log->can_migrate( $migration->get_schema() );
+			function(Database_Migration $migration) use ($excluded_table): bool {
+				return ! in_array($migration->get_table_name(), $excluded_table, true)
+				&& $this->migration_log->can_migrate($migration->get_schema());
 			}
 		);
 
 		// Upsert Tables.
-		foreach ( $to_create as $migration ) {
-			$result = $this->builder->create_table( $migration->get_schema() );
-			if ( $result === true ) {
-				$this->migration_log->upsert_migration( $migration->get_schema() );
+		foreach ($to_create as $migration) {
+			$result = $this->builder->create_table($migration->get_schema());
+			if ($result === true) {
+				$this->migration_log->upsert_migration($migration->get_schema());
 			}
 		}
 
@@ -143,23 +143,23 @@ 
 	 * @param string ...$excluded_table Table names to exclude.
 	 * @return self
 	 */
-	public function seed_tables( string ...$excluded_table ): self {
+	public function seed_tables(string ...$excluded_table): self {
 
 		// Remove excluded tables.
 		$to_seed = array_filter(
 			$this->migrations,
-			function( Database_Migration $migration ) use ( $excluded_table ): bool {
-				return ! in_array( $migration->get_table_name(), $excluded_table, true )
-				&& ! $this->migration_log->is_seeded( $migration->get_schema() );
+			function(Database_Migration $migration) use ($excluded_table): bool {
+				return ! in_array($migration->get_table_name(), $excluded_table, true)
+				&& ! $this->migration_log->is_seeded($migration->get_schema());
 			}
 		);
 
-		$seeder = new Migration_Seeder( $this->wpdb );
+		$seeder = new Migration_Seeder($this->wpdb);
 
-		foreach ( $to_seed as $migration ) {
-			$row = $seeder->seed( $migration->get_schema(), $migration->get_seeds() );
-			if ( count( $row ) !== 0 ) {
-				$this->migration_log->mark_table_seeded( $migration->get_schema() );
+		foreach ($to_seed as $migration) {
+			$row = $seeder->seed($migration->get_schema(), $migration->get_seeds());
+			if (count($row) !== 0) {
+				$this->migration_log->mark_table_seeded($migration->get_schema());
 			}
 		}
 
@@ -172,26 +172,26 @@ 
 	 * @param string ...$excluded_table Table names to exclude.
 	 * @return self
 	 */
-	public function drop_tables( string ...$excluded_table ): self {
+	public function drop_tables(string ...$excluded_table): self {
 		// Remove excluded tables.
 		$to_seed = array_filter(
 			$this->migrations,
-			function( Database_Migration $migration ) use ( $excluded_table ): bool {
-				return ! in_array( $migration->get_table_name(), $excluded_table, true );
+			function(Database_Migration $migration) use ($excluded_table): bool {
+				return ! in_array($migration->get_table_name(), $excluded_table, true);
 			}
 		);
 
-		foreach ( $to_seed as $migration ) {
+		foreach ($to_seed as $migration) {
 
-			$result = $this->builder->drop_table( $migration->get_schema() );
+			$result = $this->builder->drop_table($migration->get_schema());
 
 			// Throw exception if fails.
-			if ( $result === false ) {
-				throw Migration_Exception::failed_to_drop_table( $migration->get_table_name() );
+			if ($result === false) {
+				throw Migration_Exception::failed_to_drop_table($migration->get_table_name());
 			}
 
 			// Remove migration from log.
-			$this->migration_log->remove_migration( $migration->get_schema() );
+			$this->migration_log->remove_migration($migration->get_schema());
 		}
 
 		return $this;

src/Log/Migration_Log_Manager.php

Spacing +31 added lines, -31 removed lines

@@ -45,7 +45,7 @@ 
 	 */
 	protected $migration_details = array();
 
-	public function __construct( string $option_key = null ) {
+	public function __construct(string $option_key = null) {
 		$this->option_key = $option_key ?? 'pinkcrab_migration_log';
 		$this->set_migration_details();
 	}
@@ -57,19 +57,19 @@ 
 	 * @throws InvalidArgumentException If can not unserialize
 	 */
 	protected function set_migration_details(): void {
-		$migrations = get_option( $this->option_key, null );
+		$migrations = get_option($this->option_key, null);
 
-		if ( $migrations === null ) {
+		if ($migrations === null) {
 			return;
 		}
 
 		// Handle errors as exceptions.
 		set_error_handler( // phpcs:ignore
-			function ( int $errno, string $errstr ) { // phpcs:ignore
-				throw new InvalidArgumentException( 'Migration details as unserialize from options, failed to be decoded.' );
+			function(int $errno, string $errstr) { // phpcs:ignore
+				throw new InvalidArgumentException('Migration details as unserialize from options, failed to be decoded.');
 			}
 		);
-		$migrations = \unserialize( $migrations ); // phpcs:ignore
+		$migrations = \unserialize($migrations); // phpcs:ignore
 		restore_error_handler();
 
 		$this->migration_details = $migrations;
@@ -81,8 +81,8 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return boolean
 	 */
-	public function has_migration( Schema $schema ): bool {
-		return array_key_exists( $schema->get_table_name(), $this->migration_details );
+	public function has_migration(Schema $schema): bool {
+		return array_key_exists($schema->get_table_name(), $this->migration_details);
 	}
 
 	/**
@@ -90,8 +90,8 @@ 
 	 *
 	 * @return \PinkCrab\DB_Migration\Log\Migration_Log|null
 	 */
-	public function get_migration( Schema $schema ): ?Migration_Log {
-		return $this->has_migration( $schema )
+	public function get_migration(Schema $schema): ?Migration_Log {
+		return $this->has_migration($schema)
 			? $this->migration_details[ $schema->get_table_name() ]
 			: null;
 	}
@@ -102,8 +102,8 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return bool
 	 */
-	public function can_migrate( Schema $schema ): bool {
-		return ! $this->check_hash( $schema );
+	public function can_migrate(Schema $schema): bool {
+		return ! $this->check_hash($schema);
 	}
 
 	/**
@@ -112,8 +112,8 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return bool
 	 */
-	public function is_seeded( Schema $schema ): bool {
-		return $this->has_migration( $schema )
+	public function is_seeded(Schema $schema): bool {
+		return $this->has_migration($schema)
 		&& $this->migration_details[ $schema->get_table_name() ]->is_seeded();
 	}
 
@@ -123,15 +123,15 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return bool
 	 */
-	private function check_hash( Schema $schema ): bool {
+	private function check_hash(Schema $schema): bool {
 		// If table doesnt exist, return false;
-		if ( ! $this->has_migration( $schema ) ) {
+		if ( ! $this->has_migration($schema)) {
 			return false;
 		}
 
-		$schema_hash = Migration_Log::compose_column_hash( $schema );
+		$schema_hash = Migration_Log::compose_column_hash($schema);
 		$migration   = $this->migration_details[ $schema->get_table_name() ];
-		return strcmp( $schema_hash, $migration->schema_hash() ) === 0;
+		return strcmp($schema_hash, $migration->schema_hash()) === 0;
 	}
 
 	/**
@@ -145,20 +145,20 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return self
 	 */
-	public function upsert_migration( Schema $schema ): self {
+	public function upsert_migration(Schema $schema): self {
 		// Update if table exists and we have a new schema defined.
-		if ( $this->has_migration( $schema )
-		&& ! $this->check_hash( $schema ) ) {
+		if ($this->has_migration($schema)
+		&& ! $this->check_hash($schema)) {
 			$this->migration_details[ $schema->get_table_name() ] =
-				$this->migration_details[ $schema->get_table_name() ]->as_updated( $schema );
+				$this->migration_details[ $schema->get_table_name() ]->as_updated($schema);
 
 			$this->save();
 		}
 
 		// If a new hash.
-		if ( $this->has_migration( $schema ) === false ) {
+		if ($this->has_migration($schema) === false) {
 			$this->migration_details[ $schema->get_table_name() ] =
-				Migration_Log::new_from_schema( $schema );
+				Migration_Log::new_from_schema($schema);
 
 			$this->save();
 		}
@@ -172,9 +172,9 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return self
 	 */
-	public function remove_migration( Schema $schema ): self {
-		if ( $this->has_migration( $schema ) ) {
-			unset( $this->migration_details[ $schema->get_table_name() ] );
+	public function remove_migration(Schema $schema): self {
+		if ($this->has_migration($schema)) {
+			unset($this->migration_details[ $schema->get_table_name() ]);
 
 			$this->save();
 		}
@@ -187,8 +187,8 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return self
 	 */
-	public function mark_table_seeded( Schema $schema ): self {
-		if ( ! $this->is_seeded( $schema ) ) {
+	public function mark_table_seeded(Schema $schema): self {
+		if ( ! $this->is_seeded($schema)) {
 			$this->migration_details[ $schema->get_table_name() ] =
 				$this->migration_details[ $schema->get_table_name() ]->as_seeded();
 
@@ -204,7 +204,7 @@ 
 	 * @return void
 	 */
 	protected function save(): void {
-		\update_option( $this->option_key, serialize( $this->migration_details ), false ); // phpcs:ignore
+		\update_option($this->option_key, serialize($this->migration_details), false); // phpcs:ignore
 	}
 
 	/**
@@ -222,6 +222,6 @@ 
 	 * @return void
 	 */
 	public function clear_log(): void {
-		\delete_option( $this->option_key );
+		\delete_option($this->option_key);
 	}
 }