From d5bb0cce5de8a76c507a8ad26046f51ee8cdaa69 Mon Sep 17 00:00:00 2001
From: Athanasius <github@miggy.org>
Date: Fri, 4 Jun 2021 10:21:56 +0100
Subject: [PATCH] EDSM: credentials() should just return `None` if no apikey

---
 plugins/edsm.py | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/plugins/edsm.py b/plugins/edsm.py
index 32400c30..9519d83f 100644
--- a/plugins/edsm.py
+++ b/plugins/edsm.py
@@ -333,12 +333,9 @@ def credentials(cmdr: str) -> Optional[Tuple[str, str]]:
             and (edsm_apikeys := config.get_list('edsm_apikeys'))):
         idx = cmdrs.index(cmdr)
         # The EDSM cmdr and apikey might not exist yet!
-        if idx >= len(edsm_usernames):
+        if idx >= len(edsm_usernames) or idx >= len(edsm_apikeys):
             return None
 
-        if idx >= len(edsm_apikeys):
-            edsm_apikeys[idx] = None
-
         return (edsm_usernames[idx], edsm_apikeys[idx])
 
     else: