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
415 stars 27 forks source link

add narrower exception catching in _ordered_by_integer #53

Closed taobojlen closed 1 year ago

taobojlen commented 1 year ago

this is just a suggestion, feel free to ignore this!

it took me a second to understand the purpose of this try/except block; with this stricter error handling it's a bit clearer i think.

matthiask commented 1 year ago

This makes a lot of sense, thanks!