From a80b9d9d1ae5cf1474a751e057def209e37e2f6d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andr=C3=A9=20Polykanine?= Date: Fri, 7 Feb 2025 20:12:04 +0100 Subject: [PATCH 1/3] Make:entity: Add question about unsigned integer --- src/Maker/MakeEntity.php | 2 ++ tests/Maker/MakeEntityTest.php | 6 ++++++ 2 files changed, 8 insertions(+) diff --git a/src/Maker/MakeEntity.php b/src/Maker/MakeEntity.php index e0498f36b..6652658c5 100644 --- a/src/Maker/MakeEntity.php +++ b/src/Maker/MakeEntity.php @@ -435,6 +435,8 @@ private function askForNextField(ConsoleStyle $io, array $fields, string $entity // set type according to user decision $classProperty->type = $io->confirm('Can this field store multiple enum values', false) ? 'simple_array' : 'string'; + } else if ('integer' === $type) { + $classProperty->options['unsigned'] = $io->confirm('Is this an unsigned integer', false); } if ($io->confirm('Can this field be null in the database (nullable)', false)) { diff --git a/tests/Maker/MakeEntityTest.php b/tests/Maker/MakeEntityTest.php index 54059bad3..cb6b1a0a0 100644 --- a/tests/Maker/MakeEntityTest.php +++ b/tests/Maker/MakeEntityTest.php @@ -194,6 +194,12 @@ public function getTestDetails(): \Generator '255', // length // nullable 'y', + 'rating', + 'integer', + // Unsigned + 'y', + // Not nullable + 'n', 'createdAt', // use default datetime '', From 771a825a0fa7dd685b39f40d85b99ecbbc517ecb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andr=C3=A9=20Polykanine?= Date: Fri, 7 Feb 2025 20:20:54 +0100 Subject: [PATCH 2/3] Fix CS --- src/Maker/MakeEntity.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Maker/MakeEntity.php b/src/Maker/MakeEntity.php index 6652658c5..8e22f38d9 100644 --- a/src/Maker/MakeEntity.php +++ b/src/Maker/MakeEntity.php @@ -435,7 +435,7 @@ private function askForNextField(ConsoleStyle $io, array $fields, string $entity // set type according to user decision $classProperty->type = $io->confirm('Can this field store multiple enum values', false) ? 'simple_array' : 'string'; - } else if ('integer' === $type) { + } elseif ('integer' === $type) { $classProperty->options['unsigned'] = $io->confirm('Is this an unsigned integer', false); } From aaebc854c1fcc6590928b31b05dbef1f109fee99 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andr=C3=A9=20Polykanine?= Date: Fri, 7 Feb 2025 20:32:10 +0100 Subject: [PATCH 3/3] Attempt to fix test --- tests/Maker/MakeEntityTest.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/Maker/MakeEntityTest.php b/tests/Maker/MakeEntityTest.php index cb6b1a0a0..99da453b6 100644 --- a/tests/Maker/MakeEntityTest.php +++ b/tests/Maker/MakeEntityTest.php @@ -198,8 +198,8 @@ public function getTestDetails(): \Generator 'integer', // Unsigned 'y', - // Not nullable - 'n', + // Nullable + 'y', 'createdAt', // use default datetime '',