dresende / node-orm2

Object Relational Mapping
http://github.com/dresende/node-orm2
MIT License
3.07k stars 379 forks source link

fixed thrown error check #593

Closed keblodev closed 9 years ago

keblodev commented 9 years ago

Because if there's no 'find module' in ex.message ,the -1 value from indexOf is equal to true and you'll never know what hit you.

Right now basically on any error thrown in this try {} catch - you'll get "Connection protocol not supported" , which is wrong

Please review.

dxg commented 9 years ago

Good catch. Could you also add a test to orm-exports.js?

keblodev commented 9 years ago

yes - sure :)

keblodev commented 9 years ago

@dxg , done. Please take a look.

dxg commented 9 years ago

I wish there were 48 hours in the day... sorry for the delay.

dxg commented 9 years ago

Published as part of v2.1.23