Skip to content

Conversation

@makraz
Copy link
Contributor

@makraz makraz commented Dec 15, 2025

  • Implement association-aware JOIN creation for filters when using dotted relationship.property paths.
  • Update Boolean, Null, Choice, Comparison, Array, and Entity filters to apply conditions on joined aliases while preserving behavior for non-association properties.
  • Ensure collection-valued associations and NULL/NOT NULL comparisons handle joins consistently across filters.

Demo:

Screen.Recording.2025-12-15.at.01.15.23.mov

@makraz makraz force-pushed the feature/association-filter branch 2 times, most recently from 0c906d6 to ab21802 Compare December 15, 2025 00:50
@makraz makraz force-pushed the feature/association-filter branch from ab21802 to 3b1ecad Compare December 15, 2025 01:18
@KDederichs
Copy link
Contributor

Oh just saw this... do you know if that fixes #7298 by chance?
From looking at the code I'd say it probably has the same issue with embeds still but not sure

@makraz
Copy link
Contributor Author

makraz commented Dec 16, 2025

Oh just saw this... do you know if that fixes #7298 by chance? From looking at the code I'd say it probably has the same issue with embeds still but not sure

I think yes, I create a demo here: https://github.com/makraz/easyadmin-association-filter

image

@KDederichs
Copy link
Contributor

ah I looked at the example, you'd need to configure the book to search on the author's address to test that bug.

Like ->setSearchFields(['author.address.street']); in the book controller

@makraz
Copy link
Contributor Author

makraz commented Dec 17, 2025

ah I looked at the example, you'd need to configure the book to search on the author's address to test that bug.

Like ->setSearchFields(['author.address.street']); in the book controller

i created a new PR #7300 to add the embeddable properties support for associations

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants