feincms / django-tree-queries

Adjacency-list trees for Django using recursive common table expressions. Supports PostgreSQL, sqlite, MySQL and MariaDB.
https://django-tree-queries.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
427 stars 27 forks source link

Handle tree queryset .values() even more correctly #59

Closed glennmatthews closed 11 months ago

glennmatthews commented 11 months ago

Follow-on to #57, sorry for the trouble!

This changes the logic so that we still preserve tree traversal order when using .values() or .values_list() on it, while not reintroducing the issue that #57 was addressing.

matthiask commented 11 months ago

Thanks! This looks great.

I have released this as 0.16.1 (I think the patch qualifies as a bug fix and not as a change)