iommirocks / iommi

Your first pick for a django power cord
http://iommi.rocks
BSD 3-Clause "New" or "Revised" License
688 stars 47 forks source link

[iommi 6.4] no longer accepts non-integer Primary Keys #493

Closed jedrus2000 closed 5 months ago

jedrus2000 commented 8 months ago

My case is that I got data from JSONs, from third party service, and blindly accepted data ID of UUID4 into tables Primary Key type.

Error:

image

If it is kind of trade off, and won't fix issue then I suggest making error message more informative:

f'Could not interpret {value_string_or_filter_name} as an integer. Only integer type Primary Key is supported.'

Code point making error message:

https://github.com/iommirocks/iommi/blob/master/iommi/query.py#L992

jlubcke commented 5 months ago

Duplicate of #523