Hans Mackowiak

Some Question about understanding: The real Filter from this Bundle `sonata.admin.orm.filter.type.datetime_range` does point to `sonata.form.type.datetime_range`. This is the part which i currently Alter with my PR. but the AbstactDateFilter also...

i meant it for DateTimeRangeFilter, AbstractDateFilter sets this: https://github.com/sonata-project/SonataDoctrineORMAdminBundle/blob/4.x/src/Filter/AbstractDateFilter.php#L108 which sets the field_type there: https://github.com/sonata-project/SonataAdminBundle/blob/4.x/src/Form/Type/Filter/DateTimeRangeType.php#L41 should in these classes also a "picker" parameter added?

the problem i currently have is this: https://github.com/sonata-project/SonataAdminBundle/blob/4.x/src/Filter/Filter.php#L176 i wanted to make ```php public function getDefaultOptions(): array { return ['field_type' => DateTimeRangeType::class]; } ``` turn into this: ```php public function...

i added an MR for SonataAdminBundle to add picker as option to the Form Filter

When i remove the Default Options, then the problem is this part: https://github.com/sonata-project/SonataDoctrineORMAdminBundle/blob/9ccf050648625de7055a28251f04ab487afe3fea/src/Filter/AbstractDateFilter.php#L103-L120 with setting the `field_type` from the options. So the users can overwrite it. i'm currently strugging to...

> This should be easier now #1669 is merged if you wanna try again @Hanmac the only thing that makes it a bit tricky would be that now the default...

i think this is by design in MRI: https://www.rubyguides.com/2016/02/ruby-procs-and-lambdas/ > A lambda will return normally, like a regular method. > But a proc will try to return from the current...

really? they discuss that problem on Stackexchange a lot? i hope it isn't becaue of me :) That's why i asked at this place what the best way would be:...

@npotier this user did get the hook working https://github.com/northernco/NorthernDatagridBundle/commit/925e9af5b018965c31bf6c1a30ba34f014246aeb

This change seems to be easier than i thought, this user made that change already in their fork: https://github.com/it-studio/APYDataGridBundle/commit/4c43fe260bd3c335ba8786a56d81c501813ed8b9