Minor improvements to incident edit permissions

src/dispatch/auth/permissions.py

@@ -271,8 +271,9 @@ def has_required_permissions(
         if not current_incident:
             return False
 
-        if current_incident.reporter.individual.email == current_user.email:
-            return True
+        if current_incident.reporter:
+            if current_incident.reporter.individual.email == current_user.email:
+                return True
 
 
 class IncidentCommanderPermission(BasePermission):
@@ -285,7 +286,7 @@ def has_required_permissions(
             db_session=request.state.db, incident_id=request.path_params["incident_id"]
         )
         if not current_incident:
-            return
+            return False
 
         if current_incident.commander:
             if current_incident.commander.individual.email == current_user.email:

src/dispatch/incident/views.py

@@ -10,13 +10,13 @@
 from fastapi import APIRouter, BackgroundTasks, Depends, HTTPException, Query, status
 
 from sqlalchemy.orm import Session
+
 from dispatch.auth.permissions import (
     IncidentEditPermission,
     IncidentJoinOrSubscribePermission,
-    PermissionsDependency,
     IncidentViewPermission,
+    PermissionsDependency,
 )
-
 from dispatch.auth.models import DispatchUser
 from dispatch.auth.service import get_current_user
 from dispatch.common.utils.views import create_pydantic_include