mde/geddy — pull request #512 by ben-ng (2013-12-13 01:41:27+00:00)
Title: Fixes an issue where removing with a query object would cause an error

Thanks @zerocle for reporting this

Fixes this issue: https://github.com/zerocle/GeddyMultiDeleteError

Not yet merging this one in because I'm not sure if creating a copy of the argument is the best way to fix this.

Since model sends the object to the event handler, so there's the chance that the user (in this case geddy) changes that object and screws up what model expected.

Also, I feel like it should be model's job to perform the copy before passing it as an argument to emit.

Thoughts @mde?