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

[WIP] Fix for HasManyThrough #840

Closed

Conversation

tychokamphuis-aqqo
Copy link

@tychokamphuis-aqqo tychokamphuis-aqqo commented Aug 27, 2024

Edit: On second hand; This is not the currect solution. However, filtering on expanded HasManyThrough is still not working.

HasManyThrough relations were not filterable. To recreate this issue you can use the following database from the Laravel documentation:

projects
    id - integer
    name - string
 
environments
    id - integer
    project_id - integer
    name - string
 
deployments
    id - integer
    environment_id - integer
    commit_hash - string

When recreated the above, the following query returns all projects instead of a any with a commit hash.

http://127.0.0.1:8000/odata/Projects?$expand=Deployments($filter=commit_hash eq 'XXX')&$filter=Deployments/any(s:s/commit_hash eq 'XXX')

@tychokamphuis-aqqo tychokamphuis-aqqo changed the title Fix for HasManyThrough [WIP] Fix for HasManyThrough Aug 27, 2024
@tychokamphuis-aqqo tychokamphuis-aqqo deleted the 5.x-hasmanythrough branch August 27, 2024 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant