rails/rails — pull request #6509 by frodsan (2012-05-27 23:04:05+00:00)
Title: Remove support for CollectionAssociation#destroy by Fixnum or String

I found the next issue between CollectionAssociation delete and destroy.

class Person < ActiveRecord::Base
has_many :pets
end

person.pets.destroy(1)
# => OK, returns the destroyed object

person.pets.destroy("2")
# => OK, returns the destroyed object

person.pets.delete(1)
# => ActiveRecord::AssociationTypeMismatch

person.pets.delete("2")
# => ActiveRecord::AssociationTypeMismatch

I was not sure about adding support for delete by fixnum or string.
That's why i prefer to remove destroy by fixnum or string option.
I sincerely think that is better to delete or destroy passing a record
or an object.