search

laravel-doctrine / fluent

Fluent mapping driver for Doctrine2
http://www.laraveldoctrine.org/docs/current/fluent
MIT License
43 stars 22 forks source link

Locale has to be set on builder level #38

Closed guiwoda closed 8 years ago

guiwoda commented 8 years ago

It actually cannot be set on a mapped field, as "it makes no sense".

See https://github.com/Atlantic18/DoctrineExtensions/blob/master/lib/Gedmo/Translatable/Mapping/Driver/Annotation.php#L79

guiwoda commented 8 years ago

Also added a delay to the extension, so that field validation happens after all fields have been mapped.

guiwoda commented 8 years ago

@patrickbrouwers do you want to take a look at this? Or should I merge?

patrickbrouwers commented 8 years ago

Sorry, I completely forgot about this one :$