Simplifying `associate_existing_records` to use association's `id_reader`
Merge remote-tracking branch 'origin/master' into remove_through_associations
use association.ids_reader to get the current ids