feat: support relations on filters #7711
Conversation
Maxcastel
commented
Jan 28, 2026
Maxcastel
commented
| Q | A |
|---|---|
| Branch? | main |
| Tickets | Closes #..., closes #... |
| License | MIT |
| Doc PR | api-platform/docs#... |
Maxcastel
force-pushed
the
feature/support-relations-on-filters
branch
from
056fa57 to
fffaa15
Compare
| ); | ||
| } | ||
| $alias = $currentAlias; | ||
| } |
There was a problem hiding this comment.
could you move this to a trait? we're using this everywhere
There was a problem hiding this comment.
Will do a refactor commit
|
|
||
| foreach ($propertyNames as $property) { | ||
| $converted = $this->nameConverter?->denormalize($property) ?? $property; | ||
| if (str_contains($property, '.')) { |
There was a problem hiding this comment.
This shouldn't happen right? if you've transformed all the . properties I think we should remove line 213-215 unless I'm missing something?
There was a problem hiding this comment.
Tested on a real project, and it doesn’t work when I remove lines 213–215.
There was a problem hiding this comment.
Exemple for: search[products.name]
If I remove lines 213–215, products.name becomes products_Name after denormalization ($converted = $this->nameConverter?->denormalize($property)).
There was a problem hiding this comment.
All tests pass after removing lines 213-215, so maybe there’s a missing test that covers this case.
This reverts commit 1b5a00e.
Maxcastel
force-pushed
the
feature/support-relations-on-filters
branch
from
2a6baed to
490a9af
Compare
3 participants
