Мне нужно запустить cli команду от интерфейса Console\Command внутри сервиса, вызвать через exec или получить через di команду и вызвать функцию execute с передачей input и output

Вцелом насрать, но лучше вызывая из кернела https://symfony.com/doc/current/console/command_in_controller.html

У меня нет кернела, я только использую компонент console

Вцелом насрать, но лучше вызывая из кернела https://symfony.com/doc/current/console/command_in_controller.html

Понял, вызову через консоль в php

Ну только помни, через ехес у тебя 2 кернел тайма будет

2 процесса имеешь ввиду?

Есть ДТО, в которую десериализуются данные через атрибуты (jms serializer), сделал public readonly для св-в, но так как конструктора нет, то phpstan на эти readonly св-ва ругается Есть несколько вариантов: 1. Заигнорить ошибки в конкретном случае 2. Убрать readonly и сделать приватные св-ва и геттеры 3. Оставить св-ва просто публичными без readonly Что-то ни один из вариантов не нравится. Кто как делает?

Есть ДТО, в которую десериализуются данные через атрибуты (jms serializer), сделал public readonly для св-в, но так как конструктора нет, то phpstan на эти readonly св-ва ругается Есть несколько вариантов: 1. Заигнорить ошибки в конкретном случае 2. Убрать readonly и сделать приватные св-ва и геттеры 3. Оставить св-ва просто публичными без readonly Что-то ни один из вариантов не нравится. Кто как делает?

4) сделать их protected

4) сделать их protected

Можно так, но тогда надо тоже геттеры делать. и protected это же больше ограничение при наследовании

Можно так, но тогда надо тоже геттеры делать. и protected это же больше ограничение при наследовании

Да, но зато только геттеры и пхпстан ругаться не будет, в отличие от способа с private полями

Да, но зато только геттеры и пхпстан ругаться не будет, в отличие от способа с private полями

Это да, иначе игнор придется делать

Есть ДТО, в которую десериализуются данные через атрибуты (jms serializer), сделал public readonly для св-в, но так как конструктора нет, то phpstan на эти readonly св-ва ругается Есть несколько вариантов: 1. Заигнорить ошибки в конкретном случае 2. Убрать readonly и сделать приватные св-ва и геттеры 3. Оставить св-ва просто публичными без readonly Что-то ни один из вариантов не нравится. Кто как делает?

так добавь конструктор, проблем-то

так добавь конструктор, проблем-то

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

final readonly class UserRequestDTO { public function __construct( #[SerializedName('name')] public string $login, public string $password, ) {} }

Просто не понятно как сериализатор будет работать

и не иметь себе мозг

нормально будет работать

разве что там есть опция какая-то на проверку неинициализированных полей, ща

так добавь конструктор, проблем-то

Я бы так и сделал. Сейчас есть ограничение, что при десериализации из метадаты создается объект, вот такая хрень public function construct( DeserializationVisitorInterface $visitor, ClassMetadata $metadata, $data, array $type, DeserializationContext $context ): ?object { $className = $metadata->name; return new $className(); }

На тебе: final readonly class ValidateMissingPropertiesSubscriber implements EventSubscriberInterface { /** * @return list<array<non-empty-string, non-empty-string>> */ public static function getSubscribedEvents(): array { return [ [ 'event' => 'serializer.post_deserialize', 'method' => 'onPostDeserialize', ] ]; } /** * @api */ public function onPostDeserialize(ObjectEvent $event): void { $object = $event->getObject(); if (!\is_object($object)) { return; } $reflection = new \ReflectionClass($object); foreach ($reflection->getProperties() as $property) { if ($property->isInitialized($object)) { continue; } $context = $event->getContext(); $factory = $context->getMetadataFactory(); $message = \vsprintf('Missing required [%s] property', [ \implode('.', [ ...$context->getCurrentPath(), $this->getPropertyName($object, $factory, $property), ]), ]); throw new UnprocessableEntityHttpException($message); } } /** * @return non-empty-string|null */ private function getPropertyName( object $context, MetadataFactoryInterface $factory, \ReflectionProperty $property, ): ?string { $class = $factory->getMetadataForClass($context::class); if (!$class instanceof ClassMetadata) { return null; } $metadata = $class->propertyMetadata[$property->name] ?? null; /** @var non-empty-string */ return match (true) { $metadata instanceof JMSPropertyMetadata => $metadata->serializedName ?? $metadata->name, $metadata instanceof PropertyMetadata => $metadata->name, default => $property->name, }; } }

Я бы так и сделал. Сейчас есть ограничение, что при десериализации из метадаты создается объект, вот такая хрень public function construct( DeserializationVisitorInterface $visitor, ClassMetadata $metadata, $data, array $type, DeserializationContext $context ): ?object { $className = $metadata->name; return new $className(); }

зачем? о_0

удали нафиг

удали нафиг

Уже пробовал, все попадало, видимо какой-то костыль

Уже пробовал, все попадало, видимо какой-то костыль

Проект не твой?

Проект не твой?

Рабочий, гляну потом может получится это убрать. Там скорее всего при десериализации вызывается конструктор, чтобы инициализировать некоторые поля дефолтными значениями, если убрать то через рефлексию будет объект создаваться

в JMS опция есть

jms_serializer: default_value_property_reader_support: true

наркоманы....

jms_serializer: default_value_property_reader_support: true

Эти проперти в конструкторе заполняются, не уверен, что это сработает. Что-то типа new DateTime() и другое

final readonly class UserRequestDTO { public function __construct( #[SerializedName('name')] public string $login, public string $password, ) {} }

У меня тоже та и все работает, пхпстан доволен

Точнее public readonly

Симфони сериал затор нормально в такой класс пихает значения

Симфони сериал затор нормально в такой класс пихает значения

оп, по симфе сериалчик вышел?

народ подскажите а как исключения перехватывать в симфони консоли ? Можно как-то глобально сделать ?

народ подскажите а как исключения перехватывать в симфони консоли ? Можно как-то глобально сделать ?

да, на ConsoleEvents::ERROR подпишись

а как теперь тип исключения получить ?

а как теперь тип исключения получить ?

тип?

Exception, DomainException и т.д.

Exception, DomainException и т.д.

https://github.com/symfony/symfony/blob/7.1/src/Symfony/Component/Console/Event/ConsoleErrorEvent.php вызвать getError

да понял

просто в try catch понятное всплытие а тут надо будет подумать -)

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

Так it это вцелом про подумать. Но на самом деле вроде странное решение вешать это на листенеры. Если обработка прям однотипная - то да. Но ты рискуешь отлавливать исключения по вообще всем командам

да я хочу отправлять все критические exception в sentry

и хочу это делать глобально

да я хочу отправлять все критические exception в sentry

а, стой, сентри?

так оно само может это делать, там в настройках есть перехват исключений

так он стоит, но почему то не хватает у консоли симфони

https://docs.sentry.io/platforms/php/guides/symfony/configuration/symfony-options/

у нас к сожалению не симфони, а только некоторые модули оттуда (

кстати только понял что если исключение нигде не обрабатывается то sentry его перехыватывает, но если исключение обрабатывается то нет

кстати только понял что если исключение нигде не обрабатывается то sentry его перехыватывает, но если исключение обрабатывается то нет

Там листенер стоит, но можно свой написать и отправлять что нужно

кстати только понял что если исключение нигде не обрабатывается то sentry его перехыватывает, но если исключение обрабатывается то нет

https://docs.sentry.io/platforms/php/guides/symfony/#monolog-integration

Там листенер стоит, но можно свой написать и отправлять что нужно

а как он отправит если исключение отловлено? Или есть php хук который сам отлавливает даже то catch?

а как он отправит если исключение отловлено? Или есть php хук который сам отлавливает даже то catch?

Ну в сентри есть функции для отправления. Он же не прибит к симфе или монологу

Ну в сентри есть функции для отправления. Он же не прибит к симфе или монологу

ааа, типо в каждом месте где он сам делает catch делать ещё и отправку

ааа, типо в каждом месте где он сам делает catch делать ещё и отправку

Нет, 1 листенер

проблема то просто не в этом

кстати только понял что если исключение нигде не обрабатывается то sentry его перехыватывает, но если исключение обрабатывается то нет

Вам тогда надо разобрать что такое критикал и не критикал исключения. Опять же еррор хэндлер может быть центральный. В общем тут уже детали и как у вас сделано. Может вы в каждом контроллере ловите исключения и глушите и какие то хотите отправлять а какие то нет, тут уже конечно центально не сделать

Да вот как раз сейчас разгребаю это, логи и ошибки

Там как обычно часть пишется в файл, часть в ментри, какие-то на почту

так он стоит, но почему то не хватает у консоли симфони

Само по себе ничего не происходит. Вам нужно или интеграцию с монологом, который будет слать в цетри все критикалы. Или же например ивенты консоли ловить (выше уже говорили)

а ещё сентри любит задваивать/затраивать исключения

Я понял одно, мне не нравятся логи в файлах

Сейчас делаю в мантикору

вылетело одно исключение, а в сам сентри написалось 3 сообщения из разных контекстов

Сейчас делаю в мантикору

Ну вот опять же, в файл или в stdout - это локально. Отправлять по сети - это сетевые накладки и возможные проблемы

Вроде как если по правильному (хотя все отсносительно), сначало куда то локально (докер лог, файл), а уже потом пушить по сетке батчами отдельными сервисами

У нас мантикора локально поднята

Ну вот опять же, в файл или в stdout - это локально. Отправлять по сети - это сетевые накладки и возможные проблемы

Можно тротлить и отправлять не всё или писать не в сеть а по rpc в сайдкар

Можно тротлить и отправлять не всё или писать не в сеть а по rpc в сайдкар

Ну это да, особенно с центри полезно

Можно тротлить и отправлять не всё или писать не в сеть а по rpc в сайдкар

Но опять же, тротлить имеет смысл при отправке исключений, ака центри, для логов решение не очень. Смысл от кусочных логов, если они на то и нужны, чтобы они были полностью, когда понадобятся

А то у сентри есть неприятная особенность, она отправляет всё на шаге терминейт. Так вот если сентри не отвечает то воркеры конечно же обработают запрос, но после обработки будут висеть секунд 30 без дела и в итоге обработка запросов, очередей, журналов встанет

Логировать можно в stderr и пусть инфра это всё забирает. В файл не стоит, особенно если stateless контейнеры.

Но опять же, тротлить имеет смысл при отправке исключений, ака центри, для логов решение не очень. Смысл от кусочных логов, если они на то и нужны, чтобы они были полностью, когда понадобятся

Ну я про сентри и говорю, речь же про неё

Ну я про сентри и говорю, речь же про неё

Ну ТС про логи в файлы писал

Т.е. переключился с стектрейса ошибок на логирование

Приветствую. При генерации миграций Доктрина создаёт всратые названия у констреинтов с foreign key, а-ля FK_ABC19359135666. Мне это не нравится, и я переименовываю их во что-то более читабельное и понятное мне. Однако при последующих запусках bin/console do:mi:diff Доктрина упорно переименовывает всё обратно. Например: $this->addSql('ALTER TABLE shop DROP CONSTRAINT shop_player_id_fk'); $this->addSql('ALTER TABLE shop ADD CONSTRAINT FK_AC6A4CA299E6F5DF FOREIGN KEY (player_id) REFERENCES player (id) NOT DEFERRABLE INITIALLY IMMEDIATE'); Есть какой-нибудь способ попросить Доктрину так не делать?

Индексы добавляю пока по-старинке, через аннотации: /** * @ORM\Table( * name="shop", * indexes={ * @ORM\Index(name="shop_player_id_idx", columns={"player_id"}), * } * @ORM\Entity() */ class Shop implements VersionAwareInterface UniqueIndex-ы добавляю так же при необходимости. Однако не нашёл аналогично способа, чтобы задать название для FK

Индексы добавляю пока по-старинке, через аннотации: /** * @ORM\Table( * name="shop", * indexes={ * @ORM\Index(name="shop_player_id_idx", columns={"player_id"}), * } * @ORM\Entity() */ class Shop implements VersionAwareInterface UniqueIndex-ы добавляю так же при необходимости. Однако не нашёл аналогично способа, чтобы задать название для FK

да, я увидел что это фк, у него вроде тоже есть отдельный способ декларирования

да, я увидел что это фк, у него вроде тоже есть отдельный способ декларирования

Вот чёт я гуглил-гуглил, да не нашёл, к сожалению