fixes for adding users into the production system

app/Http/Requests/User/StoreUserRequest.php

@@ -58,7 +58,7 @@ class StoreUserRequest extends Request
     {
         $input = $this->all();
 
-//unique user rule - check company_user table for user_id / company_id  / account_id if none exist we can add the user. ELSE return false
+        //unique user rule - check company_user table for user_id / company_id  / account_id if none exist we can add the user. ELSE return false
 
         if(array_key_exists('email', $input))
             $input['email'] = trim($input['email']);

app/Http/ValidationRules/Ninja/CanAddUserRule.php

@@ -33,13 +33,18 @@ class CanAddUserRule implements Rule
     public function passes($attribute, $value)
     {
 
-          $count = CompanyUser::query()
+        /* If the user is active then we can add them to the company */
-                              ->where('company_user.account_id', auth()->user()->account_id)
+        if(User::where('email', request()->input('email'))->where('account_id', auth()->user()->account_id)->where('is_deleted',0)->exists())
-                              ->join('users', 'users.id', '=', 'company_user.user_id')
+            return true;
-                              ->whereNull('users.deleted_at')
+
-                              ->whereNull('company_user.deleted_at')
+        /* Check that we have sufficient quota to allow this to happen */
-                              ->distinct()
+        $count = CompanyUser::query()
-                              ->count('company_user.user_id');
+                          ->where('company_user.account_id', auth()->user()->account_id)
+                          ->join('users', 'users.id', '=', 'company_user.user_id')
+                          ->whereNull('users.deleted_at')
+                          ->whereNull('company_user.deleted_at')
+                          ->distinct()
+                          ->count('company_user.user_id');
 
         return $count < auth()->user()->company()->account->num_users;
 

app/Models/Company.php

@@ -171,7 +171,7 @@ class Company extends BaseModel
 
     public function users()
     {
-        return $this->hasManyThrough(User::class, CompanyUser::class, 'company_id', 'id', 'id', 'user_id');
+        return $this->hasManyThrough(User::class, CompanyUser::class, 'company_id', 'id', 'id', 'user_id')->withTrashed();
     }
 
     public function expense_categories()

app/Repositories/UserRepository.php

@@ -189,6 +189,14 @@ class UserRepository extends BaseRepository
             return;
         }
 
+        if (Ninja::isHosted()) {
+            
+            $count = User::where('account_id', auth()->user()->account_id)->count();
+            if($count >= auth()->user()->account->num_users)
+                return;
+
+        }
+
         $user->is_deleted = false;
         $user->save();
         $user->restore();