Update routes/logs.py

routes/logs.py

@@ -129,28 +129,40 @@ async def get_logs(
         def generate_message(log, user_info):
             action = log.get("action")
             reference = log.get("reference")
-            old_data = log.get("old_data")
-            new_data = log.get("new_data")
-
+            old_data = log.get("old_data") or {}
+            new_data = log.get("new_data") or {}
+        
+            if isinstance(old_data, str):
+                try:
+                    old_data = json.loads(old_data)
+                except Exception:
+                    old_data = {}
+            if isinstance(new_data, str):
+                try:
+                    new_data = json.loads(new_data)
+                except Exception:
+                    new_data = {}
+        
             email = user_info.get("email", "Alguém")
-            name2 = None
-
-            try:
-                if reference == "User" and new_data:
-                    name2 = eval(new_data).get("name")
-                elif reference == "Approval" and old_data:
-                    name2 = eval(old_data).get("name")
-                elif reference in ("Collaboration", "Onboarding") and new_data:
-                    name2 = eval(new_data).get("name")
-            except Exception:
-                name2 = None
-
+            target_email = None
+        
+            if reference == "User" and new_data:
+                target_email = new_data.get("email")
+            elif reference == "Approval" and old_data:
+                target_email = old_data.get("name")
+            elif reference == "Collaboration":
+                target_email = (
+                    new_data.get("email") or old_data.get("email")
+                )
+            elif reference == "Onboarding":
+                target_email = None
+        
             if action == "update" and reference == "User":
-                return f"{email} atualizou informações de {name2 or 'um usuário'}"
+                return f"{email} atualizou informações de {target_email or 'um usuário'}"
             elif action == "approve" and reference == "Approval":
-                return f"{email} aprovou o estilista {name2 or ''}"
+                return f"{email} aprovou o estilista {target_email or ''}"
             elif action == "deny" and reference == "Approval":
-                return f"{email} negou o estilista {name2 or ''}"
+                return f"{email} negou o estilista {target_email or ''}"
             elif action == "update" and reference == "Onboarding":
                 return f"{email} atualizou uma pergunta do onboarding"
             elif action == "add" and reference == "Onboarding":
@@ -158,12 +170,12 @@ async def get_logs(
             elif action == "delete" and reference == "Onboarding":
                 return f"{email} excluiu uma pergunta do onboarding"
             elif action == "add" and reference == "Collaboration":
-                return f"{email} adicionou um novo colaborador"
+                return f"{email} adicionou {target_email or 'um colaborador'} como novo colaborador"
             elif action == "delete" and reference == "Collaboration":
-                return f"{email} excluiu um colaborador da lista de colaboradores"
+                return f"{email} excluiu {target_email or 'um colaborador'} da lista de colaboradores"
             elif action == "update" and reference == "Collaboration":
-                return f"{email} atualizou as permissões de um colaborador"
-
+                return f"{email} atualizou as permissões de {target_email or 'um colaborador'}"
+        
             return f"{email} realizou uma ação: {action} em {reference}"
 
         enriched_logs = []