Create PSR-2 compliant migration files

aimeos · Nov 15, 2024 · a7641b0 · a7641b0
1 parent ae6a1bf
commit a7641b0
Show file tree

Hide file tree

Showing 5 changed files with 38 additions and 37 deletions.
diff --git a/src/Generate.php b/src/Generate.php
@@ -99,34 +99,34 @@ protected function col( array $def ) : string
 		foreach( $def as $name => $e )
 		{
 			if( in_array( $e['type'], $types ) ) {
-				$string = '$t->' . $e['type'] . '( \'' . $name . '\' )';
+				$string = '$t->' . $e['type'] . '(\'' . $name . '\')';
 			} else {
-				$string = '$t->col( \'' . $name . '\', \'' . $e['type'] . '\' )';
+				$string = '$t->col(\'' . $name . '\', \'' . $e['type'] . '\')';
 			}
 
 			foreach( ['seq', 'fixed', 'unsigned', 'null'] as $key )
 			{
 				if( $e[$key] ?? false ) {
-					$string .= '->' . $key . '( true )';
+					$string .= '->' . $key . '(true)';
 				}
 			}
 
 			foreach( ['length', 'precision', 'scale'] as $key )
 			{
 				if( $e[$key] ?? false ) {
-					$string .= '->' . $key . '( ' . $e[$key] . ' )';
+					$string .= '->' . $key . '(' . $e[$key] . ')';
 				}
 			}
 
 			foreach( ['default', 'comment'] as $key )
 			{
 				if( $e[$key] ?? false ) {
-					$string .= '->' . $key . '( \'' . $e[$key] . '\' )';
+					$string .= '->' . $key . '(\'' . $e[$key] . '\')';
 				}
 			}
 
 			foreach( $e['opt'] ?? [] as $key => $value ) {
-				$string .= '->opt( \'' . $key . '\', \'' . $value . '\' )';
+				$string .= '->opt(\'' . $key . '\', \'' . $value . '\')';
 			}
 
 			$lines[] = $string . ';';
@@ -148,12 +148,12 @@ protected function foreign( array $def ) : string
 
 		foreach( $def as $name => $e )
 		{
-			$string = '$t->foreign( ' . json_encode( $e['localcol'] ) . ', \'' . ( $e['fktable'] ?? '' ) . '\', ' . json_encode( $e['fkcol'] ) . ', ' . ( $e['name'] ? '\'' . $e['name'] . '\'' : 'null' ) . ' )';
+			$string = '$t->foreign(' . json_encode( $e['localcol'] ) . ', \'' . ( $e['fktable'] ?? '' ) . '\', ' . json_encode( $e['fkcol'] ) . ', ' . ( $e['name'] ? '\'' . $e['name'] . '\'' : 'null' ) . ')';
 
 			foreach( ['onDelete', 'onUpdate'] as $key )
 			{
 				if( $e[$key] ?? false ) {
-					$string .= '->' . $key . '( \'' . $e[$key] . '\' )';
+					$string .= '->' . $key . '(\'' . $e[$key] . '\')';
 				}
 			}
 
@@ -177,11 +177,11 @@ protected function index( array $def ) : string
 		foreach( $def as $name => $e )
 		{
 			if( $e['primary'] ?? false ) {
-				$lines[] = '$t->primary( ' . json_encode( $e['columns'] ) . ', ' . ( $e['name'] ? '\'' . $e['name'] . '\'' : 'null' ) . ' );';
+				$lines[] = '$t->primary(' . json_encode( $e['columns'] ) . ', ' . ( $e['name'] ? '\'' . $e['name'] . '\'' : 'null' ) . ');';
 			} elseif( $e['unique'] ?? false ) {
-				$lines[] = '$t->unique( ' . json_encode( $e['columns'] ) . ', ' . ( $e['name'] ? '\'' . $e['name'] . '\'' : 'null' ) . ' );';
+				$lines[] = '$t->unique(' . json_encode( $e['columns'] ) . ', ' . ( $e['name'] ? '\'' . $e['name'] . '\'' : 'null' ) . ');';
 			} else {
-				$lines[] = '$t->index( ' . json_encode( $e['columns'] ?? [] ) . ', ' . ( $e['name'] ? '\'' . $e['name'] . '\'' : 'null' ) . ', ' . $this->array( $e['flags'] ?? [] ) . ', ' . $this->array( $e['options'] ?? [] ) . ' );';
+				$lines[] = '$t->index(' . json_encode( $e['columns'] ?? [] ) . ', ' . ( $e['name'] ? '\'' . $e['name'] . '\'' : 'null' ) . ', ' . $this->array( $e['flags'] ?? [] ) . ', ' . $this->array( $e['options'] ?? [] ) . ');';
 			}
 		}
 
@@ -220,7 +220,7 @@ protected function sequence( array $def, string $template ) : string
 		foreach( ['cache', 'start', 'step'] as $key )
 		{
 			if( $def[$key] ?? false ) {
-				$string .= '->' . $key . '( ' . $def[$key] . ' )';
+				$string .= '->' . $key . '(' . $def[$key] . ')';
 			}
 		}
 

diff --git a/stubs/sequence.stub b/stubs/sequence.stub
@@ -7,13 +7,13 @@ use Aimeos\Upscheme\Schema\Sequence;
 
 return new class( $this ) extends Base
 {
-	public function up()
-	{
-		$this->info( 'Create sequence "{{NAME}}"' );
+    public function up()
+    {
+        $this->info('Create sequence "{{NAME}}"');
 
-		$this->db( '{{DB}}' )->sequence( '{{NAME}}', function( Sequence $s ) {
+        $this->db('{{DB}}')->sequence('{{NAME}}', function(Sequence $s) {
 
-			{{SEQUENCE}}
-		} );
-	}
+            {{SEQUENCE}}
+        });
+    }
 };
diff --git a/stubs/table.stub b/stubs/table.stub
@@ -7,23 +7,23 @@ use Aimeos\Upscheme\Schema\Table;
 
 return new class( $this ) extends Base
 {
-	public function after() : array
-	{
-		return [{{AFTER}}];
-	}
+    public function after(): array
+    {
+        return [{{AFTER}}];
+    }
 
 
-	public function up()
-	{
-		$this->info( 'Create table "{{NAME}}"' );
+    public function up()
+    {
+        $this->info('Create table "{{NAME}}"');
 
-		$this->db( '{{DB}}' )->table( '{{NAME}}', function( Table $t ) {
+        $this->db('{{DB}}')->table('{{NAME}}', function(Table $t) {
 
-			{{COLUMN}}
+            {{COLUMN}}
 
-			{{FOREIGN}}
+            {{FOREIGN}}
 
-			{{INDEX}}
-		} );
-	}
+            {{INDEX}}
+        });
+    }
 };
diff --git a/stubs/view.stub b/stubs/view.stub
@@ -7,8 +7,8 @@ return new class( $this ) extends Base
 {
 	public function up()
 	{
-		$this->info( 'Create view "{{NAME}}"' );
+		$this->info('Create view "{{NAME}}"');
 
-		$this->db( '{{DB}}' )->view( '{{NAME}}', '{{SQL}}' );
+		$this->db('{{DB}}')->view('{{NAME}}', '{{SQL}}');
 	}
 };
diff --git a/tests/config.php b/tests/config.php
@@ -1,10 +1,11 @@
 <?php
 
 return [
-	'driver' => 'pdo_sqlite',
+	'driver' => 'pdo_pgsql',
 	'path' => 'sqlite.test',
 	'host' => '127.0.0.1',
+//	'port' => 5433,
 	'dbname' => 'upscheme',
-	'user' => 'upscheme',
-	'password' => 'upscheme'
+	'user' => 'aimeos',
+	'password' => 'aimeos'
 ];