Merge branch 'master' of git.task-on.com:ktask/task-on.com

common/modules/triggers/controllers/DefaultController.php

@@ -30,7 +30,8 @@ class DefaultController extends BaseController
             'Rechecktriggers' => 'Запуск проверки срабатывания триггеров',
             'Recheckcases' => 'Запуск рассылки кейсов',
             'Settimeoflogs' => 'Фиксация времени логов',
-            'Inittrigger' => 'Initiate trigger By id'
+            'Inittrigger' => 'Initiate trigger By id',
+            'Sendcases' => 'Send cases by schedule'
         ];
     }
 
@@ -139,7 +140,7 @@ class DefaultController extends BaseController
                 // Перебираем все case-ы
                 foreach($cases as $case) {
                     // Проверили наличие отправки кейса юзеру
-                    $scheduleExist = CasesSchedule::find()->where(['id' => $case->id, 'user_id' => $user->id])->exists();
+                    $scheduleExist = CasesSchedule::find()->where(['case_id' => $case->id, 'user_id' => $user->id])->exists();
                     // Если отправка в расписании отсутствует
                     if (!$scheduleExist) {
                         // Заносим в расписание
@@ -148,9 +149,10 @@ class DefaultController extends BaseController
                         $model->case_id = $case->id;
                         $model->template_id = (!is_null($case->template_id)) ? $case->template_id : null;
                         $model->sended_date = null;
-                        $model->sended = false;
-                        $model->actual = true;
-                        $model->save();
+                        $model->sended = 0;
+                        $model->actual = 1;
+                        if (!$model->save())
+                            echo current(current($model->getErrors())).'<br>';
                     }
                 }
             }
@@ -185,9 +187,10 @@ class DefaultController extends BaseController
                     if ($sendedToUser===false) {
                         // Получили кейс
                         /** @var CoContent $case */
-                        $case = $record->getCase();
+                        $case = $record->case;
                         // Если кейс есть и у него есть шаблон
-                        if (!is_null($case) && !is_null($case->template_id)) {
+                        if (!is_null($case) && (!is_null($case->template_id) || !is_null($record->template_id))) {
+                            $template_id = (!is_null($case->template_id)) ? $case->template_id : $record->template_id;
                             // Заносим урл в переменную
                             $url = $case->url;
                             // Получаем время просмотра каждой страницы
@@ -201,11 +204,12 @@ class DefaultController extends BaseController
                                 $hash=AuthHashes::setNewHash($email);
                                 // Сформировали дату
                                 $templateData = [
-                                    'case_link' => $baseUrl.$url.'?hash='.$hash.'&email='.$email
+                                    'case_link' => $baseUrl.$url.'?hash='.$hash.'&email='.$email,
+                                    'user_fio' => $user->getFio()
                                 ];
                                 // Производим отправку
                                 /** @var \DateTime $time_now */
-                                TriggerSchedule::sendMessage($email, $case->template_id, $templateData, null);
+                                TriggerSchedule::sendMessage($email, $template_id, $templateData, null);
                                 // Произвели отправку, следовательно переопределили значение переменной
                                 $sendedToUser=true;
                                 // Редактируем запись в расписании
@@ -224,7 +228,7 @@ class DefaultController extends BaseController
                 $updatedDate->modify('+7 days');
                 /** @var User $user */
                 // Нашли пользователя
-                $user=User::findOne($this->id);
+                $user=User::findOne($user->id);
                 // Переопределили поле
                 $user->delivery_date = $updatedDate->format('Y-m-d');
                 // Сохранили изменения

common/modules/triggers/models/TriggerSchedule.php

@@ -41,7 +41,7 @@ class TriggerSchedule extends \yii\db\ActiveRecord
     {
         return [
             [['sended', 'checked', 'message_id', 'list_id', 'campaign_id', 'trigger_id', 'template_id'], 'integer'],
-            [['message', 'email', 'template_id', 'trigger_id'], 'required'],
+            [['message', 'email', 'template_id'], 'required'],
             [['message'], 'string'],
             [['time', 'date_create'], 'safe'],
             [['email'], 'string', 'max' => 255],

console/migrations/m160324_144228_add_template_column_to_co_content.php

@@ -11,6 +11,11 @@ class m160324_144228_add_template_column_to_co_content extends Migration
             'template_id',
             $this->integer(11)->defaultValue(null)
         );
+        $this->createIndex(
+            'tempalte',
+            'co_content',
+            'template_id'
+        );
         $this->addForeignKey(
             'fk_co_content_template',
             'co_content', 'template_id',