{"push_id":690149583,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"81e7a200f1d480f4c00a8d0bedbdd9d902eb7a89","before":"f2362ac0c1c0e3f84eb6585365e6efa4f02bbf4d","commits":[{"sha":"81e7a200f1d480f4c00a8d0bedbdd9d902eb7a89","author":{"email":"5c68d51cdb9ba7935b8f4d79d30eef263145b626@gmail.com","name":"Luca Wehrstedt"},"message":"Fix SmartMappedCollection\n\nFixes a bug introduced by fa5e98cda752df039abc38cb9fe783e71d6eb0f5 which\ncaused collections loaded from the DB to be empty (even if the tests\npassed). The cause was setting the parent of the child to None inside\n__setitem__. The decision to do it came from a wrong understanding of\nthe dynamics of SQLAlchemy and of the situations that method could be\ncalled in. The new code (that doesn't assume anything about the state\nthe child is in) should be more correct.\n\nI couldn't find a way to trigger the bug in a testing environment to\nmake sure to catch it in case it should pop up again.","distinct":true,"url":"https://api.github.com/repos/cms-dev/cms/commits/81e7a200f1d480f4c00a8d0bedbdd9d902eb7a89"}]}