spotorm / spot2

Spot v2.x DataMapper built on top of Doctrine's Database Abstraction Layer
http://phpdatamapper.com
BSD 3-Clause "New" or "Revised" License
601 stars 101 forks source link

Fix hasMany relations saving #236

Closed kitchenu closed 7 years ago

tuupola commented 7 years ago

What is the bug this fixes?

nebulousGirl commented 7 years ago

Seems to fix a bug with saving hasMany relations that were deleted. Needs a test case though to ensure that the bug does not reappear in the future.

vlucas commented 7 years ago

@kitchenu Can you please update this PR with a test to demonstrate the new working behavior? We'd love to get this merged in.

kitchenu commented 7 years ago

I fixed testHasManyNewEntitySaveRelation() for #236