update() breaks when sequelize configured with raw:true

Affects master, v1.4.3, etc.

A call to update() results in a Model.findById call: https://github.com/feathersjs/feathers-sequelize/blob/master/src/index.js#L168

If sequelize is configured with raw: false (the default), this returns a DAO, with a toJSON method, which is then called several lines later: https://github.com/feathersjs/feathers-sequelize/blob/master/src/index.js#L174

However, if sequelize is configured with raw: true, the findById call returns a JSON object without this method.

There are two simple fixes:

1. Only call toJSON if it exists and is a function
2. pass {raw: false} as a second argument to the findById call to force the expected behaviour

[daffl]

Makes sense. This can probably easily be fixed by changing the line you pointed out to

Object.keys(typeof instance.toJSON === 'function' ? instance.toJSON() : instance).forEach(key => {

[daffl]

Ah, I see. Well at least we know it didn’t break anything else 😉

I think we should do what patch does and use this.Model.update(omit(data, this.id), options) instead of calling save on the model instance.