ObjectID.isValid should check that `typeof toString === 'function'` otherwise if it's not an object and doesn't have a `toString` then it would return false

[cabinjs]

ObjectID.isValid should check that `typeof toString === 'function'` otherwise if it's not an object and doesn't have a `toString` then it would return false

[niftylettuce]

e.g. if you pass an empty plain object to ObjectID.isValid it throws an uncaught exception.

[niftylettuce]

Specifically https://github.com/williamkapke/bson-objectid/blob/master/objectid.js#L98 should change to if (!objectid || (typeof objectid !== 'string' && (typeof objectid !== 'object' || typeof objectid.toString !== 'function')))

[williamkapke]

seems right. I'm happy to take a PR for it!

[niftylettuce]

roger, PR inbound

[niftylettuce]

@williamkapke see #28, I also added a test case as well.

let me know of any issues, or when a new version is published to npm.

thanks many! 👍