fix: consistency, used this instead of that (#1618)

Author: robertsLando

lib/client.js

@@ -330,7 +330,7 @@ function MqttClient (streamBuilder, options) {
 
   // Send queued packets
   this.on('connect', function () {
-    const queue = this.queue
+    const queue = that.queue
 
     function deliver () {
       const entry = queue.shift()
@@ -372,23 +372,23 @@ function MqttClient (streamBuilder, options) {
 
   this.on('close', function () {
     debug('close :: connected set to `false`')
-    this.connected = false
+    that.connected = false
 
     debug('close :: clearing connackTimer')
-    clearTimeout(this.connackTimer)
+    clearTimeout(that.connackTimer)
 
     debug('close :: clearing ping timer')
     if (that.pingTimer !== null) {
       that.pingTimer.clear()
       that.pingTimer = null
     }
 
-    if (this.topicAliasRecv) {
-      this.topicAliasRecv.clear()
+    if (that.topicAliasRecv) {
+      that.topicAliasRecv.clear()
     }
 
     debug('close :: calling _setupReconnect')
-    this._setupReconnect()
+    that._setupReconnect()
   })
   EventEmitter.call(this)