ну то есть ты можешь себе представить репозиторий БЕЗ зависимостей?

соединение с базой данных это сервис? entity manager это сервис?

Ааа, зачем углубляться в понятия? Важно же решение

Ааа, зачем углубляться в понятия? Важно же решение

важно понимание

в частности чем плохо в коде делать new SomeRepo

ну и вообще в чем ограничения new

в частности чем плохо в коде делать new SomeRepo

Я не говорю, что это плохо. Все от ситуации зависит

ну и вообще в чем ограничения new

Тогда он сможет инжектить зависимости. Разве, что через контейнер тащить. Получится то же самое, что и с сервисом. Только будет лишний код

Я не говорю, что это плохо. Все от ситуации зависит

я говорю что это не очень хорошо)

Тогда он сможет инжектить зависимости. Разве, что через контейнер тащить. Получится то же самое, что и с сервисом. Только будет лишний код

еще раз - у тебя может быть репозиторий без зависимостей?

ну вот чисто теоритически даже (если не брать в расчет in memory репозитории-заглушки какие)

я говорю что это не очень хорошо)

Если у класса нет зависимостей, то тоже нужно создавать через контейнер?)

Если у класса нет зависимостей, то тоже нужно создавать через контейнер?)

it depends.

есть объекты значения, есть сущности и т.д.

it depends.

Я же это и говорю, что от ситуации все зависит

важно только добиться того что бы вызов new был в одном месте. Будь это статический метод-фабрика или еще чего

Я же это и говорю, что от ситуации все зависит

Репозиторий - конкретная ситуация

и разговор был именно об регистрации репозиториев как сервисов

автоматической регистрации

так что еще раз - в случае репозитория у тебя есть хоть один довод почему "можно без контейнера?" Естественно мы рассматриваем ситуацию когда контейнер у тебя уже есть. В ситуациях легаси где у тебя нельзя просто взять и перевести все на контейнер чуть другая история (промежуточные штуки вроде сервис локаторов)

короч я не понимаю почему нет смысла "инджектить в репы".... коннекшен к базе тебе придется заинджектить как-то

а то что ты видимо имел ввиду называется SRP

Я перечитал вопрос, весь диалог и понял, что был не прав

Прошу прощения

Что за беда с наследованием в Sonata?

Я про админский контроллер

An exception has been thrown during the rendering of a template ("Unable to find a valid admin for the class: AppBundle\Entity\User, there are too many registered: admin.user, admin.user_moderation"

судя по ошибке - ты сконфигурил две админки для одного класса

admin.user и admin.user_moderation как сконфигурированы?

так там надо прописать параметр admin_code и указать какой админкласс юзать

а две админки для одной энтити делать можно, но там есть нюансы

а две админки для одной энтити делать можно, но там есть нюансы

поподробнее :)

Да, одна сущность - два контроллера

Что за беда с наследованием в Sonata?

та же беда что и со всем наследованием)

На самом деле задача простая.. У меня есть уже весь нужный функционал в admin.user, а контроллер который наследует admin.user - там просто чутка другой запроса - выводит всех юзеров которые на модерации. в итоге при клике на юзера - ссылка должна вести на базовый контроллер. и всё

@fes0r вспоминая твои речи про логику в контроллерах имеет ли смысл выносить простые штуки (типа редактирования сущности) в какие-нибудь сервис-менеджеры, которые сами по себе будут на 4-5 строк и будут просто делегировать задачи сущностям и дергать persist/flush?

и да, твой сервис для флашей - он просто содержит один метод?)

@fes0r вспоминая твои речи про логику в контроллерах имеет ли смысл выносить простые штуки (типа редактирования сущности) в какие-нибудь сервис-менеджеры, которые сами по себе будут на 4-5 строк и будут просто делегировать задачи сущностям и дергать persist/flush?

не вижу... смысл есть только в ситуации когда у тебя более-менее большая команда и тебе проще сказать всем "всегда оркестрацию в сервисы" а не "думай головой"

и да, твой сервис для флашей - он просто содержит один метод?)

да. один метод

ну не, 3 на самом деле)

flush, commit и flushAndCommit

не вижу... смысл есть только в ситуации когда у тебя более-менее большая команда и тебе проще сказать всем "всегда оркестрацию в сервисы" а не "думай головой"

вот тут команда как раз примитивная - подменяю ту самую contactData

commit - транзакции?

но это потому что у меня и доктрина и пруф фигурировать могут в одной операции. В большинстве случаев должно вызываться только flushAndCommit

commit - транзакции?

не, это UoW от prooph

а persist там, где он нужен, дергаешь репозиторием?

Привет, как красиво сделать такую штуку. Есть у меня коллекция юзеров /users/<id>/ ну и там операции с каждым, Через что мне красиво сделать /users/me/ что б юзался тот же контролер, но подставлялся ид текущего юзера ?

создать новый роут с path: /users/me и в defaults id: 0

который ссылается на тот же контроллер

и в контроллере проверить, если id == 0, то подставить id текущего юзера

??

хм ... как вариант

да, как вариант

и важно: этот роут должен находиться выше, чем /users/{userId}

и важно: этот роут должен находиться выше, чем /users/{userId}

да

и важно: этот роут должен находиться выше, чем /users/{userId}

Это понятно )

Спасибо

или /users/{userId} - должен быть requirement, что id - только цифры

Я думал как-то в роутинге сервеис цеплять, который это будет делать ... или какой-то контроллер который сделает уже запрос на /users/56/ например. Ну типа middleware

или /users/{userId} - должен быть requirement, что id - только цифры

у меня uuid

Я думал как-то в роутинге сервеис цеплять, который это будет делать ... или какой-то контроллер который сделает уже запрос на /users/56/ например. Ну типа middleware

а почему нельзя сделать отдельный экшн /users/me и вызывать $this->usersAction($this->getUser()) или делать forward? Ну или вынести в приватную функцию общий код и вызывать из двух мест?

вариантов простых куча

а почему нельзя сделать отдельный экшн /users/me и вызывать $this->usersAction($this->getUser()) или делать forward? Ну или вынести в приватную функцию общий код и вызывать из двух мест?

Потому что там есть GET/POST/PUT/PATCH ... не хотелось дублировать. Вот думал про форвард, но предполагал что есть еще какой-то вариант )

Что за беда с наследованием в Sonata?

Разобрался с проблемой?

Именно вот как только что месага была, как у ВК с id0, редирект на текущего бзера ... но что б редирект был на серваке )

Разобрался с проблемой?

Неа..

ERROR: S client not available

Неа..

Пиши в личку или в группу Sonata, я тебе приглашение скинул

Именно вот как только что месага была, как у ВК с id0, редирект на текущего бзера ... но что б редирект был на серваке )

это называется forward =) или вызов другой функции

это называется forward =) или вызов другой функции

я так думал сделать, но решил спросить чатег, возможно есть еще какие варианты, более красивые )

Пиши в личку или в группу Sonata, я тебе приглашение скинул

Скиньте мне доступ в группу сонаты

Спасибо

кто знает как правильно мокать сервисы в функциональных тестах в 4 симфе ? раньше можно было this->getContainer()->set('idService', $mock) но теперь это deprecation

$this->createMock не работает?

oO а так можно было?

$this->createMock не работает?

не, там ведь не про это речь

или я чего-то не знаю?

just include a .yml file in your config_test.yml. Rewrite the needed service definitions by using the same key or using decorator (http://symfony.com/doc/current/service_container/service_decoration.html) in that file.

я так понял, он хочет, чтобы $this->getContainer()->get('...') возвращало ему мок

но это ж пздц?

не?

ну для тестов может и нет если не брать во внимание использование контроллера :D

это было сделано для кейсов когда в сервис инжектится контэйнер? )

а речь точно про функциональные? там же просто кравлер хтмла

я так понял, он хочет, чтобы $this->getContainer()->get('...') возвращало ему мок

Второй параметр отвечает же за поведение при отсутствующем сервисе?

Он, вроде, так и называется

а речь точно про функциональные? там же просто кравлер хтмла

Функциональные бывают разными)

Второй параметр отвечает же за поведение при отсутствующем сервисе?

так то для тестов, возможно, контроллера

Другой контейнер в тестах?)

а persist там, где он нужен, дергаешь репозиторием?

да конечно

я так понял, он хочет, чтобы $this->getContainer()->get('...') возвращало ему мок

он хочет чтобы в функциональном тесте заменялся сервис на мок

он хочет чтобы в функциональном тесте заменялся сервис на мок

а сервис откуда получаешь?

autowiring

autowiring

https://github.com/symfony/symfony-docs/issues/8203

читал эту статью - но грузить отдельный конфиг со всеми моканными сервисами как по мне ту мач. Обратился сюда - думал может кто знает более логичное / элегантное решение

читал эту статью - но грузить отдельный конфиг со всеми моканными сервисами как по мне ту мач. Обратился сюда - думал может кто знает более логичное / элегантное решение

Практикуем такую практику. Зависимость есть, брат жив

мне на самом деле тоже не нравится такое

но я не пишу функциональные, потому мне ок)

делаете алиасы на нужные сервисы с пометкой public: true, делаете set для test.* сервисов