Interesting. So we get unecessary sql calls every time a non-relation class attribute is removed (not often on most sites). It should be a very fast query, though. Might not be a problem? Alternatively we would need some sort of attribute type whitelist, or testing code. Not sure if worth it.

Otherwise, +1