В рейтинге участвует:

групп:

каналов:

Виртуальный сервер на SSD - недорого!

Аренда выделенных и виртуальных серверов (VDS/VPS), хостинг, аренда IP-адресов, администрирование, круглосуточная поддержка

qwarta.ru подробнее

Резервное копирование с проверкой на вирусы!!!

Удобный сервис создания резервных копий на любой сервер сети интернет. Отслеживайте изменения, проверяйте на вирусы. Надежно защитите свой бизнес!

go.backupland.com

Выбираете сервер? Любая конфигурация на заказ!

Аренда физических серверов любых конфигураций под любые запросы - 1С бухгалтерия, игровые сервера, нагруженные проекты, интернет-магазины!

qwarta.ru подробнее

Size: a a a

Yii Framework 3

334 membersпожаловаться на группу
2020 April 04

AM

Alexander Makarov in Yii Framework 3
Дмитрий
я решил начать заново свой рефакторинг плагина. буду кидать немного и много, ок?
желательно сигнализировать сразу, если какой-то ПР не нравится, ибо каждый следующий будет завязан на предыдущем для удобной разработки
Хорошо.
источник
12:39пожаловаться#1

AM

Alexander Makarov in Yii Framework 3
Dmitriy S
@samdark, а чего ты против того, чтобы затянуть ContainerIntereface в пакет ивент диспетчера?
Так бы можно было передавать массив листенеров в конструкторе, а функционал добавления перенести из EventConfigurator, а его выпилить.
Потому что ему не надо знать про контейнер.
источник
12:39пожаловаться#2

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
Потому что ему не надо знать про контейнер.
Ну в общем сделал без него, можешь смотреть
источник
12:42пожаловаться#3

Д

Дмитрий in Yii Framework 3
Dmitriy S
Сделал
https://github.com/yiisoft/event-dispatcher/pull/20
GitHub
Add AbstractProviderConfigurator by yiiliveext · Pull Request #20 · yiisoft/event-dispatcher
Q
A




Is bugfix?
❌


New feature?
❌


Breaks BC?
✔️


Tests pass?
✔️
добавляй reviewers к ревью)
источник
12:43пожаловаться#4

DS

Dmitriy S in Yii Framework 3
Дмитрий
добавляй reviewers к ревью)
Так я на нычку, а то поприходят всякие, а мне потом код переписывать😏
источник
12:45пожаловаться#5

Д

Дмитрий in Yii Framework 3
я вообще не понял твой пр.
добавлять нельзя, а удалять можно? тогда давай и открепление уберем.
тесты тоже не очень. нет смысла делать оверрайд над конфигуратором, если ты тестируешь не его.
тестируешь провайдер - тестируй провайдер.
тем более переопределяешь protected => public, тогда ради чего затеивалась абстракция?
источник
12:48пожаловаться#6

DS

Dmitriy S in Yii Framework 3
А блин, забыл модификатор поменять)
источник
12:49пожаловаться#7

Д

Дмитрий in Yii Framework 3
если это AbstractProviderConfigurator, то почему он не абстрактный?
источник
12:49пожаловаться#8

DS

Dmitriy S in Yii Framework 3
Дмитрий
если это AbstractProviderConfigurator, то почему он не абстрактный?
Потому что в субботу утром работать - грех, шаббат однако😝
Пофиксил все.
источник
12:51пожаловаться#9

Д

Дмитрий in Yii Framework 3
@hiqsol подскажи, что за Alternatives? много упоминаний в коде, но я так и не понял
источник
12:57пожаловаться#10

Д

Дмитрий in Yii Framework 3
насколько я знаю, у нас ни разу такое не встречалось. есть желание снести эту возможность, т.к. она не нужна была за всё время использования.
если нужна будет в дальнейшем, то сделаем и тестами и прочим
источник
12:59пожаловаться#11

DS

Dmitriy S in Yii Framework 3
Дмитрий
я вообще не понял твой пр.
добавлять нельзя, а удалять можно? тогда давай и открепление уберем.
тесты тоже не очень. нет смысла делать оверрайд над конфигуратором, если ты тестируешь не его.
тестируешь провайдер - тестируй провайдер.
тем более переопределяешь protected => public, тогда ради чего затеивалась абстракция?
Абстракция чтобы не вызывали методы attach/detach у объекта, полученного через интерфейс. Потому как в интерфейсе этих методов нет.
источник
13:01пожаловаться#12

AV

Andrii Vasyliev in Yii Framework 3
alternatives - крутая вещь и у нас используется в продакшенах в разных проектах, она нужна!
источник
13:01пожаловаться#13

Д

Дмитрий in Yii Framework 3
расскажешь, что это такое?)
источник
13:01пожаловаться#14

AV

Andrii Vasyliev in Yii Framework 3
это возможность собрать много альтернативных конфигов на базе основного
источник
13:02пожаловаться#15

AV

Andrii Vasyliev in Yii Framework 3
щас подробнее опишу
источник
13:02пожаловаться#16

AV

Andrii Vasyliev in Yii Framework 3
у нас есть проекты (сайты, панели управления, апи) под кучу разных реселлеров, в принципе проект один и тот же но у каждого реселлера свои настройки: названия сайтов, урлы, емейлы, настроки интеграций и т.д. все альтернативы обслуживаются с одного виртхоста в entry script'е такой код:
источник
13:08пожаловаться#17

AV

Andrii Vasyliev in Yii Framework 3
источник
13:08пожаловаться#18

AV

Andrii Vasyliev in Yii Framework 3
в composer.json такое:
источник
13:10пожаловаться#19

AV

Andrii Vasyliev in Yii Framework 3
источник
13:10пожаловаться#20