fix: Deduplicate accounts.

leggen/commands/balances.py

@@ -13,9 +13,9 @@ def balances(ctx: click.Context):
     """
 
     res = get(ctx, "/requisitions/")
-    accounts = []
+    accounts = set()
     for r in res.get("results", []):
-        accounts += r.get("accounts", [])
+        accounts.update(r.get("accounts", []))
 
     all_balances = []
     for account in accounts:

leggen/commands/status.py

@@ -15,7 +15,7 @@ def status(ctx: click.Context):
 
     res = get(ctx, "/requisitions/")
     requisitions = []
-    accounts = []
+    accounts = set()
     for r in res["results"]:
         requisitions.append(
             {
@@ -24,7 +24,7 @@ def status(ctx: click.Context):
                 "Created at": datefmt(r["created"]),
             }
         )
-        accounts += r.get("accounts", [])
+        accounts.update(r.get("accounts", []))
     info("Banks")
     print_table(requisitions)
 

leggen/commands/sync.py

@@ -104,10 +104,9 @@ def sync(ctx: click.Context):
     """
     info("Getting accounts details")
     res = get(ctx, "/requisitions/")
-    accounts = []
+    accounts = set()
     for r in res.get("results", []):
-        accounts += r.get("accounts", [])
-    accounts = list(set(accounts))
+        accounts.update(r.get("accounts", []))
 
     info(f"Syncing transactions for {len(accounts)} accounts")
 

leggen/commands/transactions.py

@@ -41,9 +41,9 @@ def transactions(ctx: click.Context, account: str):
         print_transactions(ctx, account_info, account_transactions)
     else:
         res = get(ctx, "/requisitions/")
-        accounts = []
+        accounts = set()
         for r in res["results"]:
-            accounts += r.get("accounts", [])
+            accounts.update(r.get("accounts", []))
         for account in accounts:
             account_details = get(ctx, f"/accounts/{account}")
             account_transactions = get(ctx, f"/accounts/{account}/transactions/").get(