Locastic / ApiPlatformTranslationBundle

Translation bundle for ApiPlatform based on Sylius translation
MIT License
85 stars 28 forks source link

Handle null locales #63

Closed guillaume-sainthillier closed 1 year ago

guillaume-sainthillier commented 1 year ago

This PR allow the handling of null locale in translation without bug on doctrine/collection^2 by skipping the add.

For me, next major version should not allow null locale in the core.

Another solution could be to treat null locale as same as the default one.

paullla commented 1 year ago

@guillaume-sainthillier thank you for the PR!