Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✨ Allow join on ExpressionBuilder comparison #312

Closed
wants to merge 1 commit into from

Conversation

FabienCharlier
Copy link

@FabienCharlier FabienCharlier commented Oct 13, 2023

Allowing for the compare method of ExpressionBuilder to join on different table before comparing.

It is currently not possible to compare two fields which are in a joined table.

@lchrusciel
Copy link
Member

Hey Fabien!

Thanks a lot for your contribution, but without a functional test, we won't merge this PR. Could you add it?

@FabienCharlier
Copy link
Author

FabienCharlier commented Oct 20, 2023

I'd like to, but I'm really not sure how to do it. There isn't a lot of examples in the Functional test folder, and nothing looks really close to what I'm trying to do.
To setup a functionnal test, I'd ideally want to have a database with two tables (one referencing another with a foreign key), a grid setup on one of those tables. This grid would need to have a custom filter, which tries to compare two fields from the joined table. Then, requiring the correct url should do the trick.

Do you have any documentation or similar test cases that I could read to help me create a test case ?

@FabienCharlier FabienCharlier closed this by deleting the head repository Nov 14, 2024
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.

2 participants