Size: a a a

2020 July 12

АН

Александр Некрасов... in Yii Framework 3
Как вариянт, добавить интерфейс, как говорил @xepozz
источник

AM

Alexander Makarov in Yii Framework 3
я и для интерфейса кейса не вижу
источник

AM

Alexander Makarov in Yii Framework 3
смёржил
источник

АН

Александр Некрасов... in Yii Framework 3
Alexander Makarov
я и для интерфейса кейса не вижу
Появится, добавим.
источник

AM

Alexander Makarov in Yii Framework 3
я чуть в парсеры влез... пушу коммиты
источник

АН

Александр Некрасов... in Yii Framework 3
https://github.com/yiisoft/yii-web/blob/master/src/ErrorHandler/ErrorCatcher.php#L43 может тоже поменять на "withRenderer" ?
источник

AM

Alexander Makarov in Yii Framework 3
да
источник
2020 July 13

Д

Дмитрий in Yii Framework 3
просто подмечу: переусердствуете с final.
если сейчас никто не может найти кейса для расширения, это абсолютно ничего не знает в глобальном масштабе.

это не только с Rules, это ко всем классам.
источник

Д

Дмитрий in Yii Framework 3
Roman Tsurkanu
Минус валидация и минус билдинг
Валидация не минус. Билдинг чего?
источник

AM

Alexander Makarov in Yii Framework 3
Дмитрий
просто подмечу: переусердствуете с final.
если сейчас никто не может найти кейса для расширения, это абсолютно ничего не знает в глобальном масштабе.

это не только с Rules, это ко всем классам.
Оставлять не final не вижу смысла если не придумывается ни один кейс.
источник

Д

Дмитрий in Yii Framework 3
Roman Tsurkanu
Контроллеры чище
Нет, как раз контролёры грязнее. Вместо объекта оперируют кучей переменных.
источник

AM

Alexander Makarov in Yii Framework 3
Alexander Makarov
Оставлять не final не вижу смысла если не придумывается ни один кейс.
А если придумывается, вероятно нужен интерфейс...
источник

AM

Alexander Makarov in Yii Framework 3
Дмитрий
Нет, как раз контролёры грязнее. Вместо объекта оперируют кучей переменных.
В этом согласен. Мне тоже не нравится ни в Symfony ни в Laravel эта фича.
источник

Д

Дмитрий in Yii Framework 3
A
Ну разграничение логики

Типа реквест занимается своим делом
контроллер своим
а моделька тупа уже сохраняет данные
В таком случае реквест = форма.
Реквест для передачи параметров внутрь приложения, больше от него не нужно.
К тому же, данные лучше сохранять не в модельке, а в каком-нибудь сервисе (а.к.а. репозитории).
А перед сохранением хорошо бы прогнать через процесс создания модели данных по образу входящих валидных данных (форма, дто).
Если всё сложить, то больше ничего и не нужно. Просто кто-то навязывает шорткаты, выдавая это за правильность.
Не всегда это нужно делать.
источник

Д

Дмитрий in Yii Framework 3
Alexander Makarov
Оставлять не final не вижу смысла если не придумывается ни один кейс.
если запрещает расширение через наследование, то обязательно нужно оставлять возможность расширения через интерфейс/абстрактный класс
источник

А

Алексей R in Yii Framework 3
Дмитрий
если запрещает расширение через наследование, то обязательно нужно оставлять возможность расширения через интерфейс/абстрактный класс
вот я того же мнения (привет пагинатор :) )
источник

Д

Дмитрий in Yii Framework 3
это мы сейчас думаем, что "будут кейсы, сделаем не final". людям будет пофигу, и они будут делать свои велосипеды/форки/прочее. 1 из 1000 возьмёт и поднимет тему, что крайне неудобно работать с %package%.
источник

Д

Дмитрий in Yii Framework 3
Алексей R
вот я того же мнения (привет пагинатор :) )
а что с пагинатором?
источник

А

Алексей R in Yii Framework 3
Дмитрий
а что с пагинатором?
OffsetPaginator final и без интерфейса
источник

А

Алексей R in Yii Framework 3
интерфейс там слишком общий на все пагинаторы
источник