refactor and clean up delete_all method

Now that `delete_all` with a `destroy` and `delete_all` dependency behave the same we no longer need this conditional. This means we can remove the `delete_all_with_dependency` method I added and go straight to delete_records from delete_all, passing `:all` and the `dependent` directly.

cc/ @tenderlove