refactor: remove redundant invite subquery from UserFilter

Now that User and CompanyUser are pinned to the production connection via
getConnectionName(), the whereHas('companyUsers') check correctly finds all
members regardless of sandbox mode. The orWhereExists invite subquery was
a workaround for the wrong root cause and is no longer needed.

Author: Ronald A Richardson

src/Http/Filter/UserFilter.php

@@ -8,27 +8,10 @@ public function queryForInternal()
     {
         $companyUuid = $this->session->get('company');
 
-        $this->builder->where(
-            function ($query) use ($companyUuid) {
-                // Include users who are already members of the company.
-                $query->whereHas(
-                    'companyUsers',
-                    function ($q) use ($companyUuid) {
-                        $q->where('company_uuid', $companyUuid);
-                    }
-                )
-                // Also include users who have a pending invite to join the company
-                // but have not yet accepted (no CompanyUser row exists yet).
-                // The invites table has no user_uuid; the link is via the user's
-                // email stored in the JSON recipients column.
-                ->orWhereExists(function ($q) use ($companyUuid) {
-                    $q->selectRaw(1)
-                      ->from('invites')
-                      ->whereRaw('JSON_CONTAINS(invites.recipients, JSON_QUOTE(users.email))')
-                      ->where('invites.company_uuid', $companyUuid)
-                      ->where('invites.reason', 'join_company')
-                      ->whereNull('invites.deleted_at');
-                });
+        $this->builder->whereHas(
+            'companyUsers',
+            function ($q) use ($companyUuid) {
+                $q->where('company_uuid', $companyUuid);
             }
         );
     }