Size: a a a

2020 February 27

DS

Dmitriy S in Yii Framework 3
https://github.com/yiisoft/assets/blob/master/src/AssetManager.php#L178
$this->publisher имеет тип AssetPublisher, а не AssetPublisherInterface, если поменять ему тип на интерфейс, то надо в интерфейс добавить метод loadBundle()
источник

AM

Alexander Makarov in Yii Framework 3
да, что-то там непорядок
источник

AM

Alexander Makarov in Yii Framework 3
Но нет, одним этим не отделаться.
источник

AM

Alexander Makarov in Yii Framework 3
Оно к AssetManager не зря прибито гвоздями.
источник

AM

Alexander Makarov in Yii Framework 3
Там и getAssetUrl и всё-всё.
источник

DS

Dmitriy S in Yii Framework 3
Значит надо что-то там отрефакторить
источник

DS

Dmitriy S in Yii Framework 3
@xepozz, зачем здесь выполнять три лишних операции? (array_values, распаковка массива, упаковка массива)
источник

DS

Dmitriy S in Yii Framework 3
источник

AM

Alexander Makarov in Yii Framework 3
Ссылку на код лучше давать, без контекста не ясно, где это...
источник

AM

Alexander Makarov in Yii Framework 3
Диспетчер?
источник

А

Алексей R in Yii Framework 3
тут потенциально лишний только array_values
источник

DS

Dmitriy S in Yii Framework 3
Алексей R
тут потенциально лишний только array_values
Я думаю что так оно быстрее работать будет
 public function getListenersForEvent(object $event): iterable
   {
       yield from $this->listenersFor([get_class($event)]);
       yield from $this->listenersFor(class_parents($event));
       yield from $this->listenersFor(class_implements($event));
   }

private function listenersFor(array $eventNames): iterable
   {
       foreach ($eventNames as $name) {
           if (isset($this->listeners[$name])) {
               yield from $this->listeners[$name];
           }
       }
   }
источник

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
Ссылку на код лучше давать, без контекста не ясно, где это...
источник

AM

Alexander Makarov in Yii Framework 3
А, так это не @xepozz сделал, если что.
источник

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
А, так это не @xepozz сделал, если что.
Может, я вроде раньше не видел этого провайдера вообще
источник

DS

Dmitriy S in Yii Framework 3
Какое-то у него название так себе
источник

AM

Alexander Makarov in Yii Framework 3
там вся разница что не юзается reflection
источник

AM

Alexander Makarov in Yii Framework 3
и класс события указывается явно
источник

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
там вся разница что не юзается reflection
Это я вижу. Так что там по getListenersForEvent, с массивом в качестве аргумента не быстрее оно будет?
источник

AM

Alexander Makarov in Yii Framework 3
🤷‍♂️
источник