Fix connect() related issues #270

Merged
merged 5 commits into from Apr 19, 2013

Projects

None yet

2 participants

@raymondfeng
Contributor

The patch fixes two issues:

  1. Set the connecting flag to be false in the schema constructor
  2. Register the connected handler before the connect() is called.
@1602 1602 commented on the diff Apr 16, 2013
lib/schema.js
@@ -66,7 +66,7 @@ function Schema(name, settings) {
// Disconnected by default
this.connected = false;
- this.connecting = true;
+ this.connecting = false;
1602
1602 Apr 16, 2013 Owner

This fix breaks everything :) Have you tried to run tests?

@1602 1602 merged commit 9a7d3f1 into 1602:master Apr 19, 2013

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment