CWebUser полубаг-полупредложение

[Антон Смирнов]

В методе loginRequired() устанавливается урл для возврата:

Код: Выделить всё

  $this->setReturnUrl($request->getUrl());  

Если сайт использует поддомены, то необходимо получить текущий полный урл, а $request->getUrl() вернет только относительный, иначе после авторизации будет редирект на неправильный урл.

[samdark]

Поддомены не такая уж частая штука. Если они используются, чуть подпилить не сложно.

[Avgusto]

Поддомены не такая уж частая штука. Если они используются, чуть подпилить не сложно.

а как потом быть с обновлениями?

[mickgeek]

Следует расширять классы фреймворка и использовать уже их в своих проектах.

[Avgusto]

И то верно, у меня всё ещё CMS головного мозга. Пардоньте.

[Антон Смирнов]

Поддомены не такая уж частая штука. Если они используются, чуть подпилить не сложно.

ЭЭЭ.. Или переопределить метод в CWebUser или использовать свой CHttpRequest - это не подпилить.

Дело не в поддоменах, дело в том, что неправильно вычисляется текущий урл, это архитектурная ошибка. Ваши слова означают "забейте" или делайте пул. Странная логика. Yii1 уже похоронили, а yii2 до сих пор не понятно в каком статусе.

[anton44eg]

Как минимум - такое изменение будет ломать обратную совместимость

[Антон Смирнов]

Как минимум - такое изменение будет ломать обратную совместимость

Нет

[samdark]

Под подпилить я имел ввиду именно свой компонент, который перекрывает один из методов стандартного.

[samdark]

Но вообще да, можно и попрвить на getHostInfo() . '/' . getUrl(). Pull request сделаете?

[Антон Смирнов]

Но вообще да, можно и попрвить на getHostInfo() . '/' . getUrl(). Pull request сделаете?

Сделаю, если смогу победить GitHub.

" getHostInfo() . '/' . getUrl()" - это, на мой взгляд, тоже неправильно, лучше вынести в отдельную переменную или метод или анонимную функцию, не могу точно сформулировать, не гуру. И тогда не будет ломаться обратная совместимость и в то же время, будет легко настраиваться.