Pink-Crab / WPDB-Table-Builder

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/Schema.php

Spacing +19 added lines, -19 removed lines

@@ -80,10 +80,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);
 		}
 	}
 
@@ -110,7 +110,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;
 	}
@@ -143,10 +143,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;
 	}
 
@@ -168,11 +168,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;
 				}
 			)
@@ -187,12 +187,12 @@ 
 	 * @return self
 	 * @throws Schema_Exception (301) If column doesn't exist.
 	 */
-	public function remove_column( string $name ): self {
-		if ( ! $this->has_column( $name ) ) {
-			throw Schema_Exception::column_not_exist( $this, $name );
+	public function remove_column(string $name): self {
+		if ( ! $this->has_column($name)) {
+			throw Schema_Exception::column_not_exist($this, $name);
 		}
 
-		unset( $this->columns[ $name ] );
+		unset($this->columns[$name]);
 
 		return $this;
 	}
@@ -205,8 +205,8 @@ 
 	 * @param string|null $key_name if not set, will use the column name a tempalte.
 	 * @return \PinkCrab\Table_Builder\Foreign_Key
 	 */
-	public function foreign_key( string $column, ?string $key_name = null ): Foreign_Key {
-		$foreign_key = new Foreign_Key( $column, $key_name );
+	public function foreign_key(string $column, ?string $key_name = null): Foreign_Key {
+		$foreign_key = new Foreign_Key($column, $key_name);
 
 		$this->foreign_keys[] = $foreign_key;
 		return $foreign_key;
@@ -218,7 +218,7 @@ 
 	 * @return bool
 	 */
 	public function has_foreign_keys(): bool {
-		return count( $this->foreign_keys ) !== 0;
+		return count($this->foreign_keys) !== 0;
 	}
 
 	/**
@@ -238,7 +238,7 @@ 
 	 * @return bool
 	 */
 	public function has_indexes(): bool {
-		return count( $this->indexes ) !== 0;
+		return count($this->indexes) !== 0;
 	}
 
 	/**
@@ -248,8 +248,8 @@ 
 	 * @param string $key_name
 	 * @return \PinkCrab\Table_Builder\Index
 	 */
-	public function index( string $column, ?string $key_name = null ): Index {
-		$index = new Index( $column, $key_name );
+	public function index(string $column, ?string $key_name = null): Index {
+		$index = new Index($column, $key_name);
 
 		$this->indexes[] = $index;
 		return $index;

src/Exception/Schema_Exception.php

Spacing +3 added lines, -3 removed lines

@@ -45,7 +45,7 @@ 
 		?Throwable $previous = null
 	) {
 		$this->schema = $schema;
-		parent::__construct( $message, $code, $previous );
+		parent::__construct($message, $code, $previous);
 	}
 
 	/**
@@ -64,10 +64,10 @@ 
 	 * @param string $column
 	 * @return Schema_Exception
 	 */
-	public static function column_not_exist( Schema $schema, string $column ): Schema_Exception {
+	public static function column_not_exist(Schema $schema, string $column): Schema_Exception {
 		return new Schema_Exception(
 			$schema,
-			\sprintf( 'column with name %s is not currently defined', $column ),
+			\sprintf('column with name %s is not currently defined', $column),
 			301
 		);
 	}

src/Exception/WPDB_DB_Delta/WPDB_Validator_Exception.php

Spacing +3 added lines, -3 removed lines

@@ -61,7 +61,7 @@ 
 	) {
 		$this->schema            = $schema;
 		$this->validation_errors = $validation_errors;
-		parent::__construct( $message, $code, $previous );
+		parent::__construct($message, $code, $previous);
 	}
 
 	/**
@@ -72,11 +72,11 @@ 
 	 * @return WPDB_Validator_Exception
 	 * @code 201
 	 */
-	public static function failed_validation( Schema $schema, array $errors ): WPDB_Validator_Exception {
+	public static function failed_validation(Schema $schema, array $errors): WPDB_Validator_Exception {
 		return new WPDB_Validator_Exception(
 			$schema,
 			$errors,
-			sprintf( '%s failed with %d errors', $schema->get_table_name(), count( $errors ) ),
+			sprintf('%s failed with %d errors', $schema->get_table_name(), count($errors)),
 			201
 		);
 	}

src/Exception/Engine_Exception.php

Spacing +5 added lines, -5 removed lines

@@ -45,7 +45,7 @@ 
 		?Throwable $previous = null
 	) {
 		$this->schema = $schema;
-		parent::__construct( $message, $code, $previous );
+		parent::__construct($message, $code, $previous);
 	}
 
 	/**
@@ -65,8 +65,8 @@ 
 	 * @return Engine_Exception
 	 * @code 101
 	 */
-	public static function create_table( Schema $schema, string $error ): Engine_Exception {
-		return new self( $schema, $error, 101 );
+	public static function create_table(Schema $schema, string $error): Engine_Exception {
+		return new self($schema, $error, 101);
 	}
 
 	/**
@@ -77,7 +77,7 @@ 
 	 * @return Engine_Exception
 	 * @code 102
 	 */
-	public static function drop_table( Schema $schema, string $error ): Engine_Exception {
-		return new self( $schema, $error, 102 );
+	public static function drop_table(Schema $schema, string $error): Engine_Exception {
+		return new self($schema, $error, 102);
 	}
 }

src/Column.php

Spacing +8 added lines, -8 removed lines

@@ -100,7 +100,7 @@ 
 	 */
 	protected $unsigned = null;
 
-	public function __construct( string $name ) {
+	public function __construct(string $name) {
 		$this->name = $name;
 	}
 
@@ -110,7 +110,7 @@ 
 	 * @param string $type
 	 * @return self
 	 */
-	public function type( string $type ): self {
+	public function type(string $type): self {
 		$this->type = $type;
 		return $this;
 	}
@@ -121,7 +121,7 @@ 
 	 * @param integer $length
 	 * @return self
 	 */
-	public function length( int $length ): self {
+	public function length(int $length): self {
 		$this->length = $length;
 		return $this;
 	}
@@ -134,7 +134,7 @@ 
 	 * @param integer $precision
 	 * @return self
 	 */
-	public function precision( int $precision ): self {
+	public function precision(int $precision): self {
 		$this->precision = $precision;
 		return $this;
 	}
@@ -145,7 +145,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;
 	}
@@ -156,7 +156,7 @@ 
 	 * @param mixed $default
 	 * @return self
 	 */
-	public function default( $default ): self {
+	public function default($default): self {
 		$this->default = $default;
 		return $this;
 	}
@@ -167,7 +167,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;
 	}
@@ -178,7 +178,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/Index.php

Spacing +8 added lines, -8 removed lines

@@ -67,8 +67,8 @@ 
 	 */
 	protected $primary = false;
 
-	public function __construct( string $column, ?string $key_name = null ) {
-		$this->key_name = $key_name ?? 'ix_' . $column;
+	public function __construct(string $column, ?string $key_name = null) {
+		$this->key_name = $key_name ?? 'ix_'.$column;
 		$this->column   = $column;
 	}
 
@@ -78,7 +78,7 @@ 
 	 * @param bool $primary  is primary key
 	 * @return self
 	 */
-	public function primary( bool $primary = true ): self {
+	public function primary(bool $primary = true): self {
 		$this->primary = $primary;
 		return $this;
 	}
@@ -90,7 +90,7 @@ 
 	 * @param boolean $unique
 	 * @return self
 	 */
-	public function unique( bool $unique = true ): self {
+	public function unique(bool $unique = true): self {
 		$this->unique = $unique;
 		return $this;
 	}
@@ -102,7 +102,7 @@ 
 	 * @param boolean $full_text
 	 * @return self
 	 */
-	public function full_text( bool $full_text = true ): self {
+	public function full_text(bool $full_text = true): self {
 		$this->full_text = $full_text;
 		return $this;
 	}
@@ -174,11 +174,11 @@ 
 	 * @return string
 	 */
 	public function get_type(): string {
-		if ( $this->is_primary() ) {
+		if ($this->is_primary()) {
 			return 'primary';
-		} elseif ( $this->is_unique() ) {
+		} elseif ($this->is_unique()) {
 			return 'unique';
-		} elseif ( $this->is_full_text() ) {
+		} elseif ($this->is_full_text()) {
 			return 'fulltext';
 		} else {
 			return '';

src/Engines/WPDB_DB_Delta/DB_Delta_Engine.php

Spacing +28 added lines, -28 removed lines

@@ -63,7 +63,7 @@ 
 	 */
 	protected $schema;
 
-	public function __construct( \wpdb $wpdb ) {
+	public function __construct(\wpdb $wpdb) {
 		$this->wpdb = $wpdb;
 
 		// Set the translator and valiator
@@ -78,10 +78,10 @@ 
 	 * @return void
 	 * @throws WPDB_Validator_Exception if fails validation (code 201)
 	 */
-	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 WPDB_Validator_Exception::failed_validation(
 				$schema,
 				$this->validator->get_errors()
@@ -98,26 +98,26 @@ 
 	 * @throws WPDB_Validator_Exception if fails validation (code 201)
 	 * @throws Engine_Exception If errors thrown creating table (code 101)
 	 */
-	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';
+		require_once ABSPATH.'wp-admin/includes/upgrade.php';
 
 		\ob_start();
-		dbDelta( $query );
+		dbDelta($query);
 		$output = \ob_get_clean();
 
 		// If output captured, throw.
-		if ( '' !== $output ) {
-			throw Engine_Exception::create_table( $schema, $output ?: '' );
+		if ('' !== $output) {
+			throw Engine_Exception::create_table($schema, $output ?: '');
 		}
 
 		// Throw if WPDB has errors.
-		if ( '' !== $this->wpdb->last_error ) {
-			throw Engine_Exception::create_table( $schema, $this->wpdb->last_error );
+		if ('' !== $this->wpdb->last_error) {
+			throw Engine_Exception::create_table($schema, $this->wpdb->last_error);
 		}
 
 		return true;
@@ -130,8 +130,8 @@ 
 	 * @return string
 	 * @throws WPDB_Validator_Exception if fails validation (code 201)
 	 */
-	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();
 	}
 
@@ -142,11 +142,11 @@ 
 	 * @return void
 	 * @throws WPDB_Validator_Exception If fails schema validation (code 201)
 	 */
-	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 ) ) {
-			throw WPDB_Validator_Exception::failed_validation( $schema, $this->validator->get_errors() );
+		if ( ! $this->validator->validate($schema)) {
+			throw WPDB_Validator_Exception::failed_validation($schema, $this->validator->get_errors());
 		}
 	}
 
@@ -158,22 +158,22 @@ 
 	 * @throws WPDB_Validator_Exception If fails schema validation (code 201)
 	 * @throws Engine_Exception If error thrown dropping table. (code 102)
 	 */
-	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);
 
 		\ob_start();
-		$this->wpdb->get_results( $query ); // phpcs:disable WordPress.DB.PreparedSQL.NotPrepared
+		$this->wpdb->get_results($query); // phpcs:disable WordPress.DB.PreparedSQL.NotPrepared
 		$output = \ob_get_clean();
 
 		// If output captured, throw.
-		if ( '' !== $output ) {
-			throw Engine_Exception::drop_table( $schema, $output ?: '' );
+		if ('' !== $output) {
+			throw Engine_Exception::drop_table($schema, $output ?: '');
 		}
 
 		// Throw if WPDB has errors.
-		if ( '' !== $this->wpdb->last_error ) {
-			throw Engine_Exception::drop_table( $schema, $this->wpdb->last_error );
+		if ('' !== $this->wpdb->last_error) {
+			throw Engine_Exception::drop_table($schema, $this->wpdb->last_error);
 		}
 
 		return true;
@@ -186,8 +186,8 @@ 
 	 * @return string
 	 * @throws WPDB_Validator_Exception If fails schema validation (code 201)
 	 */
-	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()};";
 	}
 
@@ -201,10 +201,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

Spacing +34 added lines, -34 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(), $column->get_precision() ),
+					$this->type_mapper($column->get_type() ?? '', $column->get_length(), $column->get_precision()),
 					$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, ?int $precision ): string {
-		$type = strtoupper( $type );
-		switch ( $type ) {
+	protected function type_mapper(string $type, ?int $length, ?int $precision): string {
+		$type = strtoupper($type);
+		switch ($type) {
 			// With length
 			case 'CHAR':
 			case 'VARCHAR':
@@ -100,7 +100,7 @@ 
 			case 'DATETIME':
 			case 'TIMESTAMP':
 			case 'TIME':
-				return is_null( $length ) ? $type : "{$type}({$length})";
+				return is_null($length) ? $type : "{$type}({$length})";
 
 			// Floats
 			case 'FLOAT':
@@ -109,7 +109,7 @@ 
 			case 'DECIMAL':
 			case 'DEC':
 				$precision = $precision ?? 1;
-				return is_null( $length ) ? $type : "{$type}({$length},{$precision})";
+				return is_null($length) ? $type : "{$type}({$length},{$precision})";
 
 			default:
 				return $type;
@@ -123,15 +123,15 @@ 
 	 * @param mixed|null $default
 	 * @return string
 	 */
-	protected function parse_default( string $type, $default ): string {
-		if ( is_null( $default ) ) {
+	protected function parse_default(string $type, $default): string {
+		if (is_null($default)) {
 			return '';
 		}
 
-		$type = strtoupper( $type );
+		$type = strtoupper($type);
 
 		// String values.
-		if ( in_array( $type, array( 'JSON', 'CHAR', 'VARCHAR', 'BINARY', 'VARBINARY', 'TEXT', 'BLOB' ), true ) ) {
+		if (in_array($type, array('JSON', 'CHAR', 'VARCHAR', 'BINARY', 'VARBINARY', 'TEXT', 'BLOB'), true)) {
 			return " DEFAULT '{$default}'";
 		}
 
@@ -146,14 +146,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();
 				}
 			)
@@ -166,16 +166,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_key_name();
 				$index_type = $index_group[0]->get_type();
 				$columns    = array_map(
-					function( $e ) {
+					function($e) {
 						return $e->get_column();
 					},
 					$index_group
@@ -183,13 +183,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)
 		);
 	}
 
@@ -199,17 +199,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_key_name(),
 					$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()
@@ -222,16 +222,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_key_name() . '_' . $index->get_type() ][] = $index;
+				$carry[$index->get_key_name().'_'.$index->get_type()][] = $index;
 
 				return $carry;
 			},