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

оказывается в функции sort первым аргументом идёт не array, а arg https://www.php.net/manual/en/function.sort.php

16:33пожаловаться #1

у count() это var... вот и пиши тесты на internal функции

16:34пожаловаться #2

anton in Yii Framework 3

Алексей R

оказывается в функции sort первым аргументом идёт не array, а arg https://www.php.net/manual/en/function.sort.php

Всмысле в сорцах arg?

16:39пожаловаться #3

anton

Всмысле в сорцах arg?

рефлексия

16:39пожаловаться #4

рефлексия говорит, что имя первого аргумента arg

16:39пожаловаться #5

Dmitriy S in Yii Framework 3

Это что за новость?

16:49пожаловаться #6

я ничё не ломал )

16:50пожаловаться #7

у меня норма отработало

16:51пожаловаться #8

https://github.com/yiisoft/injector/pull/7

GitHub

Simplify invoke() by samdark · Pull Request #7 · yiisoft/injector

\Closure::fromCallable() is used instead of callable type checks.
Since we already have an instance of reflection we may call it instead of using \call_user_func_array().

Q
A

Is bugfix?
❌

...

17:46пожаловаться #9

https://github.com/yiisoft/injector/pull/7

Alexander Makarov

GitHub

Simplify invoke() by samdark · Pull Request #7 · yiisoft/injector

\Closure::fromCallable() is used instead of callable type checks.
Since we already have an instance of reflection we may call it instead of using \call_user_func_array().

Q
A

Is bugfix?
❌

...

У меня примерно тоже самое сейчас в ветке контекста

17:48пожаловаться #10

referenced-arguments?

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

Нет, я не пушил ещё.

17:49пожаловаться #12

Я написал под это phpbench, но он ничего дельного не выдал.

17:50пожаловаться #13

Потом сравнил opcode-ы и в новом варианте их ну прям сильно меньше

17:50пожаловаться #14

Это все равно будет нужно делать для контекста

17:50пожаловаться #15

Added support of arguments to be passed by reference by roxblnfk · Pull Request #6 · yiisoft/injector

дока на https://github.com/yiisoft/injector/pull/6 не очень нужна

GitHub

$foo = [1, 5, 7, 2, 0];

$result = (new Injector($container))->invoke('sort', ['arg' => &$foo]);

$sorted = $foo === [0, 1, 2, 5, 7]; // true

Tests
Docs?

17:51пожаловаться #16

Единственное, ты убрал первое условие. Не уверен, что под него даже тесты есть...

17:51пожаловаться #17

это ожидается, в общем, что reference-ы будут работать

17:51пожаловаться #18

Там важно понять, что надо аргумент по ссылке передать

17:51пожаловаться #19

Алексей R

Единственное, ты убрал первое условие. Не уверен, что под него даже тесты есть...

ты про __invoke?