dymmond / edgy

🔥 The perfect ORM to work with complex databases 🔥
https://edgy.dymmond.com
BSD 3-Clause "New" or "Revised" License
193 stars 9 forks source link

double initialization of metaclasses / fields contain references to old fields #86

Closed devkral closed 5 months ago

devkral commented 5 months ago

Have a look at the fixmes in: edgy/core/db/models/metaclasses.py of my PR feature/composite_field and run hatch run test:test tests/fields to see the crashes when the workaround is removed

The fields are carried over when dropping models (or maybe the fields should be remembered and only the class partly rewired). Somehow even the field attribute is remembered. This is not good.

tarsil commented 5 months ago

@devkral not sure if I understand. This happens in your PR or is something already happening?

devkral commented 5 months ago

fixed