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

Size: a a a

Yii Framework 3

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

2020 October 13

Дмитрий in Yii Framework 3

Открой онлайн регулярные выражения и потренеруйся :)

источник

09:35пожаловаться #1

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

Alexander Makarov

https://github.com/yiisoft/yii-demo/pull/157/files @roxblnfk конструктор не мешает Cycle?

GitHub

Setting Post title/content in constructor - #156 by brucealdridge · Pull Request #157 · yiisoft/yii-demo

Q
A

Is bugfix?
✔️

New feature?
❌

Breaks BC?
❌

Fixed issues
#156

А он примерно такой раньше и был, но кто-то поменял. Вроде бы

источник

09:36пожаловаться #2

СП

Сергей Предводителев... in Yii Framework 3

Дмитрий

Открой онлайн регулярные выражения и потренеруйся :)

Я к тому, что логика работы функции очень не явная, если не сказать не корректная

источник

09:37пожаловаться #3

СП

Сергей Предводителев... in Yii Framework 3

https://github.com/yiisoft/html/issues/23

GitHub

The implicit logic of Html::escapeJsRegularExpression · Issue #23 · yiisoft/html

Now function work with correct regexps: Html::escapeJsRegularExpression('/([a-z0-9-]+)/Ugimex'); // /([a-z0-9-]+)/gim Html::escapeJsRegularExpression('/mag/imgx'); /...

источник

09:42пожаловаться #4

Roman Tsurkanu in Yii Framework 3

Sergei Iamskoi

Окей, с постом и даннными все ок. Но если клиент запрашивает GET с

Content-Type: application/json

то

json_decode()

в парсере падает, т.к. body = ‘’. Наверное надо обработать такую ситцацию в парсере?

Заведешь issue?

источник

10:41пожаловаться #5

Sergei Iamskoi in Yii Framework 3

Roman Tsurkanu

Заведешь issue?

Да я думаю можно сразу PR сделать. В своем проекте уже сделали фикс.

источник

10:43пожаловаться #6

Sergei Iamskoi in Yii Framework 3

Roman Tsurkanu

Заведешь issue?

https://github.com/yiisoft/request-body-parser/pull/6

GitHub

Fix response for JsonParser, if $rawBody is empty string. by siamskoi · Pull Request #6 · yiisoft/request-body-parser

Q
A

Is bugfix?
✔️

New feature?
❌

Breaks BC?
❌

источник

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

Alexander Makarov in Yii Framework 3

Sergei Iamskoi

https://github.com/yiisoft/request-body-parser/pull/6

GitHub

Fix response for JsonParser, if $rawBody is empty string. by siamskoi · Pull Request #6 · yiisoft/request-body-parser

Q
A

Is bugfix?
✔️

New feature?
❌

Breaks BC?
❌

✅

источник

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

vitaly litvinov in Yii Framework 3

читал что возвращать null плохо, а почему здесь это норм?

источник

13:10пожаловаться #9

vitaly litvinov in Yii Framework 3

конкретно в этом issue

источник

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

Alexander Makarov in Yii Framework 3

потому что надо :)

источник

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

Alexander Makarov in Yii Framework 3

не всё то плохо, что обычно плохо

источник

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

Alexander Makarov in Yii Framework 3

есть исключения

источник

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

Roman Tsurkanu in Yii Framework 3

А почему возвращать null плохо?

источник

13:19пожаловаться #14

Alexander Makarov in Yii Framework 3

Roman Tsurkanu

А почему возвращать null плохо?

Потому что можно забыть на него проверить.

источник

13:21пожаловаться #15

Alexander Makarov in Yii Framework 3

В языках с сильно строгой типизацией типа Java это почти единственная и очень заметная проблема с типами.

источник

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

Roman Tsurkanu in Yii Framework 3

то есть плохо когда метод возврашает значение 2 типов?

источник

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

Дмитрий in Yii Framework 3

NullPointerException :)

источник

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

Alexander Makarov in Yii Framework 3

Roman Tsurkanu

то есть плохо когда метод возврашает значение 2 типов?

В общем случае да.

источник

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

Roman Tsurkanu in Yii Framework 3

понятно

источник

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