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

Size: a a a

Yii Framework 3

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

2020 April 25

Алексей R in Yii Framework 3

я вот добавил тесты на исключения, а показатель не изменился

источник

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

Alexander Makarov in Yii Framework 3

построй PHPUnit-ом

источник

14:25пожаловаться #2

Dmitriy S in Yii Framework 3

Почему здесь User оперирует риспонсом? Вроде это еще пару недель назад обсуждали.
https://github.com/yiisoft/yii-web/pull/235/files#diff-d91a256d62d773d5f19b922b169dba7cR182

GitHub

AutoLoginMiddleware #218 by mapeveri · Pull Request #235 · yiisoft/yii-web

Q
A

Is bugfix?
❌

New feature?
✔️

Breaks BC?
❌

Tests pass?
✔️

This PR is incomplete, I have some questions:

I understand that the 'remember' cookie is not defined. Wher...

источник

14:26пожаловаться #3

Alexander Makarov in Yii Framework 3

Dmitriy S

GitHub

AutoLoginMiddleware #218 by mapeveri · Pull Request #235 · yiisoft/yii-web

Q
A

Is bugfix?
❌

New feature?
✔️

Breaks BC?
❌

Tests pass?
✔️

This PR is incomplete, I have some questions:

I understand that the 'remember' cookie is not defined. Wher...

Потому что человек не понял...

источник

14:28пожаловаться #4

Dmitriy S in Yii Framework 3

Как минимум там можно создать класс вроде LoggedIdentity с Identity и Cookie в конструкторе, и методами getAuthCookie() и getIdentity() и возвращать его в методе login() при успешном логине или null в противоположном. Хотя создание куки - это тоже не задача класса User.

источник

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

Alexander Makarov in Yii Framework 3

Dmitriy S

Да, это задача middleware.

источник

15:04пожаловаться #6

Alexander Makarov in Yii Framework 3

Я уже про это писал в комментах там. Но, видно, плохо пишу.

источник

15:05пожаловаться #7

Дмитрий in Yii Framework 3

по плагину:
т.к. мы переделываем на обязательный параметр baseDir в rebuild, то может стоит сделать обязательным и в path()?
если так, то тогда проще сделать

$builder =  new Builder($baseDir, $outputDir = null);
$builder->rebuild() / $builder->path($configName)

источник

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

Алексей R in Yii Framework 3

а ребилд не статик разве?

источник

16:47пожаловаться #9

Дмитрий in Yii Framework 3

статик

источник

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

Дмитрий in Yii Framework 3

если мы добавим обязательным параметр только к нему, то будет непонятки:
Builder::rebuild($baseDir)
и Builder::path($configName, $baseDir = null)

отсюда получается несогласованность - для пересоздания обязательно нужен путь, а для чтения не обязателен путь :)

источник

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

Дмитрий in Yii Framework 3

и везде писать
Builder::path($configName dirname(__DIR__)); очень костыльно

источник

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

Дмитрий in Yii Framework 3

@hiqsol что думаешь по этому?

источник

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

Дмитрий in Yii Framework 3

@yiiliveext не хочешь подрубить к консоли дебаггер? :)

источник

16:53пожаловаться #14

Dmitriy S in Yii Framework 3

Дмитрий

@yiiliveext не хочешь подрубить к консоли дебаггер? :)

можно, но зачем?

источник

16:54пожаловаться #15

Дмитрий in Yii Framework 3

чтобы можно было профилировать консоль, конечно же

источник

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

Дмитрий in Yii Framework 3

в демке офк

источник

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

Andrii Vasyliev in Yii Framework 3

Дмитрий

@hiqsol что думаешь по этому?

Также как и раньше: для нормальной ситуации (дефолтные пути к ведору и плагину) - работает без указания basedir, те у кого нестандартно - указывают basedir

источник

16:55пожаловаться #18

Dmitriy S in Yii Framework 3

Дмитрий

чтобы можно было профилировать консоль, конечно же

А, так это так и будет, он и для консоли подойдет

источник

16:55пожаловаться #19

Дмитрий in Yii Framework 3

Dmitriy S

А, так это так и будет, он и для консоли подойдет

уже работает?

источник

16:56пожаловаться #20