roleable icon indicating copy to clipboard operation
roleable copied to clipboard

remove_role bypasses rails callbacks

Open EvanWillms opened this issue 13 years ago • 1 comments

Roleable.subject.remove_role bypasses the rails callbacks by using delete_all instead of destroy_all. This is causing a problem with our model lifecycle auditing callbacks, as the removal of a role does not trigger appropriate tracking.

EvanWillms avatar Aug 13 '12 22:08 EvanWillms

Yeah, a destoy_all definitely makes more sense there. Pull request are very welcome ;-P ! Else, I'll keep this issue open and get to this when I'm able.

mcrowe avatar Aug 14 '12 00:08 mcrowe