]> gitweb.erp-flowers.ru Git - erp24_rep/yii-erp24/.git/commitdiff
ERP[293] ошибка в случае отсутствия пользователя с таким номером origin/bug_smirnov_2025_01_23_user_is_null
authorAlexander Smirnov <fredeom@mail.ru>
Thu, 23 Jan 2025 12:13:32 +0000 (15:13 +0300)
committerAlexander Smirnov <fredeom@mail.ru>
Thu, 23 Jan 2025 12:13:32 +0000 (15:13 +0300)
erp24/api2/controllers/ClientController.php

index d4a621abd96371364f362cb731db67f5d5b1ca57..5337288332409c7c4aee23fb9011573b506fada9 100644 (file)
@@ -966,7 +966,7 @@ class ClientController extends BaseController {
         }
 
         if (!isset($result['telegram_is_subscribed'])) {
-            return $this->asJson(["error_id" => 2, "error" => "is_subscribed is required"]);
+            return $this->asJson(["error_id" => 2, "error" => "telegram_is_subscribed is required"]);
         }
 
         $result['phone'] = $phone = ClientHelper::phoneClear($result['phone']);
@@ -978,6 +978,10 @@ class ClientController extends BaseController {
         Users::updateAll(['telegram_is_subscribed' => $result['telegram_is_subscribed'] == 1 ? 1 : 0], ['phone' => $phone]);
 
         $user = Users::find()->where(['phone' => $phone])->one();
+        if (!$user) {
+            return $this->asJson(["error_id" => 2, "error" => "no User with mobile $phone in db"]);
+        }
+
         /** @var $user Users */
         if ($result['telegram_is_subscribed'] == 1 && $user->telegram_created_at == null) {
             $user->telegram_created_at = date("Y-m-d H:i:s");