Yes, then we should convert all the calls to destroy. I only see two
instances of ActiveRecord::Base.delete:

```
lib/sufia/files_controller_behavior.rb:71:      @generic_file.delete
app/controllers/batch_edits_controller.rb:59:          gf.delete
```

-Justin



On Tue, Jan 15, 2013 at 11:49 AM, cam156 <notifications@github.com> wrote:

> In lib/sufia/generic_file.rb:
>
> > @@ -37,6 +37,12 @@ module GenericFile
> >        around_save :characterize_if_changed, :retry_warming
> >      end
> >
> > +    def delete
>
> We are using delete all over the code base. Wouldn't we need to convert
> those calls all to destory before the before_destroy hook would get called?
> My impression was that a call to delete bypassed the hook.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/curationexperts/sufia/pull/41/files#r2653029>.
>
>