Size: a a a

2020 July 21

В

Виктор in Yii Framework 3
Dmitriy S
Я не о том, там вообще параметров нет, то в примере в пакете ошибка. Я о том, что добавление параметров в метод интерфейса не нарушает интерфейс, хотя и делает легкую несовместимость, но поскольку интерфейс внутренний, то все ок.
Либо я тебя не так понял, либо так не прокатит. Добавил в execute параметр, теперь при создании объекта ожидаемо получаю ошибку.
PHP Fatal error: Declaration of rssBot\queue\jobs\SourceFetchJob::execute(Psr\EventDispatcher\EventDispatcherInterface $dispatcher): void must be compatible with Yiisoft\Yii\Queue\Job\JobInterface::execute()
источник

В

Виктор in Yii Framework 3
Оформил вопрос в тикет, нужны мнения. Мне больше нравится первый вариант: https://github.com/yiisoft/yii-queue/issues/51
источник

DS

Dmitriy S in Yii Framework 3
Виктор
Либо я тебя не так понял, либо так не прокатит. Добавил в execute параметр, теперь при создании объекта ожидаемо получаю ошибку.
PHP Fatal error: Declaration of rssBot\queue\jobs\SourceFetchJob::execute(Psr\EventDispatcher\EventDispatcherInterface $dispatcher): void must be compatible with Yiisoft\Yii\Queue\Job\JobInterface::execute()
Дефолное значение null ему поставь
источник

DS

Dmitriy S in Yii Framework 3
Еще можно упаковать все зависимости в JobService и инжектить его одним параметром, а потом доставать из него $jobService->getSourceFetcher();
источник

В

Виктор in Yii Framework 3
Ну... Не комильфо, как по мне. Неоднозначно.
А вообще, будет здорово, если опишешь плюсы именно такого подхода в тикете по ссылке выше.
источник

DS

Dmitriy S in Yii Framework 3
Виктор
Ну... Не комильфо, как по мне. Неоднозначно.
А вообще, будет здорово, если опишешь плюсы именно такого подхода в тикете по ссылке выше.
Та я пока только варианты предлагаю)
источник

В

Виктор in Yii Framework 3
Dmitriy S
Та я пока только варианты предлагаю)
Я их из чата собрал и в тикете консолидировал. Глянь, как время будет, пожалуйста 😊
источник

DS

Dmitriy S in Yii Framework 3
Виктор
Я их из чата собрал и в тикете консолидировал. Глянь, как время будет, пожалуйста 😊
Ок
источник

А

Алексей R in Yii Framework 3
Виктор
Я их из чата собрал и в тикете консолидировал. Глянь, как время будет, пожалуйста 😊
мой последний запиши, чтоли )
источник

А

Алексей R in Yii Framework 3
ну хотя да, оно совместимо с тм, что ты написал про Message
источник

В

Виктор in Yii Framework 3
Алексей R
мой последний запиши, чтоли )
Я тебя не так понял) Я думал, что третьим вариантом твой записал 😄
источник

А

Алексей R in Yii Framework 3
Виктор
Я тебя не так понял) Я думал, что третьим вариантом твой записал 😄
больше похоже на то, что предлагается реализовать Serializable интерфейс
источник

В

Виктор in Yii Framework 3
Алексей R
больше похоже на то, что предлагается реализовать Serializable интерфейс
Угумс. Я совершенно неверно тебя понял)
источник

DS

Dmitriy S in Yii Framework 3
Виктор
Угумс. Я совершенно неверно тебя понял)
Там можно прям сейчас сделать какой-нить JobEventInterface, чекать в воркере и если он, то диспетчить задание как ивент, а если нет - выполнять. И будет работать и так и так.
источник

Д

Дмитрий in Yii Framework 3
Так я же в том году создавал тикет, в котором писал про практически это...
источник

В

Виктор in Yii Framework 3
Дмитрий
Так я же в том году создавал тикет, в котором писал про практически это...
Значит, мы единомышленники)
источник

А

Алексей R in Yii Framework 3
Дмитрий
Так я же в том году создавал тикет, в котором писал про практически это...
рефни
источник

AM

Alexander Makarov in Yii Framework 3
Дмитрий
Так я же в том году создавал тикет, в котором писал про практически это...
И где он?
источник

Д

Дмитрий in Yii Framework 3
🤷‍♂
источник

Д

Дмитрий in Yii Framework 3
Может, я в слак писал...
источник