Телеграмм чат группы yii3ru страница 2020

Size: a a a

Yii Framework 3

494 membersпожаловаться на группу

2020 September 27

Дмитрий in Yii Framework 3

https://github.com/yiisoft/factory/pull/45
а вот и багфикс

GitHub

Added temporary solution for container bugs by xepozz · Pull Request #45 · yiisoft/factory

Q
A

Is bugfix?
✔️

New feature?
❌

Breaks BC?
❌

Fixed issues

Added temporary solution for container bugs
yiisoft/di#162
yiisoft/di#161

источник

00:15пожаловаться #1

Дмитрий in Yii Framework 3

Andrii Vasyliev

> has возвращает true, но на самом деле в контейнере еще нет сконфигурированного инжектора.
> Это бага контейнера, которую никто чинить не хочет
В чём бага? has возвращает то что и должен возвращать в соответствии с описаним psr интерфейса

Я интерпретирую это как баг, потому что сейчас наш контейнер - хитрый проныра. Если у него нет знания о запрашиваемом ID, то он начинает автоматически резолвить это.
Такое поведение указано в PSR?

Проще говоря, вот этот тест должен фейлить, т.к. контейнер совсем пустой и ничего не должен знать.
А сейчас он начинает искать классы и пытается их создать.

источник

00:20пожаловаться #2

Дмитрий in Yii Framework 3

Хотел покодить, в итоге 3 часа дебажил эти контейнеры ваши :)

источник

00:21пожаловаться #3

Andrii Vasyliev in Yii Framework 3

в PSR написано что если контейнер может создать класс то has должен возвращать true, а именно так и работает наш контейнер, ему не обязательно иметь определение чтоб создать объект класса

источник

00:23пожаловаться #4

Andrii Vasyliev in Yii Framework 3

Дмитрий

Хотел покодить, в итоге 3 часа дебажил эти контейнеры ваши :)

то место которое ты нашёл - известно, что проблема из-за него, но это существенная фича
https://github.com/yiisoft/factory/pull/35
за неё можно побороться, а не бросать при первых же проблемах

GitHub

Extend `CallableDefinition` to allow more complex definitions by hiqsol · Pull Request #35 · yiisoft/factory

used injector to resolve callbacks with any number of arguments resolved automatically
added Reference::toCall($object, $method, $args)

This PR enables complex call definitions like:
$definitions ...

источник

00:27пожаловаться #5

Alexander Makarov in Yii Framework 3

Andrii Vasyliev

я так и подумал что идея такая. Только почему такая уверенность? Типа заинжектить как-то по другому совершенно невозможно?

Я считаю что так. Нам другого не надо. Ну и это не единственный случай. Не инжектор так ещё что-то.

источник

00:35пожаловаться #6

Andrii Vasyliev in Yii Framework 3

Вобщем других инжекторов или опций у инжектора не ожидается?

источник

00:36пожаловаться #7

Alexander Makarov in Yii Framework 3

Нет, не ожидается.

источник

00:37пожаловаться #8

Andrii Vasyliev in Yii Framework 3

тогда можно прибивать гвоздями

источник

00:37пожаловаться #9

Alexander Makarov in Yii Framework 3

Давай

источник

00:37пожаловаться #10

Alexey Samoylov in Yii Framework 3

Andrii Vasyliev

has MUST return true if an entry identifier is known to the container and false if it is not

источник

03:11пожаловаться #11

Alexey Samoylov in Yii Framework 3

Known - значит дефиниция в контейнер загружена

источник

03:11пожаловаться #12

Nurbek Suyunov in Yii Framework 3

Hi guys.How can i prepare advanced to product mode

источник

06:18пожаловаться #13

Andrii Vasyliev in Yii Framework 3

Alexey Samoylov

has MUST return true if an entry identifier is known to the container and false if it is not

что значит known? ;)

https://github.com/php-fig/container/blob/master/src/ContainerInterface.php#L23-L27

тут более явно написано
мы уже переделывали has по этому поводу

GitHub

php-fig/container

Contribute to php-fig/container development by creating an account on GitHub.