Use Model.findById() rather than get()

[evanbarter]

Calling this.get() from within remove() will not populate params.user object, and if you have a before get hook validating the presence of a user object (i.e. feathers-authentication.hooks.requireAuth) it will throw an error.

This is a bit of a weird situation because of some inconsistencies in the API. update() takes an id and Model.findById() is used so I guess the assumption is the author will validate access, permissions etc in a before update hook. patch() takes an id or params but ultimately does a find and passes the id in to the params.where object, if present.

This PR follows the update() method of retrieving a single instance for remove().

[daffl]

Thank you. You are right, that behaviour is indeed inconsistent. I also noticed that passing on to find will cause issues when pagination is enabled so I'll do another review before getting this change out in a patch release.

[daffl]

Thanks again. I made a 1.0.8 patch release with your fix and created a follow-up issue to review the consistency of that part of the API.