Enumeration support #513

Merged
merged 3 commits into from Nov 15, 2012

Conversation

Projects
None yet
3 participants
Owner

FabioBatSilva commented Nov 13, 2012

This patch adds support for @Enum. doctrine/common#167

Please let me know if there is another annotations where we can use @Enum.

Hello,

thank you for positing this Pull Request. I have automatically opened an issue on our Jira Bug Tracker for you with the details of this Pull-Request. See the Link:

http://doctrine-project.org/jira/browse/DDC-2142

@beberlei beberlei commented on the diff Nov 14, 2012

lib/Doctrine/ORM/Mapping/ManyToMany.php
public $inversedBy;
- /** @var array<string> */
+
+ /**
+ * @var array<string>
@beberlei

beberlei Nov 14, 2012

Owner

we know the values here, but i think the casing doesn't matter here

@FabioBatSilva

FabioBatSilva Nov 14, 2012

Owner

Yes, The @Enum works just for integer | string values, cascade is a string array.

@beberlei beberlei added a commit that referenced this pull request Nov 15, 2012

@beberlei beberlei Merge pull request #513 from FabioBatSilva/annot-enum
Enumeration support
d0428df

@beberlei beberlei merged commit d0428df into doctrine:master Nov 15, 2012

1 check passed

default The Travis build passed
Details

FabioBatSilva deleted the FabioBatSilva:annot-enum branch Jan 18, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment