silverstripe/silverstripe-framework — pull request #2225 by tractorcow (2013-07-12 03:44:00+00:00)
Title: BUG Fixes issue with HasManyList::remove

HasManyList::remove would reset the foreign key on the removed object without bothering to check if that object was even in the list to remove.

This PR will force the list to check against the current getForeignID() value before removal.