Size: a a a

2021 January 26

A

Alexander in symfony
в любом случае, можно поставить дополнительный if
источник

A

Alexander in symfony
сейчас проверю, что будет, если вызвать из консоли
источник

I

Ivan in symfony
Код еще не написал, а уже есть к нему костыли)
источник

VM

Volodymyr Melko in symfony
Alexander
по-моему, null
так то ты можешь впилить проверку на нулл, но это всеравно костыль и вам ридется это все переписывать, если таки с консоли надо будет отправлять письма
аля bin/console my:command —send-emails
источник

C

CvekCoder in symfony
@Olek0707 просто возьмите  за правило - ваша бизнес-логика не должна ничего знать про то, откуда она была вызвана. Она должна работать самодостаточно. Знание про Request должно остаться в контроллере и пропасть из области видимости как можно раньше. Тогда ваш бизнес-код будет максимально изолирован и самодостаточен.
источник

C

CvekCoder in symfony
И гибок - дергай его откуда угодно, соблюдая контракт вызова
источник

C

CvekCoder in symfony
А Request в бизнес-логике - это явная протечка и будущая бага
источник

VM

Volodymyr Melko in symfony
CvekCoder
А Request в бизнес-логике - это явная протечка и будущая бага
особенно весело когда переметр переименовывают и надо реплейсить в куче мест, вместо одного контроллера или даже одного аргмент резолвера
источник

A

Alexander in symfony
Volodymyr Melko
так то ты можешь впилить проверку на нулл, но это всеравно костыль и вам ридется это все переписывать, если таки с консоли надо будет отправлять письма
аля bin/console my:command —send-emails
Да ну, это уже какие-то дебри. Это уже чисто теоретические допущения, которые вряд ли когда-то могут быть в реальности.
источник

VM

Volodymyr Melko in symfony
Alexander
Да ну, это уже какие-то дебри. Это уже чисто теоретические допущения, которые вряд ли когда-то могут быть в реальности.
дело твое, но выше правильно говорят, что реквест не должен покидать пределов контроллера и в твоих сервисах ему не место, это порочная практика
источник

A

Alexander in symfony
ну как его перебросить? Надо менять десятки методов, которые никакого отношения не имеют в данной задаче
источник

I

Ivan in symfony
Значит надо менять и понижать связность кода
источник

A

Alexander in symfony
вот вызов сервиса $macro->run($case);
это метод абстрактного класса. Получается, нужно менять на $macro->run($case, $mailtocustomer);
источник

VM

Volodymyr Melko in symfony
а что такое $case?
источник

A

Alexander in symfony
этот абстрактный класс наследуют десятки сервисов
источник

A

Alexander in symfony
Volodymyr Melko
а что такое $case?
объект, передаваемый сервису
источник

VM

Volodymyr Melko in symfony
Alexander
объект, передаваемый сервису
выглядит так, что он содержит в себе данные для выполнения =)
источник

A

Alexander in symfony
имеешь в виду, что можно туда "дописать" emailtocutomer?
источник

A

Alexander in symfony
Volodymyr Melko
выглядит так, что он содержит в себе данные для выполнения =)
логично, что он содержит в себе данные 😊
источник

I

Ivan in symfony
$case mixed?
источник