CopyOnAttach=false for collections likely doesn't handle element removal - check it

[andyjefferson]

The handling for CopyOnAttach=false is likely problematic and needs more testing. Code in AttachFieldManager only seems to handle added elements in collections. If an element is deleted there seems to be no check with current datastore contents, and any subsequent datastore removal. Needs more unit tests