From d9981e1d7116659bec577a37281b2671f787aa08 Mon Sep 17 00:00:00 2001
From: Kyle Spearrin <kyle.spearrin@gmail.com>
Date: Sat, 9 Sep 2017 12:25:35 -0400
Subject: [PATCH] cleaned providers should be an obj, not array

---
 src/app/accounts/accountsLoginController.js | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/app/accounts/accountsLoginController.js b/src/app/accounts/accountsLoginController.js
index 179cd14b..9055b5ac 100644
--- a/src/app/accounts/accountsLoginController.js
+++ b/src/app/accounts/accountsLoginController.js
@@ -117,18 +117,18 @@ angular
             }
 
             var keys = Object.keys(twoFactorProviders);
-            var cleanedProviders = [];
             for (var i = 0; i < keys.length; i++) {
                 var provider = $filter('filter')(constants.twoFactorProviderInfo, {
                     type: keys[i],
                     active: true,
                     requiresUsb: false
                 });
-                if (provider.length) {
-                    cleanedProviders.push(twoFactorProviders[keys[i]]);
+                if (!provider.length) {
+                    delete twoFactorProviders[keys[i]];
                 }
             }
-            return cleanedProviders;
+
+            return twoFactorProviders;
         }
 
         // ref: https://stackoverflow.com/questions/11381673/detecting-a-mobile-browser