Спасибо. а на счет разделения шаблонов в зависимости от типа

мероприятия. в каком месте правильно вычислять, какой именно шаблон использовать?
сейчасть логика такая:
есть некий EmailManager, у которого есть метод sendEmailMessage(string $emailType, array $inputParams). данный метод, в зависимости от $emailType собирает нужный массив данных для письма($emailVars) и складывает их в БД. Дальше крон-скрипт берет из БД данные($emailType + $emailVars), в зависимости от $emailType выбирает нужный шаблон, собирает из $emailVars HTML письма и собственно отправляет его в smtp.

Мне кажется, что правильно выбирать шаблон как раз на этапе EmailManager-а, но тогда придется эту инфу прокидывать через БД в скрипт отправки. это норм?

7 ответов

11 просмотров

что мешает в $emailVars добавить нужный шаблон?

Yuri-Barsukov Автор вопроса
𝔏𝔦𝔩𝔦𝔱𝔥
что мешает в $emailVars добавить нужный шаблон?

ээ... а так можно было? 😂😂😂😂😂 спасибо. почему-то в эту сторону вообще не подумал :)

𝔏𝔦𝔩𝔦𝔱𝔥
что мешает в $emailVars добавить нужный шаблон?

я бы так не делал... плохая идея ложить в бд что-то, что хоть как-то опирается на структуру проекта. Пути к шаблонам, полные имена классов и т.п Типа что-то порефакторили, перенесли шаблон или переименовали и какой-то залипший меседж уже не уйдет. Если писем немного, то можно прямо в кнстанте положить мапу тайп-шаблон. Если много, то можно вынести в отдельный классик, который по типу вернет нужный шаблон

Yuri-Barsukov Автор вопроса
Volodymyr Melko
я бы так не делал... плохая идея ложить в бд что-т...

у нас сейчас так и сделано. в БД хранится $emailType, есть список соотвествия $emailType => $emailTemplate но сейчас появляется еще одно условие. на один и тот же $emailType два разных шаблона, в зависимости от определенных данных(тип мероприятия, для которого идет отправка письма)

Yuri Barsukov
у нас сейчас так и сделано. в БД хранится $emailTy...

Ну вот если появилась такая логика,то есть смысл вынести в отдельный класс, и может даже не один, если логика прямо развесистая для разных типов

Похожие вопросы

Обсуждают сегодня

Добрый вечер, Пока не совсем понимаю как наладить общение между телеграм ботом и ПО для работы с сим боксом. По самому боту так понял: - Нужен некий баланс, который можно поп...
Magic
6
сделал сайт, прикрутил в боте сайт, и виджет логина. как автоматически логинить пользователя в аккаунт(телеграм), при входе с бота?
Александра Чернивецкая
5
Объясните, пожалуйста, почему компилятор ругается на использование в условии неинициализированной переменной: int x; Task.Run(async () => { x = await somefunc(); }).Wait...
Александр
5
Ребят, подскажите, пожалуйста, почему в префиксе к ассетам, которые генерируются через фильтр | theme в шаблоне, стал вдруг появляться index.php? Вот так выглядит ссылка на а...
Виталий
1
Всем привет. Ребята, подскажите, пожалуйста. у ботов есть ограничение на отправку сообщений - 30 сообщений в секунду, эти ограничения накладываются на все сообщения? или на со...
Artem Stormageddon
4
Блин, ребята, сори за тупые вопросы. А можно ли как-то открыть вебапку по нажатию на кнопку в меню(которое появляется слева, команды)?
Artem Stormageddon
3
а плаксы из-под питона умеют только в комфортных условиях что-то выдавить из себя?)
Lencore
9
Но, может, есть уже проверенная? Наши требования такие: 1. Сообщения должны приходить из Инста в CRM оду 2. Должна быть возможность подключить несколько экаунтов Инстаграм. Р...
Alexander Sharoiko MSE / Александр Шаройко
13
Это может быть все-таки не флудвейт? у меня ботфазер принимает изменения и отображает даже что они изменились, на видео видно что он прислал якобы уже измененное описание, н...
OVERLINK
13
Коллеги, может знает кто, можно ли цвет бейджа счётчика в BackendMenu менять без бубнов?
Alex Blaze
3
Карта сайта