laravel-doctrine icon indicating copy to clipboard operation
laravel-doctrine copied to clipboard

Published 20 hours ago • verified publisher icon mitchellvanw

Added configurable naming strategy

Open Supericy opened this issue 10 years ago • 4 comments

Allows for the user to choose the naming strategy.

eg. 'naming_strategy' => 'Doctrine\ORM\Mapping\UnderscoreNamingStrategy'

Supericy avatar Dec 22 '14 23:12 Supericy

Possible suggestion: Keep the metadata config the same, and add a separate 'metadata_naming_strategy' key to preserve backwards compatibility with old config files.

Supericy avatar Dec 22 '14 23:12 Supericy

I think we might put this in the release for L5 rather than the current one, due to the config change (L5 requires a different approach to configuration anyway). @mitchellvanw ?

kirkbushell avatar Jan 26 '15 16:01 kirkbushell

@kirkbushell Yeah. Or use the alternate naming I proposed in my comment to preserve backwards compatibility..

Supericy avatar Jan 26 '15 20:01 Supericy

@Supericy I'm not a big fan of that - we'd end up with conditionals every time a config file changes just to support previous versions, when all they'd need to do is update their configuration file.

kirkbushell avatar Jan 28 '15 12:01 kirkbushell