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

а, ну тогда PR сделай, конечно

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

чего два раза одно и то же фигачить...

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

У меня демо другое уже, на том чтов мастере отдельно менять нужно)

15:01пожаловаться #3

Но сделать могу, как тесты закончу на фастроут

15:02пожаловаться #4

ок, тогда я лучше сам

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

Fix route injection, add tests by yiiliveext · Pull Request #25 · yiisoft/router-fastroute

Пофиксил баг для RR и добавил тесты
https://github.com/yiisoft/router-fastroute/pull/25

GitHub

Q
A

Is bugfix?
✔️

New feature?
❌

Breaks BC?
❌

Tests pass?
✔️

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

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

RR?

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

Алексей R

RR?

RoadRunner

15:39пожаловаться #8

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

тчо был за баг?

15:40пожаловаться #9

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

я чёт походу прошляпил )

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

Алексей R

тчо был за баг?

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

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

Хотя мне и не нравится это решение, но пока не придумал как лучше сделать.

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

Может метод match сделать protected?

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

эм... зачем?

15:57пожаловаться #14

он же юзается как раз напрямую

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

https://github.com/yiisoft/router/blob/master/src/Middleware/Router.php#L25

GitHub

yiisoft/router

Router is a request matcher and URL generator. Contribute to yiisoft/router development by creating an account on GitHub.

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

вот

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

PR норм. Если ничего туда добавлять не планируется, солью.

15:59пожаловаться #18

Alexander Makarov

PR норм. Если ничего туда добавлять не планируется, солью.

Стой, есть один вопрос

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

Как вы файловую систему мокаете? Хочу тесты на кеш добить. vfsStream можно использовать?