Implicit Support for SQLAlchemy's Query-API has been removed about odata-query

[OliverHofkens]

My bad, the test case was incomplete. Hopefully 0.7.2 fixes it for real! 😅

[OliverHofkens]

This was definitely not intended, so thanks for reporting this and for the simple reproducing code!
I think it'd be relatively easy to add a test on this behavior, fix the regression, and add it to the documentation/type hint while we're at it. I'll have a look soon!

[OliverHofkens]

I just released 0.7.1 which should fix this. Feel free to try it out!

[tstadel]

I guess something like

    clause_element: ClauseElement = query
    if isinstance(query, Query):
        # For now, we keep supporting the 1.x style of queries unofficially.
        # GITHUB-34
        clause_element = query.__clause_element__()

    model = clause_element.columns_clause_froms[0].entity_namespace

should resolve this. But I'm not an expert on SQLAlchemy internals.

[tstadel]

I just released 0.7.1 which should fix this. Feel free to try it out!

@OliverHofkens I tried it out, but unfortunately if I run the example code from above I get the following error when running query.all():

AttributeError: 'Select' object has no attribute 'all'

I guess that's due to returning a Select and not as previously a Query object.