Pink-Crab / WPDB-Table-Builder

src/Column_Types.php

Spacing +36 added lines, -36 removed lines

@@ -36,10 +36,10 @@ 
 	 * @param int|null $length
 	 * @return Column
 	 */
-	public function varchar( ?int $length = null ): Column {
-		$this->type( 'varchar' );
-		if ( $length ) {
-			$this->length( $length );
+	public function varchar(?int $length = null): Column {
+		$this->type('varchar');
+		if ($length) {
+			$this->length($length);
 		}
 		return $this;
 	}
@@ -51,10 +51,10 @@ 
 	 * @param int|null $length
 	 * @return Column
 	 */
-	public function text( ?int $length = null ): Column {
-		$this->type( 'text' );
-		if ( $length ) {
-			$this->length( $length );
+	public function text(?int $length = null): Column {
+		$this->type('text');
+		if ($length) {
+			$this->length($length);
 		}
 		return $this;
 	}
@@ -66,10 +66,10 @@ 
 	 * @param int|null $length
 	 * @return Column
 	 */
-	public function int( ?int $length = null ): Column {
-		$this->type( 'int' );
-		if ( $length ) {
-			$this->length( $length );
+	public function int(?int $length = null): Column {
+		$this->type('int');
+		if ($length) {
+			$this->length($length);
 		}
 		return $this;
 	}
@@ -81,10 +81,10 @@ 
 	 * @param int|null $length
 	 * @return Column
 	 */
-	public function float( ?int $length = null ): Column {
-		$this->type( 'float' );
-		if ( $length ) {
-			$this->length( $length );
+	public function float(?int $length = null): Column {
+		$this->type('float');
+		if ($length) {
+			$this->length($length);
 		}
 		return $this;
 	}
@@ -96,10 +96,10 @@ 
 	 * @param int|null $length
 	 * @return Column
 	 */
-	public function double( ?int $length = null ): Column {
-		$this->type( 'double' );
-		if ( $length ) {
-			$this->length( $length );
+	public function double(?int $length = null): Column {
+		$this->type('double');
+		if ($length) {
+			$this->length($length);
 		}
 		return $this;
 	}
@@ -111,10 +111,10 @@ 
 	 * @param string|null $default
 	 * @return Column
 	 */
-	public function datetime( ?string $default = null ): Column {
-		$this->type( 'datetime' );
-		if ( $default ) {
-			$this->default( $default );
+	public function datetime(?string $default = null): Column {
+		$this->type('datetime');
+		if ($default) {
+			$this->default($default);
 		}
 		return $this;
 	}
@@ -126,10 +126,10 @@ 
 	 * @param string|null $default
 	 * @return Column
 	 */
-	public function timestamp( ?string $default = null ): Column {
-		$this->type( 'timestamp' );
-		if ( $default ) {
-			$this->default( $default );
+	public function timestamp(?string $default = null): Column {
+		$this->type('timestamp');
+		if ($default) {
+			$this->default($default);
 		}
 		return $this;
 	}
@@ -141,11 +141,11 @@ 
 	 * @param int|null $length
 	 * @return Column
 	 */
-	public function unsigned_int( ?int $length = null ): Column {
-		$this->type( 'int' );
+	public function unsigned_int(?int $length = null): Column {
+		$this->type('int');
 		$this->unsigned();
-		if ( $length ) {
-			$this->length( $length );
+		if ($length) {
+			$this->length($length);
 		}
 		return $this;
 	}
@@ -157,11 +157,11 @@ 
 	 * @param int|null $length
 	 * @return Column
 	 */
-	public function unsigned_medium( ?int $length = null ): Column {
-		$this->type( 'mediumint' );
+	public function unsigned_medium(?int $length = null): Column {
+		$this->type('mediumint');
 		$this->unsigned();
-		if ( $length ) {
-			$this->length( $length );
+		if ($length) {
+			$this->length($length);
 		}
 		return $this;
 	}

src/Schema.php

Spacing +18 added lines, -18 removed lines

@@ -79,10 +79,10 @@ 
 	 * @param string $table_name
 	 * @param callable(Schema):void|null $configure
 	 */
-	public function __construct( string $table_name, ?callable $configure = null ) {
+	public function __construct(string $table_name, ?callable $configure = null) {
 		$this->table_name = $table_name;
-		if ( is_callable( $configure ) ) {
-			$configure( $this );
+		if (is_callable($configure)) {
+			$configure($this);
 		}
 	}
 
@@ -109,7 +109,7 @@ 
 	 * @param string|null $prefix
 	 * @return self
 	 */
-	public function prefix( ?string $prefix = null ): self {
+	public function prefix(?string $prefix = null): self {
 		$this->prefix = $prefix;
 		return $this;
 	}
@@ -142,10 +142,10 @@ 
 	 * @param string $name
 	 * @return Column
 	 */
-	public function column( string $name ): Column {
-		$column = new Column( $name );
+	public function column(string $name): Column {
+		$column = new Column($name);
 
-		$this->columns[ $name ] = $column;
+		$this->columns[$name] = $column;
 		return $column;
 	}
 
@@ -167,11 +167,11 @@ 
 	 * @param string $name
 	 * @return bool
 	 */
-	public function has_column( string $name ): bool {
+	public function has_column(string $name): bool {
 		return count(
 			array_filter(
 				$this->get_columns(),
-				function( Column $column ) use ( $name ): bool {
+				function(Column $column) use ($name): bool {
 					return $column->get_name() === $name;
 				}
 			)
@@ -186,8 +186,8 @@ 
 	 * @return self
 	 * @throws Exception If columnn doesnt exist.
 	 */
-	public function remove_column( string $name ): self {
-		if ( ! $this->has_column( $name ) ) {
+	public function remove_column(string $name): self {
+		if ( ! $this->has_column($name)) {
 			throw new Exception(
 				sprintf(
 					'%s doest exist in table %s',
@@ -198,7 +198,7 @@ 
 			);
 		}
 
-		unset( $this->columns[ $name ] );
+		unset($this->columns[$name]);
 
 		return $this;
 	}
@@ -211,8 +211,8 @@ 
 	 * @param string|null $keyname if not set, will use the column name a tempalte.
 	 * @return \PinkCrab\Table_Builder\Foreign_Key
 	 */
-	public function foreign_key( string $column, ?string $keyname = null ): Foreign_Key {
-		$foreign_key = new Foreign_Key( $column, $keyname );
+	public function foreign_key(string $column, ?string $keyname = null): Foreign_Key {
+		$foreign_key = new Foreign_Key($column, $keyname);
 
 		$this->foreign_keys[] = $foreign_key;
 		return $foreign_key;
@@ -224,7 +224,7 @@ 
 	 * @return bool
 	 */
 	public function has_foreign_keys(): bool {
-		return count( $this->foreign_keys ) !== 0;
+		return count($this->foreign_keys) !== 0;
 	}
 
 	/**
@@ -244,7 +244,7 @@ 
 	 * @return bool
 	 */
 	public function has_indexes(): bool {
-		return count( $this->indexes ) !== 0;
+		return count($this->indexes) !== 0;
 	}
 
 	/**
@@ -254,8 +254,8 @@ 
 	 * @param string $keyname
 	 * @return \PinkCrab\Table_Builder\Index
 	 */
-	public function index( string $column, ?string $keyname = null ): Index {
-		$index = new Index( $column, $keyname );
+	public function index(string $column, ?string $keyname = null): Index {
+		$index = new Index($column, $keyname);
 
 		$this->indexes[] = $index;
 		return $index;

src/Column.php

Spacing +7 added lines, -7 removed lines

@@ -92,7 +92,7 @@ 
 	 */
 	protected $unsigned = null;
 
-	public function __construct( string $name ) {
+	public function __construct(string $name) {
 		$this->name = $name;
 	}
 
@@ -102,7 +102,7 @@ 
 	 * @param string $type
 	 * @return self
 	 */
-	public function type( string $type ): self {
+	public function type(string $type): self {
 		$this->type = $type;
 		return $this;
 	}
@@ -113,7 +113,7 @@ 
 	 * @param integer $length
 	 * @return self
 	 */
-	public function length( int $length ): self {
+	public function length(int $length): self {
 		$this->length = $length;
 		return $this;
 	}
@@ -124,7 +124,7 @@ 
 	 * @param boolean $nullable
 	 * @return self
 	 */
-	public function nullable( bool $nullable = true ): self {
+	public function nullable(bool $nullable = true): self {
 		$this->nullable = $nullable;
 		return $this;
 	}
@@ -135,7 +135,7 @@ 
 	 * @param string $default
 	 * @return self
 	 */
-	public function default( string $default ): self {
+	public function default(string $default): self {
 		$this->default = $default;
 		return $this;
 	}
@@ -146,7 +146,7 @@ 
 	 * @param boolean $auto_increment
 	 * @return self
 	 */
-	public function auto_increment( bool $auto_increment = true ): self {
+	public function auto_increment(bool $auto_increment = true): self {
 		$this->auto_increment = $auto_increment;
 		return $this;
 	}
@@ -157,7 +157,7 @@ 
 	 * @param boolean $unsigned
 	 * @return self
 	 */
-	public function unsigned( bool $unsigned = true ): self {
+	public function unsigned(bool $unsigned = true): self {
 		$this->unsigned = $unsigned;
 		return $this;
 	}

src/Builder.php

Spacing +6 added lines, -6 removed lines

@@ -38,9 +38,9 @@ 
 	 */
 	protected $engine;
 
-	public function __construct( Engine $engine, ?callable $engine_config = null ) {
+	public function __construct(Engine $engine, ?callable $engine_config = null) {
 		$this->engine = $engine_config
-			? $engine_config( $engine )
+			? $engine_config($engine)
 			: $engine;
 	}
 
@@ -50,8 +50,8 @@ 
 	 * @param Schema $schema
 	 * @return bool
 	 */
-	public function create_table( Schema $schema ): bool {
-		return $this->engine->create_table( $schema );
+	public function create_table(Schema $schema): bool {
+		return $this->engine->create_table($schema);
 	}
 
 	/**
@@ -60,7 +60,7 @@ 
 	 * @param Schema $schema
 	 * @return bool
 	 */
-	public function drop_table( Schema $schema ): bool {
-		return $this->engine->drop_table( $schema );
+	public function drop_table(Schema $schema): bool {
+		return $this->engine->drop_table($schema);
 	}
 }

src/Foreign_Key.php

Spacing +7 added lines, -7 removed lines

@@ -79,8 +79,8 @@ 
 	 */
 	protected $on_delete = '';
 
-	public function __construct( string $column, ?string $keyname = null ) {
-		$this->keyname = $keyname ?? 'fk_' . $column;
+	public function __construct(string $column, ?string $keyname = null) {
+		$this->keyname = $keyname ?? 'fk_'.$column;
 		$this->column  = $column;
 	}
 
@@ -91,7 +91,7 @@ 
 	 * @param string $reference_column
 	 * @return self
 	 */
-	public function reference( string $reference_table, string $reference_column ): self {
+	public function reference(string $reference_table, string $reference_column): self {
 		$this->reference_column = $reference_column;
 		$this->reference_table  = $reference_table;
 		return $this;
@@ -104,7 +104,7 @@ 
 	 * @param string $reference_table
 	 * @return self
 	 */
-	public function reference_table( string $reference_table ): self {
+	public function reference_table(string $reference_table): self {
 		$this->reference_table = $reference_table;
 		return $this;
 	}
@@ -116,7 +116,7 @@ 
 	 * @param string $reference_column
 	 * @return self
 	 */
-	public function reference_column( string $reference_column ): self {
+	public function reference_column(string $reference_column): self {
 		$this->reference_column = $reference_column;
 		return $this;
 	}
@@ -128,7 +128,7 @@ 
 	 * @param string $action
 	 * @return self
 	 */
-	public function on_update( string $action ): self {
+	public function on_update(string $action): self {
 		$this->on_update = $action;
 		return $this;
 	}
@@ -141,7 +141,7 @@ 
 	 * @param string $action
 	 * @return self
 	 */
-	public function on_delete( string $action ): self {
+	public function on_delete(string $action): self {
 		$this->on_delete = $action;
 		return $this;
 	}

src/Engines/Engine.php

Spacing +4 added lines, -4 removed lines

@@ -35,7 +35,7 @@ 
 	 * @param Schema $schema
 	 * @return bool
 	 */
-	public function create_table( Schema $schema ): bool;
+	public function create_table(Schema $schema): bool;
 
 	/**
 	 * Returns the query generated to create a table.
@@ -43,7 +43,7 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return string
 	 */
-	public function create_table_query( Schema $schema ): string;
+	public function create_table_query(Schema $schema): string;
 
 	/**
 	 * Drops a table based on the schema passed.
@@ -51,7 +51,7 @@ 
 	 * @param Schema $schema
 	 * @return bool
 	 */
-	public function drop_table( Schema $schema): bool;
+	public function drop_table(Schema $schema): bool;
 
 	/**
 	 * Returns the query generated to drop a table.
@@ -59,5 +59,5 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return string
 	 */
-	public function drop_table_query( Schema $schema ): string;
+	public function drop_table_query(Schema $schema): string;
 }

src/Engines/WPDB_DB_Delta/DB_Delta_Engine.php

Spacing +21 added lines, -21 removed lines

@@ -61,7 +61,7 @@ 
 	 */
 	protected $schema;
 
-	public function __construct( \wpdb $wpdb ) {
+	public function __construct(\wpdb $wpdb) {
 		$this->wpdb = $wpdb;
 
 		// Set the translator and valiator
@@ -76,15 +76,15 @@ 
 	 * @return void
 	 * @throws \Exception If fails validation. (code 1)
 	 */
-	private function set_query_for_create( Schema $schema ): void {
+	private function set_query_for_create(Schema $schema): void {
 		$this->schema = $schema;
 
-		if ( ! $this->validator->validate( $schema ) ) {
+		if ( ! $this->validator->validate($schema)) {
 			throw new \Exception(
 				sprintf(
 					'Failed to create table %s as failed validation: %s',
 					$schema->get_table_name(),
-					join( ', ' . PHP_EOL, $this->validator->get_errors() )
+					join(', '.PHP_EOL, $this->validator->get_errors())
 				),
 				1
 			);
@@ -99,14 +99,14 @@ 
 	 * @return bool
 	 * @throws \Exception If fails validation. (code 1)
 	 */
-	public function create_table( Schema $schema ): bool {
+	public function create_table(Schema $schema): bool {
 
 		// Generate the query from the passed schema.
-		$query = $this->create_table_query( $schema );
+		$query = $this->create_table_query($schema);
 
 		// Include WP dbDelta.
-		require_once ABSPATH . 'wp-admin/includes/upgrade.php';
-		dbDelta( $query );
+		require_once ABSPATH.'wp-admin/includes/upgrade.php';
+		dbDelta($query);
 
 		return $this->wpdb->last_error === '';
 	}
@@ -118,8 +118,8 @@ 
 	 * @return string
 	 * @throws \Exception If fails validation. (code 1)
 	 */
-	public function create_table_query( Schema $schema ): string {
-		$this->set_query_for_create( $schema );
+	public function create_table_query(Schema $schema): string {
+		$this->set_query_for_create($schema);
 		return $this->compile_create_sql_query();
 	}
 
@@ -130,15 +130,15 @@ 
 	 * @return void
 	 * @throws \Exception If fails validation. (code 2)
 	 */
-	private function set_query_for_drop( Schema $schema ): void {
+	private function set_query_for_drop(Schema $schema): void {
 		$this->schema = $schema;
 
-		if ( ! $this->validator->validate( $schema ) ) {
+		if ( ! $this->validator->validate($schema)) {
 			throw new \Exception(
 				sprintf(
 					'Failed to drop table %s as failed validation: %s',
 					$schema->get_table_name(),
-					join( ', ' . PHP_EOL, $this->validator->get_errors() )
+					join(', '.PHP_EOL, $this->validator->get_errors())
 				),
 				2
 			);
@@ -152,11 +152,11 @@ 
 	 * @return bool
 	 * @throws \Exception If fails validation. (code 2)
 	 */
-	public function drop_table( Schema $schema ): bool {
+	public function drop_table(Schema $schema): bool {
 
-		$query = $this->drop_table_query( $schema );
+		$query = $this->drop_table_query($schema);
 
-		$this->wpdb->get_results( $query ); // phpcs:disable WordPress.DB.PreparedSQL.NotPrepared
+		$this->wpdb->get_results($query); // phpcs:disable WordPress.DB.PreparedSQL.NotPrepared
 		return $this->wpdb->last_error === '';
 	}
 
@@ -167,8 +167,8 @@ 
 	 * @return string
 	 * @throws \Exception If fails validation. (code 2)
 	 */
-	public function drop_table_query( Schema $schema ): string {
-		$this->set_query_for_drop( $schema );
+	public function drop_table_query(Schema $schema): string {
+		$this->set_query_for_drop($schema);
 		return "DROP TABLE IF EXISTS {$this->schema->get_table_name()};";
 	}
 
@@ -182,10 +182,10 @@ 
 		// Compile query partials.
 		$table   = $this->schema->get_table_name();
 		$body    = join(
-			',' . PHP_EOL,
+			','.PHP_EOL,
 			array_merge(
-				$this->translator->translate_columns( $this->schema ),
-				$this->translator->translate_indexes( $this->schema )
+				$this->translator->translate_columns($this->schema),
+				$this->translator->translate_indexes($this->schema)
 			)
 		);
 		$collate = $this->wpdb->collate;

src/Engines/WPDB_DB_Delta/DB_Delta_Translator.php

Indentation +13 added lines, -13 removed lines

@@ -72,12 +72,12 @@ 
 	}
 
 		/**
-	 * Maps types with length if set.
-	 *
-	 * @param string $type
-	 * @param int|null $length
-	 * @return string
-	 */
+		 * Maps types with length if set.
+		 *
+		 * @param string $type
+		 * @param int|null $length
+		 * @return string
+		 */
 	protected function type_mapper( string $type, ?int $length ): string {
 		$type = strtoupper( $type );
 		switch ( $type ) {
@@ -136,13 +136,13 @@ 
 	}
 
 		/**
-	 * Parses the primary key from the defined schema.
-	 *
-	 * This should only return an array with a single value if any.
-	 *
-	 * @param \PinkCrab\Table_Builder\Schema $schema
-	 * @return array<string>
-	 */
+		 * Parses the primary key from the defined schema.
+		 *
+		 * This should only return an array with a single value if any.
+		 *
+		 * @param \PinkCrab\Table_Builder\Schema $schema
+		 * @return array<string>
+		 */
 	protected function transform_primary( Schema $schema ): array {
 		return array_map(
 			function( $index ) {

Spacing +33 added lines, -33 removed lines

@@ -38,18 +38,18 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return array<string>
 	 */
-	public function translate_columns( Schema $schema ): array {
+	public function translate_columns(Schema $schema): array {
 		return array_map(
-			function( Column $column ): string {
+			function(Column $column): string {
 
 				return sprintf(
 					'%s %s%s%s%s%s',
 					$column->get_name(),
-					$this->type_mapper( $column->get_type() ?? '', $column->get_length() ),
+					$this->type_mapper($column->get_type() ?? '', $column->get_length()),
 					$column->is_unsigned() ? ' UNSIGNED' : '',
 					$column->is_nullable() ? ' NULL' : ' NOT NULL',
 					$column->is_auto_increment() ? ' AUTO_INCREMENT' : '',
-					$this->parse_default( $column->get_type() ?? '', $column->get_default() )
+					$this->parse_default($column->get_type() ?? '', $column->get_default())
 				);
 			},
 			$schema->get_columns()
@@ -63,11 +63,11 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return array<string>
 	 */
-	public function translate_indexes( Schema $schema ): array {
+	public function translate_indexes(Schema $schema): array {
 		return array_merge(
-			$this->transform_primary( $schema ),
-			$this->transform_indexes( $schema ),
-			$this->transform_foreign_keys( $schema )
+			$this->transform_primary($schema),
+			$this->transform_indexes($schema),
+			$this->transform_foreign_keys($schema)
 		);
 	}
 
@@ -78,9 +78,9 @@ 
 	 * @param int|null $length
 	 * @return string
 	 */
-	protected function type_mapper( string $type, ?int $length ): string {
-		$type = strtoupper( $type );
-		switch ( $type ) {
+	protected function type_mapper(string $type, ?int $length): string {
+		$type = strtoupper($type);
+		switch ($type) {
 			// With length
 			case 'CHAR':
 			case 'VARCHAR':
@@ -106,7 +106,7 @@ 
 			case 'DATETIME':
 			case 'TIMESTAMP':
 			case 'TIME':
-				return is_null( $length ) ? $type : "{$type}({$length})";
+				return is_null($length) ? $type : "{$type}({$length})";
 
 			default:
 				return $type;
@@ -120,15 +120,15 @@ 
 	 * @param string|null $defualt
 	 * @return string
 	 */
-	protected function parse_default( string $type, ?string $defualt ): string {
-		if ( is_null( $defualt ) ) {
+	protected function parse_default(string $type, ?string $defualt): string {
+		if (is_null($defualt)) {
 			return '';
 		}
 
-		$type = strtoupper( $type );
+		$type = strtoupper($type);
 
 		// String values.
-		if ( in_array( $type, array( 'CHAR', 'VARCHAR', 'BINARY', 'VARBINARY', 'TEXT', 'BLOB' ), true ) ) {
+		if (in_array($type, array('CHAR', 'VARCHAR', 'BINARY', 'VARBINARY', 'TEXT', 'BLOB'), true)) {
 			return " DEFAULT '{$defualt}'";
 		}
 
@@ -143,14 +143,14 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return array<string>
 	 */
-	protected function transform_primary( Schema $schema ): array {
+	protected function transform_primary(Schema $schema): array {
 		return array_map(
-			function( $index ) {
+			function($index) {
 				return "PRIMARY KEY  ({$index->get_column()})";
 			},
 			array_filter(
 				$schema->get_indexes(),
-				function( $index ): bool {
+				function($index): bool {
 					return $index->is_primary();
 				}
 			)
@@ -163,16 +163,16 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return array<string>
 	 */
-	protected function transform_indexes( Schema $schema ): array {
+	protected function transform_indexes(Schema $schema): array {
 		return array_map(
 			/** @param Index[] $index_group */
-			function( array $index_group ): string {
+			function(array $index_group): string {
 
 				// Extract all parts from group.
 				$key_name   = $index_group[0]->get_keyname();
 				$index_type = $index_group[0]->get_type();
 				$columns    = array_map(
-					function( $e ) {
+					function($e) {
 						return $e->get_column();
 					},
 					$index_group
@@ -180,13 +180,13 @@ 
 
 				return sprintf(
 					'%sINDEX %s (%s)',
-					\strlen( $index_type ) !== 0 ? \strtoupper( $index_type ) . ' ' : '',
+					\strlen($index_type) !== 0 ? \strtoupper($index_type).' ' : '',
 					$key_name,
-					join( ', ', $columns )
+					join(', ', $columns)
 				);
 
 			},
-			$this->group_indexes( $schema )
+			$this->group_indexes($schema)
 		);
 	}
 
@@ -196,17 +196,17 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return array<string>
 	 */
-	protected function transform_foreign_keys( Schema $schema ): array {
+	protected function transform_foreign_keys(Schema $schema): array {
 		return array_map(
-			function( Foreign_Key $foreign_key ): string {
+			function(Foreign_Key $foreign_key): string {
 				return \sprintf(
 					'FOREIGN KEY %s(%s) REFERENCES %s(%s)%s%s',
 					$foreign_key->get_keyname(),
 					$foreign_key->get_column(),
 					$foreign_key->get_reference_table(),
 					$foreign_key->get_reference_column(),
-					\strlen( $foreign_key->get_on_update() ) ? " ON UPDATE {$foreign_key->get_on_update()}" : '',
-					\strlen( $foreign_key->get_on_delete() ) ? " ON DELETE {$foreign_key->get_on_delete()}" : ''
+					\strlen($foreign_key->get_on_update()) ? " ON UPDATE {$foreign_key->get_on_update()}" : '',
+					\strlen($foreign_key->get_on_delete()) ? " ON DELETE {$foreign_key->get_on_delete()}" : ''
 				);
 			},
 			$schema->get_foreign_keys()
@@ -219,16 +219,16 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return array<string, Index[]>
 	 */
-	protected function group_indexes( Schema $schema ): array {
+	protected function group_indexes(Schema $schema): array {
 		return array_reduce(
 			$schema->get_indexes(),
-			function( array $carry, Index $index ): array {
+			function(array $carry, Index $index): array {
 				// Remove all primiary keys.
-				if ( $index->is_primary() ) {
+				if ($index->is_primary()) {
 					return $carry;
 				}
 
-				$carry[ $index->get_keyname() . '_' . $index->get_type() ][] = $index;
+				$carry[$index->get_keyname().'_'.$index->get_type()][] = $index;
 
 				return $carry;
 			},

src/Engines/WPDB_DB_Delta/DB_Delta_Validator.php

Spacing +34 added lines, -34 removed lines

@@ -45,14 +45,14 @@ 
 	 * @param Schema $schema
 	 * @return bool
 	 */
-	public function validate( Schema $schema ): bool {
+	public function validate(Schema $schema): bool {
 		// Reset errors.
 		$this->errors = array();
 
-		$this->validate_columns( $schema );
-		$this->validate_primary_key( $schema );
-		$this->validate_index_columns( $schema );
-		$this->validate_foreign_keys( $schema );
+		$this->validate_columns($schema);
+		$this->validate_primary_key($schema);
+		$this->validate_index_columns($schema);
+		$this->validate_foreign_keys($schema);
 
 		return ! $this->has_errors();
 	}
@@ -72,7 +72,7 @@ 
 	 * @return bool
 	 */
 	public function has_errors(): bool {
-		return count( $this->errors ) !== 0;
+		return count($this->errors) !== 0;
 	}
 
 	/**
@@ -80,11 +80,11 @@ 
 	 *
 	 * @return void
 	 */
-	protected function validate_columns( Schema $schema ): void {
+	protected function validate_columns(Schema $schema): void {
 		$result = array_reduce(
 			$schema->get_columns(),
-			function( array $result, Column $column ): array {
-				if ( is_null( $column->get_type() ) ) {
+			function(array $result, Column $column): array {
+				if (is_null($column->get_type())) {
 					$result[] = $column;
 				}
 				return $result;
@@ -92,12 +92,12 @@ 
 			array()
 		);
 
-		if ( count( $result ) !== 0 ) {
+		if (count($result) !== 0) {
 			$this->errors = array_merge(
 				$this->errors,
 				array_map(
-					function( Column $column ): string {
-						return \sprintf( 'Column "%s" has no type defined', $column->get_name() );
+					function(Column $column): string {
+						return \sprintf('Column "%s" has no type defined', $column->get_name());
 					},
 					$result
 				)
@@ -111,17 +111,17 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return void
 	 */
-	protected function validate_primary_key( Schema $schema ): void {
+	protected function validate_primary_key(Schema $schema): void {
 
 		$primary_keys = array_filter(
 			$schema->get_indexes(),
-			function( Index $index ): bool {
+			function(Index $index): bool {
 				return $index->is_primary();
 			}
 		);
 
-		if ( count( $primary_keys ) > 1 ) {
-			$this->errors[] = \sprintf( '%d Primary keys are defined in schema, only a single primary key can be set.', count( $primary_keys ) );
+		if (count($primary_keys) > 1) {
+			$this->errors[] = \sprintf('%d Primary keys are defined in schema, only a single primary key can be set.', count($primary_keys));
 		}
 	}
 
@@ -131,21 +131,21 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return void
 	 */
-	protected function validate_index_columns( Schema $schema ): void {
+	protected function validate_index_columns(Schema $schema): void {
 		// All defined colum names.
-		$column_names = array_keys( $schema->get_columns() );
+		$column_names = array_keys($schema->get_columns());
 
 		/** @var array<Index> */
 		$missing_columns = array_filter(
 			$schema->get_indexes(),
-			function( Index $index ) use ( $column_names ): bool {
-				return ! in_array( $index->get_column(), $column_names, true );
+			function(Index $index) use ($column_names): bool {
+				return ! in_array($index->get_column(), $column_names, true);
 			}
 		);
 
-		if ( count( $missing_columns ) > 0 ) {
-			foreach ( $missing_columns as $missing_column ) {
-				$this->errors[] = \sprintf( 'Index column %s not defined as a column in schema', $missing_column->get_column() );
+		if (count($missing_columns) > 0) {
+			foreach ($missing_columns as $missing_column) {
+				$this->errors[] = \sprintf('Index column %s not defined as a column in schema', $missing_column->get_column());
 			}
 		}
 	}
@@ -156,23 +156,23 @@ 
 	 * @param \PinkCrab\Table_Builder\Schema $schema
 	 * @return void
 	 */
-	public function validate_foreign_keys( Schema $schema ): void {
+	public function validate_foreign_keys(Schema $schema): void {
 		// All defined colum names.
-		$column_names = array_keys( $schema->get_columns() );
+		$column_names = array_keys($schema->get_columns());
 
 		// Missing columns in local table
 
 		/** @var array<Foreign_Key> */
 		$missing_columns = array_filter(
 			$schema->get_foreign_keys(),
-			function( Foreign_Key $foreign_key ) use ( $column_names ): bool {
-				return ! in_array( $foreign_key->get_column(), $column_names, true );
+			function(Foreign_Key $foreign_key) use ($column_names): bool {
+				return ! in_array($foreign_key->get_column(), $column_names, true);
 			}
 		);
 
-		if ( count( $missing_columns ) > 0 ) {
-			foreach ( $missing_columns as $missing_column ) {
-				$this->errors[] = \sprintf( 'Foreign Keys column %s not defined as a column in schema', $missing_column->get_column() );
+		if (count($missing_columns) > 0) {
+			foreach ($missing_columns as $missing_column) {
+				$this->errors[] = \sprintf('Foreign Keys column %s not defined as a column in schema', $missing_column->get_column());
 			}
 		}
 
@@ -181,14 +181,14 @@ 
 		/** @var array<Foreign_Key> */
 		$missing_references = array_filter(
 			$schema->get_foreign_keys(),
-			function( Foreign_Key $foreign_key ) : bool {
+			function(Foreign_Key $foreign_key) : bool {
 				return $foreign_key->get_reference_table() === null || $foreign_key->get_reference_column() === null;
 			}
 		);
 
-		if ( count( $missing_references ) > 0 ) {
-			foreach ( $missing_references as $missing_reference ) {
-				$this->errors[] = \sprintf( 'Foreign Keys column %s has missing reference table or column details', $missing_reference->get_keyname() );
+		if (count($missing_references) > 0) {
+			foreach ($missing_references as $missing_reference) {
+				$this->errors[] = \sprintf('Foreign Keys column %s has missing reference table or column details', $missing_reference->get_keyname());
 			}
 		}
 	}