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

Size: a a a

Yii Framework 3

2020 June 01

public function testConstructWithoutOrder(): void
{
$aliases = new Aliases([
'@gii' => '@yii/gii',
'@yii' => '/yii',
]);

$this->assertEquals('/yii/gii', $aliases->get('@gii'));
}

источник

21:34пожаловаться #1

Dmitriy S in Yii Framework 3

TradersVE

in commons work fine.

If it doesn't work then this is Aliases bug

источник

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

Alexander Makarov in Yii Framework 3

yes

источник

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

Alexander Makarov in Yii Framework 3

well, kind of

источник

21:34пожаловаться #4

Alexander Makarov in Yii Framework 3

it resolves sub-aliases in value in set()

источник

21:35пожаловаться #5

Alexander Makarov in Yii Framework 3

so order matters

источник

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

TradersVE in Yii Framework 3

it is best to define aliases with the package name in common.

источник

21:35пожаловаться #7

Alexander Makarov in Yii Framework 3

likely not a good thing

источник

21:35пожаловаться #8

TradersVE in Yii Framework 3

unique names unless you want to overwrite

источник

21:36пожаловаться #9

TradersVE in Yii Framework 3

problem solved

источник

21:36пожаловаться #10

TradersVE in Yii Framework 3

'aliases' => [
'@Yiisoft/Yii/Db/Migration' => dirname(__DIR__),
],

источник

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

TradersVE in Yii Framework 3

https://github.com/yiisoft/yii-db-migration/pull/30

GitHub

Enhacements by terabytesoftw · Pull Request #30 · yiisoft/yii-db-migration

Q
A

Is bugfix?
❌

New feature?
✔️

Breaks BC?
❌

Tests pass?
✔️

rework aliases.
change view for webview (always defined in the application).
add final class create migrations.

источник

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

Alexander Makarov in Yii Framework 3

Now order does not matter in aliases https://github.com/yiisoft/aliases/pull/10

GitHub

Fix order dependency by samdark · Pull Request #10 · yiisoft/aliases

In case sub-alias is defined before root, an exception is thrown. It should work no matter what order aliases provided in.

Q
A

Is bugfix?
✔️

New feature?
❌

Breaks BC?
✔️

Tests pass?
❌
...

источник

23:54пожаловаться #13

Alexander Makarov in Yii Framework 3

Going to add some more tests to it. Test set isn't good enough.

источник

23:55пожаловаться #14

Alexander Makarov in Yii Framework 3

TradersVE

https://github.com/yiisoft/yii-db-migration/pull/30

GitHub

Enhacements by terabytesoftw · Pull Request #30 · yiisoft/yii-db-migration

Q
A

Is bugfix?
❌

New feature?
✔️

Breaks BC?
❌

Tests pass?
✔️

rework aliases.
change view for webview (always defined in the application).
add final class create migrations.

Left some comments.

источник

23:58пожаловаться #15

2020 June 02

TradersVE in Yii Framework 3

Alexander Makarov

Left some comments.

done.

источник

00:02пожаловаться #16

Alexander Makarov in Yii Framework 3

TradersVE

done.

✅

источник

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

Alexander Makarov in Yii Framework 3

Хочу нарелизить https://github.com/yiisoft/aliases. Тесты поднял до 100%, также прогнал на мутационное тестирование. Мутантов убежавших нет. Единственное, не уверен, нужен ли метод getRoot().

GitHub

yiisoft/aliases

Named paths and URLs storage. Contribute to yiisoft/aliases development by creating an account on GitHub.

источник

02:32пожаловаться #18

Alexander Makarov in Yii Framework 3

Что думаете?

источник

02:32пожаловаться #19

TradersVE in Yii Framework 3

Root can be any alias

источник

02:41пожаловаться #20