On Thu, 2013-02-07 at 13:31 +0100, Michal Fojtik wrote: > On 02/06, lut...@redhat.com wrote: > > I really like the simplification you did. Altrough I will need to look > deeper to understand how exactly the code works, I like it :-)
The one thing I don't like too much about what I did is that I'd like to load the Entity as soon as a model object is created, but that's not possible - we need to have an id before we can associate with a DB Entity. I therefore try to load the entity from initialize, and whenever the :id attribute is set. It's a little ugly in that you can clobber changes if you set the id after you made changes to the model object. I couldn't think of a way to avoid that, and the current state of affairs is a little less error-prone than what we had before. David
