Правки активность рассылки

diff --git a/erp24/commands/CronController.php b/erp24/commands/CronController.php
index b3a95f1dcb1bd617174f85eff566b24a7deb5a2a..01e03e33809af0984f146ea26afe7aaca5b2d80d 100644 (file)
--- a/erp24/commands/CronController.php
+++ b/erp24/commands/CronController.php
@@ -188,11 +188,24 @@ class CronController extends Controller
     public function actionGenerateTargetKogorts()
     {
         $messagesSettings = UsersMessageManagement::find()->one();
-        if (empty($messagesSettings) && $messagesSettings->active == 0) {
-            $this->stdout("Рассылка неактивна. Генерация корорт прервана.\n", BaseConsole::FG_RED);
+        if (!$messagesSettings) {
+            $this->stdout(
+                "Рассылка неактивна (не найдена настройка). Генерация корорт прервана.\n",
+                BaseConsole::FG_RED
+            );
             return ExitCode::UNAVAILABLE;
         }
 
+        if (!$messagesSettings->active) {
+            $this->stdout(
+                "Рассылка неактивна (поле active = 0). Генерация корорт прервана.\n",
+                BaseConsole::FG_RED
+            );
+            return ExitCode::UNAVAILABLE;
+        }
+
+        $this->stdout("Рассылка активна. Начинаем генерацию когорт...\n", BaseConsole::FG_GREEN);
+
         date_default_timezone_set('Europe/Moscow');
 
         if ($this->time && !is_numeric($this->time)) {
@@ -219,9 +232,15 @@ class CronController extends Controller
 
             if (!empty($kogortPhones)) {
                 Users::saveKogort($kogortPhones, $kogortDate, $targetDate, SentKogort::KOGORT_NUMBERS['target']);
-                $this->stdout("Когорта на {$kogortDate} для целевой даты {$targetDate} успешно сохранена.\n", BaseConsole::FG_GREEN);
+                $this->stdout(
+                    "Когорта таргета на {$kogortDate} для целевой даты {$targetDate} успешно сохранена.\n",
+                    BaseConsole::FG_GREEN
+                );
             } else {
-                $this->stdout("Нет данных для формирования когорты для целевой даты {$targetDate}.\n", BaseConsole::FG_RED);
+                $this->stdout(
+                    "Нет данных для формирования когорты для целевой даты {$targetDate}.\n",
+                    BaseConsole::FG_RED
+                );
             }
         }
 
@@ -233,9 +252,15 @@ class CronController extends Controller
 
         if (!empty($kogortPhones)) {
             Users::saveKogort($kogortPhones, $kogortDate, $targetDate, SentKogort::KOGORT_NUMBERS['target']);
-            $this->stdout("Когорта на {$kogortDate} для целевой даты {$targetDate} успешно сохранена.\n", BaseConsole::FG_GREEN);
+            $this->stdout(
+                "Когорта таргета на {$kogortDate} для целевой даты {$targetDate} успешно сохранена.\n",
+                BaseConsole::FG_GREEN
+            );
         } else {
-            $this->stdout("Нет данных для формирования когорты для целевой даты {$targetDate}.\n", BaseConsole::FG_RED);
+            $this->stdout(
+                "Нет данных для формирования когорты для целевой даты {$targetDate}.\n",
+                BaseConsole::FG_RED
+            );
         }
 
         // Генерация для будущих дат (от текущей даты на $step1 дней вперёд)
@@ -247,9 +272,15 @@ class CronController extends Controller
 
             if (!empty($kogortPhones)) {
                 Users::saveKogort($kogortPhones, $kogortDate, $targetDate, SentKogort::KOGORT_NUMBERS['target'], true);
-                $this->stdout("Когорта на {$kogortDate} для целевой даты {$targetDate} успешно сохранена.\n", BaseConsole::FG_GREEN);
+                $this->stdout(
+                    "Когорта таргета на {$kogortDate} для целевой даты {$targetDate} успешно сохранена.\n",
+                    BaseConsole::FG_GREEN
+                );
             } else {
-                $this->stdout("Нет данных для формирования когорты для целевой даты {$targetDate}.\n", BaseConsole::FG_RED);
+                $this->stdout(
+                    "Нет данных для формирования когорты для целевой даты {$targetDate}.\n",
+                    BaseConsole::FG_RED
+                );
             }
         }
 
@@ -265,10 +296,23 @@ class CronController extends Controller
     public function actionGenerateWhatsappKogorts()
     {
         $messagesSettings = UsersMessageManagement::find()->one();
-        if (empty($messagesSettings) && $messagesSettings->active == 0) {
-            $this->stdout("Рассылка неактивна. Генерация корорт прервана.\n", BaseConsole::FG_RED);
+        if (!$messagesSettings) {
+            $this->stdout(
+                "Рассылка неактивна (не найдена настройка). Генерация корорт прервана.\n",
+                BaseConsole::FG_RED
+            );
+            return ExitCode::UNAVAILABLE;
+        }
+
+        if (!$messagesSettings->active) {
+            $this->stdout(
+                "Рассылка неактивна (поле active = 0). Генерация корорт прервана.\n",
+                BaseConsole::FG_RED
+            );
             return ExitCode::UNAVAILABLE;
         }
+
+        $this->stdout("Рассылка активна. Начинаем генерацию когорт...\n", BaseConsole::FG_GREEN);
         date_default_timezone_set('Europe/Moscow');
 
         if ($this->time && !is_numeric($this->time)) {
@@ -310,9 +354,15 @@ class CronController extends Controller
 
             if (!empty($kogortPhones)) {
                 Users::saveKogort($kogortPhones, $kogortDate, $targetDate, SentKogort::KOGORT_NUMBERS['whatsapp'], true);
-                $this->stdout("Когорта на {$kogortDate} для целевой даты {$targetDate} успешно сохранена.\n", BaseConsole::FG_GREEN);
+                $this->stdout(
+                    "Когорта для Вотсапа на {$kogortDate} для целевой даты {$targetDate} успешно сохранена.\n",
+                    BaseConsole::FG_GREEN
+                );
             } else {
-                $this->stdout("Нет данных для формирования когорты для целевой даты {$targetDate}.\n", BaseConsole::FG_RED);
+                $this->stdout(
+                    "Нет данных для формирования когорты для целевой даты {$targetDate}.\n",
+                    BaseConsole::FG_RED
+                );
             }
         }
 
@@ -322,10 +372,24 @@ class CronController extends Controller
     public function actionGenerateCallKogorts()
     {
         $messagesSettings = UsersMessageManagement::find()->one();
-        if (empty($messagesSettings) && $messagesSettings->active == 0) {
-            $this->stdout("Рассылка неактивна. Генерация корорт прервана.\n", BaseConsole::FG_RED);
+        if (!$messagesSettings) {
+            $this->stdout(
+                "Рассылка неактивна (не найдена настройка). Генерация корорт прервана.\n",
+                BaseConsole::FG_RED
+            );
+            return ExitCode::UNAVAILABLE;
+        }
+
+        if (!$messagesSettings->active) {
+            $this->stdout(
+                "Рассылка неактивна (поле active = 0). Генерация корорт прервана.\n",
+                BaseConsole::FG_RED
+            );
             return ExitCode::UNAVAILABLE;
         }
+
+        $this->stdout("Рассылка активна. Начинаем генерацию когорт...\n", BaseConsole::FG_GREEN);
+
         date_default_timezone_set('Europe/Moscow');
 
         if ($this->time && !is_numeric($this->time)) {
@@ -365,9 +429,15 @@ class CronController extends Controller
 
             if (!empty($kogortPhones)) {
                 Users::saveKogort($kogortPhones, $kogortDate, $targetDate, SentKogort::KOGORT_NUMBERS['call'], true);
-                $this->stdout("Когорта на {$kogortDate} для целевой даты {$targetDate} успешно сохранена.\n", BaseConsole::FG_GREEN);
+                $this->stdout(
+                    "Когорта звонка на {$kogortDate} для целевой даты {$targetDate} успешно сохранена.\n",
+                    BaseConsole::FG_GREEN
+                );
             } else {
-                $this->stdout("Нет данных для формирования когорты для целевой даты {$targetDate}.\n", BaseConsole::FG_RED);
+                $this->stdout(
+                    "Нет данных для формирования когорты для целевой даты {$targetDate}.\n",
+                    BaseConsole::FG_RED
+                );
             }
         }