Skip to content

Commit 16472cf

Browse files
committed
feat: use native PrintChanges from Cycle/Database package
1 parent d03489c commit 16472cf

File tree

4 files changed

+10
-180
lines changed

4 files changed

+10
-180
lines changed

src/Bridge/Laravel/Console/Commands/ORM/Generators/ShowChanges.php

Lines changed: 0 additions & 170 deletions
This file was deleted.

src/Bridge/Laravel/Console/Commands/ORM/MigrateCommand.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,11 @@
77
use Cycle\Migrations\State;
88
use Cycle\Schema\Compiler as CycleSchemaCompiler;
99
use Cycle\Schema\Generator\Migrations\GenerateMigrations;
10+
use Cycle\Schema\Generator\PrintChanges;
1011
use Cycle\Schema\Registry;
1112
use Symfony\Component\Console\Command\Command;
1213
use WayOfDev\Cycle\Bridge\Laravel\Console\Commands\Migrations\AbstractCommand;
1314
use WayOfDev\Cycle\Bridge\Laravel\Console\Commands\Migrations\MigrateCommand as DatabaseMigrateCommand;
14-
use WayOfDev\Cycle\Bridge\Laravel\Console\Commands\ORM\Generators\ShowChanges;
1515
use WayOfDev\Cycle\Contracts\CacheManager as CacheManagerContract;
1616
use WayOfDev\Cycle\Contracts\GeneratorLoader;
1717
use WayOfDev\Cycle\Schema\Compiler;
@@ -40,13 +40,13 @@ public function handle(
4040

4141
foreach ($this->migrator->getMigrations() as $migration) {
4242
if ($migration->getState()->getStatus() !== State::STATUS_EXECUTED) {
43-
$this->warn('Outstanding migrations found, run `cycle:orm:migrate` first!');
43+
$this->warn('Outstanding migrations found, run `cycle:migrate` first!');
4444

4545
return self::FAILURE;
4646
}
4747
}
4848

49-
$diff = new ShowChanges($this->output);
49+
$diff = new PrintChanges($this->output);
5050
$queue = $generators->add(GeneratorLoader::GROUP_RENDER, $diff);
5151

5252
$schemaCompiler = Compiler::compile($registry, $queue);

src/Bridge/Laravel/Console/Commands/ORM/SyncCommand.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,10 @@
44

55
namespace WayOfDev\Cycle\Bridge\Laravel\Console\Commands\ORM;
66

7+
use Cycle\Schema\Generator\PrintChanges;
78
use Cycle\Schema\Generator\SyncTables;
89
use Cycle\Schema\Registry;
910
use Illuminate\Console\Command;
10-
use WayOfDev\Cycle\Bridge\Laravel\Console\Commands\ORM\Generators\ShowChanges;
1111
use WayOfDev\Cycle\Contracts\CacheManager as CacheManagerContract;
1212
use WayOfDev\Cycle\Contracts\GeneratorLoader;
1313
use WayOfDev\Cycle\Schema\Compiler;
@@ -28,7 +28,7 @@ public function handle(
2828
Registry $registry,
2929
CacheManagerContract $cache
3030
): int {
31-
$diff = new ShowChanges($this->output);
31+
$diff = new PrintChanges($this->output);
3232
$queue = $generators
3333
->add(GeneratorLoader::GROUP_RENDER, $diff)
3434
->add(GeneratorLoader::GROUP_POSTPROCESS, new SyncTables());

tests/src/Bridge/Laravel/Console/Commands/ORM/MigrateCommandTest.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,11 @@ class MigrateCommandTest extends TestCase
1414
public const USER_MIGRATION = [
1515
'default.users',
1616
'create table',
17-
'add column id',
18-
'add column user_id',
19-
'add column name',
17+
'add column [id]',
18+
'add column [user_id]',
19+
'add column [name]',
2020
'add index on [user_id]',
21-
'add foreign key on user_id',
21+
'add foreign key on [user_id]',
2222
];
2323

2424
protected function setUp(): void
@@ -84,7 +84,7 @@ class Tag
8484
$this->assertConsoleCommandOutputContainsStrings('cycle:orm:migrate', ['-r' => true], [
8585
'default.tags',
8686
'create table',
87-
'add column id',
87+
'add column [id]',
8888
]);
8989

9090
File::delete($entity);

0 commit comments

Comments
 (0)