Size: a a a

2020 August 17

Д

Дмитрий in Yii Framework 3
Arman давай в следующий раз хотя бы на порядок поменьше изменений. Не представляю, как это ревьювить.
Если есть возможность, то сделай итерациями, так быстрее пройдет процесс ревью.
источник

RM

Rustam Mamadaminov in Yii Framework 3
как я и предполагал :)
источник

DS

Dmitriy S in Yii Framework 3
Alexander Makarov
@yiiliveext я верно помню?
Группа не иммутабельна, поэтому с ней могут быть проблемы при работе с рр
источник

DS

Dmitriy S in Yii Framework 3
Дмитрий
Arman давай в следующий раз хотя бы на порядок поменьше изменений. Не представляю, как это ревьювить.
Если есть возможность, то сделай итерациями, так быстрее пройдет процесс ревью.
Та пусть до конца у себя пакет сделает, так будет проще его покрутить
источник

RM

Rustam Mamadaminov in Yii Framework 3
источник

DS

Dmitriy S in Yii Framework 3
Arman Poghosyan
Добавил очень много буков в описании к PR-у по Роутеру, что поможет понять контекст всех этих изменений. Также в описании есть некоторые вопросы и мысли, которые я написал для себя, но буду рад, если поможете и с ними. По мере новой информации опять буду обновлять описание.
А группы там вообще предполагаются?
источник

DS

Dmitriy S in Yii Framework 3
Да, я смотрел уже. А зачем тебе атрибут _host?
источник

RM

Rustam Mamadaminov in Yii Framework 3
Dmitriy S
Да, я смотрел уже. А зачем тебе атрибут _host?
Без этого никак, потому что постоянно матчица с host + path https://github.com/yiisoft/router-fastroute/pull/30/files#diff-ed12989ca60f9bcd667d029622127944R117
источник

DS

Dmitriy S in Yii Framework 3
а, понял
источник

AP

Arman Poghosyan in Yii Framework 3
Я так не планировал, но в итоге получилось так. Маленькими PR никак, так как по сути, там сразу многое поменялось и так легче. Ну теоретически может и возможно по частям, но это будет страшно на части разбивать. Я же говорил не по Files Changed смотреть, так как без смысла (имена и структура пакета тоже изменилась), лучше сразу по ветке смотреть, а PR я открыл для удобства, чтоб оставлять там комментарии можно было.
источник

AP

Arman Poghosyan in Yii Framework 3
Dmitriy S
А группы там вообще предполагаются?
Да, доделаю сегодня недостающее.
источник

DS

Dmitriy S in Yii Framework 3
Arman Poghosyan
Я так не планировал, но в итоге получилось так. Маленькими PR никак, так как по сути, там сразу многое поменялось и так легче. Ну теоретически может и возможно по частям, но это будет страшно на части разбивать. Я же говорил не по Files Changed смотреть, так как без смысла (имена и структура пакета тоже изменилась), лучше сразу по ветке смотреть, а PR я открыл для удобства, чтоб оставлять там комментарии можно было.
Хендлеры наружу торчат, можно в любом порядке их выполнять
источник

AP

Arman Poghosyan in Yii Framework 3
Dmitriy S
Хендлеры наружу торчат, можно в любом порядке их выполнять
То есть? Ну можно если прям захотеть. Dispatcher решает, как их выполнять. Если кому-то прям приспичит их в другом порядке исполнять или вообще не исполнять в своем диспатчере, то так тому и быть. Или ты не про это?
источник

AP

Arman Poghosyan in Yii Framework 3
Диспатчер по умолчанию все делает правильно вроде.
источник

DS

Dmitriy S in Yii Framework 3
Arman Poghosyan
То есть? Ну можно если прям захотеть. Dispatcher решает, как их выполнять. Если кому-то прям приспичит их в другом порядке исполнять или вообще не исполнять в своем диспатчере, то так тому и быть. Или ты не про это?
Именно про это. Как бы смысл middlewares теряется, получается слишком много свободы для всяких нехороших штук
источник

AP

Arman Poghosyan in Yii Framework 3
Dmitriy S
Именно про это. Как бы смысл middlewares теряется, получается слишком много свободы для всяких нехороших штук
Ну я типо не пойму, почему если кому-то прям нужны нехорошие штуки - не дать эту возможность. Нехорошее можно сделать почти в любом месте приложения/кода/фреймворка, если захотеть.
источник

А

Алексей R in Yii Framework 3
Arman ты Apple юзаешь?
источник

AP

Arman Poghosyan in Yii Framework 3
Оно так будет в любом раскладе. Если даже handling/processing убрать вообще из пакета и оставить только matching и собирание маршрутов (как в laminas-router и symfony), то надо будет и возможность ставить middleware-и для маршрутов тоже убирать, чтоб прям никто никак этим не злоупотреблял.
источник

AP

Arman Poghosyan in Yii Framework 3
Алексей R
Arman ты Apple юзаешь?
Нет, Win10 в основном, мне пакет Adobe часто нужен. Mac купил жене, чтоб на нем компиляции для ios приложений делать))) Если ты про телефон, то опять нет - андроид))) Мне яблоко как-то неудобно.
источник

DS

Dmitriy S in Yii Framework 3
Arman Poghosyan
Оно так будет в любом раскладе. Если даже handling/processing убрать вообще из пакета и оставить только matching и собирание маршрутов (как в laminas-router и symfony), то надо будет и возможность ставить middleware-и для маршрутов тоже убирать, чтоб прям никто никак этим не злоупотреблял.
ну может быть и так
источник