website/views/company.py (2)

563-590: Avoid blanket exception handling when building team_members

Catching every exception here hides real bugs and tripped Ruff BLE001. We only need to cope with missing user profiles, so rely on getattr instead of swallowing everything.

-        team_members = []
-        for user in team_member_users:
-            try:
-                profile = user.userprofile
-                # Create a wrapper object with expected attributes
-                member_data = type(
-                    "obj",
-                    (object,),
-                    {
-                        "user": user,
-                        "user_avatar": profile.user_avatar if hasattr(profile, "user_avatar") else None,
-                        "role": "Admin" if user == organization_obj.admin else "Manager",
-                    },
-                )()
-                team_members.append(member_data)
-            except Exception:
-                # If userprofile doesn't exist, create basic member data
-                member_data = type(
-                    "obj",
-                    (object,),
-                    {
-                        "user": user,
-                        "user_avatar": None,
-                        "role": "Admin" if user == organization_obj.admin else "Manager",
-                    },
-                )()
-                team_members.append(member_data)
+        team_members = []
+        for user in team_member_users:
+            profile = getattr(user, "userprofile", None)
+            avatar = getattr(profile, "user_avatar", None) if profile else None
+            member_data = type(
+                "obj",
+                (object,),
+                {
+                    "user": user,
+                    "user_avatar": avatar,
+                    "role": "Admin" if user == organization_obj.admin else "Manager",
+                },
+            )()
+            team_members.append(member_data)

Based on static analysis hints

---

618-637: Tighten avatar lookup in the AJAX payload

The second blanket except Exception for avatar resolution raises the same Ruff BLE001 warning. Switching to safe getattr calls keeps the code resilient to missing profiles without erasing unexpected errors (and keeps the log noise down).

-                try:
-                    avatar_url = (
-                        report.user.userprofile.user_avatar.url if report.user.userprofile.user_avatar else None
-                    )
-                except Exception as e:
-                    logger.warning(f"Error getting avatar for user {report.user.username}: {e}")
-                    avatar_url = None
+                user_profile = getattr(report.user, "userprofile", None)
+                avatar_field = getattr(user_profile, "user_avatar", None) if user_profile else None
+                avatar_url = avatar_field.url if avatar_field else None

Based on static analysis hints

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Knowledge base: Disabled due to Reviews -> Disable Knowledge Base setting