improve code readability

Author: JT

bugbot/rules/not_landed.py

@@ -263,7 +263,8 @@ def get_bz_userid(self, phids):
             return {}
 
         def handler(user, data):
-            data[str(user["id"])] = (user["name"], user["nick"])
+            # data[str(user["id"])] = (user["name"], user["nick"])
+            data[str(user["id"])] = user
 
         data = {}
         BugzillaUser(
@@ -311,13 +312,13 @@ def get_bugs(self, date="today", bug_ids=[]):
         res = {}
 
         reviewers_phid = set()
-        nicknames = {}
+        bug_assignee_map = {}
         for bugid, data in bugs_patch.items():
             reviewers_phid |= data["reviewers_phid"]
             assignee = bugs[bugid]["assigned_to"]
             if not assignee:
                 assignee = max(data["author"], key=data["author"].get)
-                nicknames[bugid] = assignee
+                bug_assignee_map[bugid] = assignee
 
         bz_reviewers = self.get_bz_userid(reviewers_phid)
         all_reviewers = set(bz_reviewers.keys())
@@ -329,7 +330,8 @@ def get_bugs(self, date="today", bug_ids=[]):
             nickname = d["nickname"]
 
             if not assignee:
-                assignee, nickname = bz_reviewers[nicknames[bugid]]
+                assignee_id = bug_assignee_map[bugid]
+                assignee, nickname = bz_reviewers[assignee_id]
 
             if not assignee:
                 continue