Skip to content

Commit c9ae401

Browse files
committed
bug #991 Check if json_array type exists before unsetting it (HypeMC)
This PR was squashed before being merged into the 1.0-dev branch. Discussion ---------- Check if json_array type exists before unsetting it Fixes #987 Commits ------- e0ffa99 Fix composer package names 8b1806d Check if json_array type exists before unsetting it
2 parents fd758db + e0ffa99 commit c9ae401

File tree

5 files changed

+20
-12
lines changed

5 files changed

+20
-12
lines changed

src/Maker/MakeCrud.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -260,7 +260,7 @@ public function configureDependencies(DependencyBuilder $dependencies)
260260

261261
$dependencies->addClassDependency(
262262
DoctrineBundle::class,
263-
'orm-pack'
263+
'orm'
264264
);
265265

266266
$dependencies->addClassDependency(

src/Maker/MakeEntity.php

+14-6
Original file line numberDiff line numberDiff line change
@@ -374,9 +374,7 @@ private function askForNextField(ConsoleStyle $io, array $fields, string $entity
374374
}
375375

376376
$type = null;
377-
$types = Type::getTypesMap();
378-
// remove deprecated json_array
379-
unset($types[Type::JSON_ARRAY]);
377+
$types = $this->getTypesMap();
380378

381379
$allValidTypes = array_merge(
382380
array_keys($types),
@@ -428,9 +426,7 @@ private function askForNextField(ConsoleStyle $io, array $fields, string $entity
428426

429427
private function printAvailableTypes(ConsoleStyle $io)
430428
{
431-
$allTypes = Type::getTypesMap();
432-
// remove deprecated json_array
433-
unset($allTypes[Type::JSON_ARRAY]);
429+
$allTypes = $this->getTypesMap();
434430

435431
if ('Hyper' === getenv('TERM_PROGRAM')) {
436432
$wizard = 'wizard 🧙';
@@ -885,4 +881,16 @@ private function getEntityNamespace(): string
885881
{
886882
return $this->doctrineHelper->getEntityNamespace();
887883
}
884+
885+
private function getTypesMap(): array
886+
{
887+
$types = Type::getTypesMap();
888+
889+
// remove deprecated json_array if it exists
890+
if (\defined(sprintf('%s::JSON_ARRAY', Type::class))) {
891+
unset($types[Type::JSON_ARRAY]);
892+
}
893+
894+
return $types;
895+
}
888896
}

src/Maker/MakeRegistrationForm.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -479,7 +479,7 @@ public function configureDependencies(DependencyBuilder $dependencies): void
479479

480480
$dependencies->addClassDependency(
481481
DoctrineBundle::class,
482-
'orm-pack'
482+
'orm'
483483
);
484484

485485
$dependencies->addClassDependency(

tests/Maker/MakeEntityLegacyTest.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -509,7 +509,7 @@ public function getTestDetails()
509509
'',
510510
])
511511
->setRequiredPhpVersion(70200)
512-
->addExtraDependencies('ux-turbo-mercure')
512+
->addExtraDependencies('symfony/ux-turbo-mercure')
513513
->configureDatabase()
514514
->addReplacement(
515515
'.env',
@@ -546,7 +546,7 @@ public function getTestDetails()
546546
])
547547
->setRequiredPhpVersion(70200)
548548
->addExtraDependencies('api')
549-
->addExtraDependencies('ux-turbo-mercure')
549+
->addExtraDependencies('symfony/ux-turbo-mercure')
550550
->setFixtureFilesPath(__DIR__.'/../fixtures/legacy/MakeEntity/MakeEntity')
551551
->configureDatabase()
552552
->updateSchemaAfterCommand()

tests/Maker/MakeEntityTest.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -619,7 +619,7 @@ public function getTestDetails()
619619
])
620620
->setRequiredPhpVersion(80000)
621621
->useDoctrineAttributeMapping()
622-
->addExtraDependencies('ux-turbo-mercure')
622+
->addExtraDependencies('symfony/ux-turbo-mercure')
623623
->configureDatabase()
624624
->addReplacement(
625625
'.env',
@@ -657,7 +657,7 @@ public function getTestDetails()
657657
->setRequiredPhpVersion(80000)
658658
->useDoctrineAttributeMapping()
659659
->addExtraDependencies('api')
660-
->addExtraDependencies('ux-turbo-mercure')
660+
->addExtraDependencies('symfony/ux-turbo-mercure')
661661
->setFixtureFilesPath(__DIR__.'/../fixtures/MakeEntity')
662662
->configureDatabase()
663663
->updateSchemaAfterCommand()

0 commit comments

Comments
 (0)