From ac5c35905348e3310abcadc7ad02ba09037aaa2a Mon Sep 17 00:00:00 2001 From: lilia Date: Fri, 17 Oct 2014 16:57:22 -0700 Subject: [PATCH] Fix strange loop in sendMessageProto Don't declare a new `var i` within the scope of an existing `var i`. Fixes #63 --- js/sendmessage.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/js/sendmessage.js b/js/sendmessage.js index 46a0672a9d..73daf9135f 100644 --- a/js/sendmessage.js +++ b/js/sendmessage.js @@ -175,9 +175,9 @@ window.textsecure.messaging = function() { var devicesForNumber = textsecure.storage.devices.getDeviceObjectsForNumber(number); var promises = []; - for (var i in devicesForNumber) - if (devicesForNumber[i].registrationId === undefined) - promises[promises.length] = getKeysForNumber(number, [parseInt(textsecure.utils.unencodeNumber(devicesForNumber[i].encodedNumber)[1])]); + for (var j in devicesForNumber) + if (devicesForNumber[j].registrationId === undefined) + promises[promises.length] = getKeysForNumber(number, [parseInt(textsecure.utils.unencodeNumber(devicesForNumber[j].encodedNumber)[1])]); Promise.all(promises).then(function() { devicesForNumber = textsecure.storage.devices.getDeviceObjectsForNumber(number);