Update user model

app/Models/User/Load.php

@@ -23,9 +23,15 @@ class Load extends Action
 
     /**
      * Получает пользователя по id
+     *
+     * @throws InvalidArgumentException
      */
     public function load(int $id): ?User
     {
+        if ($id < 1) {
+            throw new InvalidArgumentException('Expected a positive user id');
+        }
+
         $vars = [':id' => $id];
         $sql  = $this->createQuery('u.id=?i:id');
         $data = $this->c->DB->query($sql, $vars)->fetch();
@@ -35,9 +41,17 @@ class Load extends Action
 
     /**
      * Получает массив пользователей по ids
+     *
+     * @throws InvalidArgumentException
      */
     public function loadByIds(array $ids): array
     {
+        foreach ($ids as $id) {
+            if (! \is_int($id) || $id < 1) {
+                throw new InvalidArgumentException('Expected a positive user id');
+            }
+        }
+
         $vars = [':ids' => $ids];
         $sql  = $this->createQuery('u.id IN (?ai:ids)');
         $data = $this->c->DB->query($sql, $vars)->fetchAll();

app/Models/User/Manager.php

@@ -4,7 +4,6 @@ namespace ForkBB\Models\User;
 
 use ForkBB\Models\ManagerModel;
 use ForkBB\Models\User\Model as User;
-use InvalidArgumentException;
 
 class Manager extends ManagerModel
 {
@@ -22,15 +21,9 @@ class Manager extends ManagerModel
 
     /**
      * Получает пользователя по id
-     *
-     * @throws InvalidArgumentException
      */
     public function load(int $id): ?User
     {
-        if ($id < 1) {
-            throw new InvalidArgumentException('Expected id > 0');
-        }
-
         if ($this->isset($id)) {
             return $this->get($id);
         } else {
@@ -42,8 +35,6 @@ class Manager extends ManagerModel
 
     /**
      * Получает массив пользователей по ids
-     *
-     * @throws InvalidArgumentException
      */
     public function loadByIds(array $ids): array
     {
@@ -51,9 +42,6 @@ class Manager extends ManagerModel
         $data   = [];
 
         foreach ($ids as $id) {
-            if (! \is_int($id) || $id < 1) {
-                throw new InvalidArgumentException('Expected id > 0');
-            }
             if ($this->isset($id)) {
                 $result[$id] = $this->get($id);
             } else {