From 340cff082a5f6ad86dce3685d618ed1250b5f368 Mon Sep 17 00:00:00 2001
From: Arie Peterson <arie@greenhost.nl>
Date: Tue, 21 Jan 2025 15:20:20 +0100
Subject: [PATCH] Fix show_user in cli

---
 backend/cliapp/cliapp/cli.py | 35 ++++++++++++++++++-----------------
 1 file changed, 18 insertions(+), 17 deletions(-)

diff --git a/backend/cliapp/cliapp/cli.py b/backend/cliapp/cliapp/cli.py
index c1cf93d8..e82bd87f 100644
--- a/backend/cliapp/cliapp/cli.py
+++ b/backend/cliapp/cliapp/cli.py
@@ -272,24 +272,25 @@ def show_user(email):
     :param email: Email address of the user to show
     """
     user = KratosUser.find_by_email(kratos_identity_api, email)
-    if user is not None:
-        print(user)
-        print("")
-        print(f"UUID:     {user.uuid}")
-        print(f"Username: {user.username}")
-        print(f"Updated:  {user.updated_at}")
-        print(f"Created:  {user.created_at}")
-        print(f"State:    {user.state}")
-        print(f"Roles:")
-        results = db.session.query(AppRole, Role).join(App, Role)\
-            .add_entity(App).add_entity(Role)\
-            .filter(AppRole.user_id == user.uuid)
-        for entry in results:
-            app = entry[-2]
-            role = entry[-1]
-            print(f"  {role.name: >9} on {app.name}")
-    else:
+    if user is None:
         print(f"User with email address '{email}' was not found")
+        return
+    print(user)
+    print("")
+    print(f"UUID:     {user.uuid}")
+    print(f"Username: {user.username}")
+    print(f"Updated:  {user.updated_at}")
+    print(f"Created:  {user.created_at}")
+    print(f"State:    {user.state}")
+    print(f"Roles:")
+    results = db.session.query(AppRole)\
+        .filter_by(user_id=user.uuid)\
+        .join(App).join(Role)\
+        .add_entity(App).add_entity(Role)
+    for entry in results:
+        app = entry[-2]
+        role = entry[-1]
+        print(f"  {role.name: >9} on {app.name}")
 
 
 @user_cli.command("update")
-- 
GitLab