Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add support for array of enums #9497

Merged
merged 16 commits into from
Apr 4, 2022
Merged

Add support for array of enums #9497

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
14865cf
Add support for array of enums
Feb 9, 2022
4010052
Reference is_array and array_map through a use statement nstead of gl…
Feb 9, 2022
f9f7c24
Return the value of an array of enums correctly
Feb 9, 2022
7235413
Add enum array mapping test
Feb 9, 2022
948f125
Fix order of use parameters
Feb 9, 2022
ea22b7c
Fix return type docblock
Feb 9, 2022
98c25cb
Apply phpcs feedback
Feb 9, 2022
c35d880
Fix static closure
Feb 9, 2022
3edfe52
Add missing return type to static closure
Feb 9, 2022
01a1b75
Add helper method for enum initialization to reduce code duplication
Feb 20, 2022
12fd738
Merge branch '2.12.x' into feature/enum-arrays
sanderdlm Apr 4, 2022
c09fe29
Fix CS
derrabus Apr 4, 2022
b6e7472
Replace mixed typehints with more specific ones
Apr 4, 2022
a1b5400
Update docblock type hint to allow for array of string/int
Apr 4, 2022
c6e57a3
Fix types
derrabus Apr 4, 2022
0eee71f
Fix types
derrabus Apr 4, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch '2.12.x' into feature/enum-arrays
  • Loading branch information
@sanderdlm
sanderdlm committed Apr 4, 2022
commit 12fd73804061c59ea295162d3403f707d3fa65ac
13 changes: 13 additions & 0 deletions tests/Doctrine/Tests/ORM/Functional/EnumTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -180,4 +180,17 @@ public function testEnumArrayMapping(): void
$this->assertContains(Unit::Gram, $fetchedScale->supportedUnits);
$this->assertContains(Unit::Meter, $fetchedScale->supportedUnits);
}

public function testEnumWithDefault(): void
{
$this->setUpEntitySchema([CardWithDefault::class]);

$table = $this->_em->getClassMetadata(CardWithDefault::class)->getTableName();
$cardId = uniqid('', true);

$this->_em->getConnection()->insert($table, ['id' => $cardId]);
$card = $this->_em->find(CardWithDefault::class, $cardId);

self::assertSame(Suit::Hearts, $card->suit);
}
}
You are viewing a condensed version of this merge commit. You can view the full changes here.