NO
Переименовать в isLoggedIn либо isAuthenticated и оставить в Authentication
Size: a a a
2021 January 15
NO
И похоже это деление не поможет этому классу. Этот класс предоставляет методы для работы с текущим пользователем.
Нет
СП
Нет
Почему нет?
СП
В нём хранится текущий identity для него же идёт проверка прав и он же разлогинивается и т. д.
А
В нём хранится текущий identity для него же идёт проверка прав и он же разлогинивается и т. д.
Вот это и надо фиксить
СП
Вот это и надо фиксить
но где-то же это всё равно придётся хранить для удобства использования
А
но где-то же это всё равно придётся хранить для удобства использования
В том месте, которого у нас пока нет
А
Или не тем способом, как сейчас
СП
Вообще у нас весь этот пакет подразумевает работу с текущим пользователем именно, это как бы оболочка над аутентификацией и проверкой прав. Для этого есть другие пакеты.
СП
Authentication и AccessChecker в пакете yiisoft/user будут работать с текущим пользователем и обращаться к пакетам yiisoft/auth и yiisoft/access.
DS
Authentication и AccessChecker в пакете yiisoft/user будут работать с текущим пользователем и обращаться к пакетам yiisoft/auth и yiisoft/access.
Тогда логично будет эти два класса в них и вынести, а пакет юзер дропнуть
СП
Тогда логично будет эти два класса в них и вынести, а пакет юзер дропнуть
возможно, да
NO
NO
если есть auth и access то в них и вынести)
2021 January 17
AP
Он там в предыдущих строчкахин инжектится. У тебя интерфейс диспетчера не определен в контейнере.
То есть без контейнера все таки роутер нельзя никак использовать?
DS
Arman Poghosyan
То есть без контейнера все таки роутер нельзя никак использовать?
Там диспетчер, а не контейнер
AP
Там диспетчер, а не контейнер
Ок, всё виже, то есть просто надо $matchingResult->withDispatcher($dispatcher) делать
AP
Тогда он инжектится в роут
DS
Arman Poghosyan
Ок, всё виже, то есть просто надо $matchingResult->withDispatcher($dispatcher) делать
Это вариант с ленивым добавлением диспетчера. Можно вместо этого указать диспетчер на корневую группу.
AP
Это вариант с ленивым добавлением диспетчера. Можно вместо этого указать диспетчер на корневую группу.
Угу, то есть везде в принципе можно подставлять свой диспатчер, как я тогда планировал в своем PR, это гуд. Остается мне понять, что там еще было у меня, что может быть полезно поменять