Merge pull request #1571 from JoseAntonioRodriguez/fix-console-connector

Fix loop indexes are being meshed up in the ConsoleConnector

Author: Stevenic

Node/core/src/bots/ConsoleConnector.ts

@@ -40,7 +40,7 @@ export class ConsoleConnector implements ub.IConnector {
     private handler: (events: IEvent[], cb?: (err: Error) => void) => void;
     private rl: readline.ReadLine;
     private replyCnt = 0;
-    
+
     public listen(): this {
         this.rl = readline.createInterface({ input: process.stdin, output: process.stdout, terminal: false });
         this.rl.on('line', (line: string) => {
@@ -72,11 +72,11 @@ export class ConsoleConnector implements ub.IConnector {
         }
         return this;
     }
-    
+
     public onEvent(handler: (events: IEvent[], cb?: (err: Error) => void) => void): void {
         this.handler = handler;
     }
-    
+
     public send(messages: IMessage[], done: (err: Error) => void): void {
         for (var i = 0; i < messages.length; i++ ){
             if (this.replyCnt++ > 0) {
@@ -87,14 +87,14 @@ export class ConsoleConnector implements ub.IConnector {
                 log(msg.text);
             }
             if (msg.attachments && msg.attachments.length > 0) {
-                for (var i = 0; i < msg.attachments.length; i++) {
-                    if (i > 0) {
+                for (var j = 0; j < msg.attachments.length; j++) {
+                    if (j > 0) {
                         console.log();
                     }
-                    renderAttachment(msg.attachments[i]);
+                    renderAttachment(msg.attachments[j]);
                 }
             }
-        }        
+        }
 
         done(null);
     }