connect() is asynchronous, API (esp. isConnected()) encourages synchronous interface

[fatuhoku]

Node-DBI users are encouraged to write code like the following (in Coffeescript):

db = new DBWrapper 'sqlite3', { ... }
db.connect()
console.error 'Failed to connect to database' if not db.isConnected()

The two solutions to this issue are:

• use async, and document this in README.md:

db = new DBWrapper 'sqlite3', { ... }
async.series [
   -> db.connect()
   -> console.error 'Failed to connect to database' if not db.isConnected()
]

• provide a callback in the connect(): i.e. change the interface to connect(callback)

[olivierphi]

@fatuhoku Hello !

Thank you for this report. Currently there is a "connection" Event emitted by the DBWrapper instance when the DB connection is resolved, with a "err" param. But you're right, it may be more "Nodish" to be able to provide a callback on the "connect()" method. I'm going to try to add this feature...