fix(photos): preserve source page after photo deletion

[mrksbgg]

если в настройках основной страницы стояла лента, при удалении фото со страницы, отправляло на /feed вместо страницы юзера из-за того, что /id0 отправляет в ленту

не знаю, является ли поведение отправки /id0 в ленту при значении Main page - My feed, вместо страницы пользователя (хз, как будто должно быть в любом случае id0 = текущий юзер, не?)

im so fucking confused

[fayzetwin1]

в целом все круто, но мне кажется стоит кое-что подправить

[fayzetwin1]

я бы предпочел брать путь более строгим путем или проверять регуляркой типа:
!str_starts_with($returnTo, '//') && !str_starts_with($returnTo, '/\\')

насколько я помню parse_url в PHP в некоторых случаях работает не всегда точно
то есть, грубо говоря, я бы добавил еще проверку на !str_starts_with($returnTo, "/\\")

[fayzetwin1]

по сути оно делает то же самое - берет сугубо $parsedReturnTo["path"] и отбрасывает хост

для простоты я считаю можно просто разрешать только безопасные относительные пути без протоколов и доменов

к примеру:

   if (
       is_string($returnTo) 
       && str_starts_with($returnTo, "/") 
       && !str_starts_with($returnTo, "//") 
       && !str_starts_with($returnTo, "/\\")
   ) {
       $parsed = parse_url($returnTo);
       if ($parsed !== false && !isset($parsed["scheme"]) && !isset($parsed["host"])) {
           $redirect =$returnTo;
       }
   }

[fayzetwin1]

реферер может содержать чисто в теории что-то лишнее (внешний сайт условно говоря), и ссылка в шаблоне может тогда содержать лишний мусор

мне кажется реферер можно определять прямо в PhotosPresenter через $this->getHttpRequest()->getReferer() как fallback если мы не передали return_to

[fayzetwin1]

..или если не хочется делать велосипед (а его и не надо делать) и если баг был только в том, что /id0 редиректит в ленту из-за настроек главной страницы, то весь этот блок с query-параметрами и валидацией хостов лишний

то есть заменить в fallback хардкод на реальный url овнера:

if (!is_null($album =$photo->getAlbum())) {
    $redirect = '/album' .$album->getPrettyId();
} else {
    $redirect =$photo->getOwner()->getURL();
}

[fayzetwin1]

если так можно конечно ибо я сам хз чуть-чуть

[mrksbgg]

я так понял, /id0 кидает на / либо на свою страницу в зависимости от выбора главной страницы
конечно слегка не понятен прикол, зачем после удаления фотографии отправлять на /d0
ну и велосипед конечно я создаю

fayzetwin, я изучу твои комментарии и постараюсь поправить этот ужас