Pink-Crab / WPDB-Table-Builder

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_key_name() );
+		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_key_name());
 			}
 		}
 	}

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,8 +60,8 @@ 
 	 * @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 $key_name = null ) {
-		$this->key_name = $key_name ?? 'fk_' . $column;
+	public function __construct(string $column, ?string $key_name = null) {
+		$this->key_name = $key_name ?? '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;
 	}