Support for destroy() to DELETE ALL entities of the given model

Currently (v. 0.9.2), the documentation on model.destroy() says:

destroy performs a delete on the model, using the model’s idAttribute to constrain the query.

Without a constraining idAttribute, destroy() fails with the error message

A model cannot be destroyed without a "where" clause or an idAttribute.

However, it would be desirable to be able to actually do a

DELETE FROM MY_MODEL

…deleting all entities of the given model.

Yes, it’s a dangerous operation, but a useful one for cleanup / prototyping purposes. If you really want to keep an additional security net in the API, please consider adding a flag to destroy() to override the “idAttribute constraint” such that we can purposefully do a “delete all”.

[codebulb]

OK, so I found a workaround. The obvious solution is to use a where() clause that is always true:

myModel.where('id', '!=', '0').destroy().then(...)

This settles my interest in this issue. Feel free to close it.

[gambhirpulkit]

Alternatively, please suggest how to implement this with the current API.

I tried it with

const destroyed = []
myModel.fetchAll().then(function (collection) {
  collection.each(function(it) {
    destroyed.push(it.destroy())
  })
})
Promise.all(destroyed).then(function (entity) {
  // ok
}).catch(function (error) {
  // failed
})

but the problem is that if one of the it.destroy() fails (e.g. because of a foreign key violation), the entire thing fails with

Unhandled rejection Error: ER_ROW_IS_REFERENCED_2: Cannot delete or update a parent row: a foreign key constraint fails...

(Note: Maybe I didn’t exactly understand how to work with promises in this case.)

Use this - https://github.com/seegno/bookshelf-cascade-delete