-
Notifications
You must be signed in to change notification settings - Fork 23
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
Support conditions that use nested relations #32
Comments
The functionality to support relations is implemented through 2 options:
I tested this functionality only for direct relations (and not for relations of relations - i.e., deeper relations). After that, I just haven't had to time look again at this functionality. As currently, this is not on my priority list. And very likely, I will look into it only in the next year |
Hi, sorry for the late reply. Thank you! |
I've made some changes to the typeorm adapter and the Query class in order to add this functionality. Unfortunately I have no experience with the other ORMs and I can't be sure that my changes don't affect the other adapters. Here's my changes if someone need it as a starting point for a patch: |
Add support for fields with nested relations (multiple levels) in the typeORM adapter. The other adapters treat these fields as simple fields and they don't join the relations. Fix issue stalniy#32 for TypeORM adapter.
Add support for fields with nested relations (multiple levels) in the typeORM adapter. The other adapters treat these fields as simple fields and they don't join the relations. Fix issue stalniy#32 for TypeORM adapter.
Add support for fields with nested relations (multiple levels) in the typeORM adapter. The other adapters treat these fields as simple fields and they don't join the relations. Fix issue stalniy#32 for TypeORM adapter.
It seems that sql interpreter doesn't support conditions with nested relations.
Indeed the condition
o { operator: 'eq', value: 2, field: 'x.y.z' }
is translated intoWHERE 'x'.'y.z' = :1
.Are you planning to support this type of conditions?
The text was updated successfully, but these errors were encountered: