Conflict with sonata-project/intl-bundle about sonataintlbundle

[franmomu]

Thanks for reporting! I'll try to take a look later

[phansys]

Please, note that "twig/intl-extra" is not an opt in replacement, since it doesn't provide the same features than this project.
By instance, the locale inference provided here through LocaleDetectorInterface is not available.
Other example pointed here #280 (comment).

[franmomu]

Fixed in sonata-project/SonataAdminBundle#5981

[core23]

This issue is not fixed at all. There is just a fix for one of our bundles. The error still occurs if you try to call format_datetime somewhere in your code.

See: sonata-project/SonataNewsBundle#591

[core23]

IMHO the only way to fix this issue is to check the arguments you pass to the filter and delegate this to the twig filter if necesary.

[core23]

This could fix the problem: #302

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[VincentLanglet]

I just encountered this bug

Twig\Extra\Intl\IntlExtension::formatDateTime(): Argument #5 ($pattern) must be of type string, null given

I'm using both SonataAdmin and SonataIntl.

Shouldn't we prefix every method here by sonata_ and deprecate the old name ?

@franmomu @phansys @core23 @jordisala1991 WDYT ?

[jordisala1991]

We could probably move the release of 3.0 for this bundle (it was never done) and rename those functions in the process.

[jordisala1991]

Because this bundle adds some value right? Or the things done here can be done with a combination of official twig extra packages?

[VincentLanglet]

We could probably move the release of 3.0 for this bundle (it was never done) and rename those functions in the process.

Yes, this is what I had in mind with #512