Size: a a a

2021 January 26

VM

Volodymyr Melko in symfony
Alexander
имеешь в виду, что можно туда "дописать" emailtocutomer?
ну я не Ванга, чтоб гадать, что у вас можно, а что нет.
Просто выглядит, что case содержит какие-то начальные данные, которые нужны для выполнения
источник

VM

Volodymyr Melko in symfony
Alexander
логично, что он содержит в себе данные 😊
чем тогда mailtocustomer отличается от других данных?
источник

A

Alexander in symfony
Volodymyr Melko
ну я не Ванга, чтоб гадать, что у вас можно, а что нет.
Просто выглядит, что case содержит какие-то начальные данные, которые нужны для выполнения
логично. для того его и передают сервису 😊
источник

A

Alexander in symfony
mailtocustomer - это и есть гет-параметр, получаемый из урла
источник

VM

Volodymyr Melko in symfony
Alexander
mailtocustomer - это и есть гет-параметр, получаемый из урла
ну так почему бы его не запихнуть в кейс? что-то мешает?
источник

A

Alexander in symfony
Volodymyr Melko
ну так почему бы его не запихнуть в кейс? что-то мешает?
теоретически можно.
$case->emailtocustomer = $emailcustomer
только по-моему, это костыль ещё больший. case - объект, полученный из базы, ничего не знающий о emailtocustomer
источник

VM

Volodymyr Melko in symfony
Alexander
теоретически можно.
$case->emailtocustomer = $emailcustomer
только по-моему, это костыль ещё больший. case - объект, полученный из базы, ничего не знающий о emailtocustomer
ну тогда я хз что вы там за архитектуру строите =)
источник

I

Ivan in symfony
там проще все удалить и с нуля написать)
источник

C

CvekCoder in symfony
Alexander
теоретически можно.
$case->emailtocustomer = $emailcustomer
только по-моему, это костыль ещё больший. case - объект, полученный из базы, ничего не знающий о emailtocustomer
Сделайте DTO, и положите в него ваш объект из базы + необходимые параметры из реквеста.
Но переделывать код придется, это нормально. Рефакторинг никто не отменял
источник

C

CvekCoder in symfony
Если оставите ваш Request в бизнес-логике, то через годик, если проект будет расти, вы наступите на эти грабли, и придется переделывать больше кода, чем сейчас
источник

C

CvekCoder in symfony
А если вам и тогда будет лень переделать, то вы налепите костыли и проект будет продолжать наращивать вероятность умереть под тяжестью подобного техдолга
источник

C

CvekCoder in symfony
Я таких проектов повидал, попеределывал. Знаю что говорю)
источник

A

Alexander in symfony
да-да 😊
источник

СБ

Сергей Белоусов... in symfony
Alexander
Всем привет!
Не могу понять, почему не инжектится сервис. В чём может быть проблема?
Получаю ошибку.
Cannot autowire service "App\Macros\Cases\Transitions\Draft\ToDocumentReviewMacro": argument "$request" of method "__construct()" references class "Symfony\Component\HttpFoundation\Request" but no such service exists.

В этом же проекте этот же Symfony\Component\HttpFoundation\Request прекрасно инжектится в другие сервисы
Так а нельзя сделать параметр в методе не обязательным, и передавать его только там где нужно, а по умолчанию пусть будет false ?
источник

A

Alexander in symfony
Сергей Белоусов
Так а нельзя сделать параметр в методе не обязательным, и передавать его только там где нужно, а по умолчанию пусть будет false ?
К сожалению, нет. Точнее, конечно, можно, но тогда придётся менять методы в нескольких десятках классов, т.к. метод объявлен в абстрактном классе
источник

A

Alexander in symfony
CvekCoder
Сделайте DTO, и положите в него ваш объект из базы + необходимые параметры из реквеста.
Но переделывать код придется, это нормально. Рефакторинг никто не отменял
Извините, не совсем понял.
Объект из базы достаётся с помощью Paramconvertr'a
@Rest\Put("/case/diesel/{case}/commit", requirements={"case" = "\d+"})
что значит "Сделайте DTO"?
источник

I

Ivan in symfony
Alexander
Извините, не совсем понял.
Объект из базы достаётся с помощью Paramconvertr'a
@Rest\Put("/case/diesel/{case}/commit", requirements={"case" = "\d+"})
что значит "Сделайте DTO"?
источник

A

Alexander in symfony
непонятно, что имеется в виду
источник

I

Ivan in symfony
Ну значит будут костыли в этом проекте
источник

C

CvekCoder in symfony
Alexander
непонятно, что имеется в виду
Почитайте про DTO в других источниках. Концепция простая, должна одолеть
источник