From 718ff6b7abb2576c1ea1336affc4c8506eb6a505 Mon Sep 17 00:00:00 2001
From: Matt Young <matt@mattyoung.us>
Date: Thu, 27 Jun 2024 23:42:28 -0500
Subject: [PATCH] Remove TODO for future issues

---
 app/Policies/SchoolPolicy.php | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/app/Policies/SchoolPolicy.php b/app/Policies/SchoolPolicy.php
index fa561a1..e227f74 100644
--- a/app/Policies/SchoolPolicy.php
+++ b/app/Policies/SchoolPolicy.php
@@ -4,20 +4,23 @@ namespace App\Policies;
 
 use App\Models\School;
 use App\Models\User;
-use Illuminate\Auth\Access\Response;
+
 use function is_null;
 
 class SchoolPolicy
 {
-    // TODO Blanket admin policy is not appropriate for schools as it may break things in the audition process
     /**
      * Grant admin users access to all functions
      */
-    public function before(User $user, string $ability): bool|null
+    public function before(User $user, string $ability): ?bool
     {
-        if($user->is_admin) return true;
+        if ($user->is_admin) {
+            return true;
+        }
+
         return null;
     }
+
     /**
      * Determine whether the user can view any models.
      */