Changed that usage to self. connection.changeUser now sets inTransaction to false.

Author: balihoo-jflitton

lib/Connection.js

@@ -74,6 +74,8 @@ Connection.prototype.connect = function(cb) {
 };
 
 Connection.prototype.changeUser = function(options, cb){
+  cb = cb || function() {};
+
   this._implyConnect();
 
   if (typeof options === 'function') {
@@ -85,13 +87,18 @@ Connection.prototype.changeUser = function(options, cb){
     ? Config.getCharsetNumber(options.charset)
     : this.config.charsetNumber;
 
+  var self = this;
+
   return this._protocol.changeUser({
     user          : options.user || this.config.user,
     password      : options.password || this.config.password,
     database      : options.database || this.config.database,
     charsetNumber : charsetNumber,
     currentConfig : this.config
-  }, cb);
+  }, function(err) {
+    self.inTransaction = false;
+    return cb(err);
+  });
 };
 
 Connection.prototype.beginTransaction = function(cb) {
@@ -101,9 +108,9 @@ Connection.prototype.beginTransaction = function(cb) {
 
   this._implyConnect();
 
-  var that = this;
+  var self = this;
   var query = Connection.createQuery('BEGIN', function(err) {
-    that.inTransaction = true;
+    self.inTransaction = true;
     return cb(err);
   });
 
@@ -118,9 +125,9 @@ Connection.prototype.commit = function(cb) {
 
   this._implyConnect();
 
-  var that = this;
+  var self = this;
   var query = Connection.createQuery('COMMIT', function(err) {
-    that.inTransaction = false;
+    self.inTransaction = false;
     return cb(err);
   });
 
@@ -135,9 +142,9 @@ Connection.prototype.rollback = function(cb) {
 
   this._implyConnect();
 
-  var that = this;
+  var self = this;
   var query = Connection.createQuery('ROLLBACK', function(err) {
-    that.inTransaction = false;
+    self.inTransaction = false;
     return cb(err);
   });