1
$url = "https://" . $_SERVER["HTTP_HOST"] . $_SERVER["REQUEST_URI"];
if($url != preg_replace('/([^:])(\/{2,})/', '$1/', $url)){
    header('Location: '. preg_replace('/([^:])(\/{2,})/', '$1/', $url), true, 301);
  exit();
}

Задача склеить все "/", которые идут подряд и сделать редирект на правильный адрес. Подскажите, что не так?

4
  • заменить на /$1а не /$1 ?
    – teran
    25 дек 2017 в 8:41
  • либо (?<!:)(\/{2,}) заменять на /
    – teran
    25 дек 2017 в 8:46
  • что-то я смешное написал в первом комменте :D а что не так то? вроде заменяет ведь все исправно? не ясно правда, зачем вы клеите весь url, и потом усложняете замену, если можно было сделать замену только в части request_uri. да и делать такое пожалуй надо на уровне апача, а не пхп.
    – teran
    25 дек 2017 в 8:54
  • Решение через .htaccess ru.stackoverflow.com/a/600073/186083
    – Visman
    25 дек 2017 в 8:58

1 ответ 1

0

Можно так переписать:

$url = "https://" . $_SERVER["HTTP_HOST"] . $_SERVER["REQUEST_URI"];
$fixed_url = preg_replace('/([^:])(\/{2,})/', '$1/', $url);
if($url != $fixed_url){
    header('Location: '. $fixed_url, true, 301);
  exit();
}
3
  • 1
    дак что принципиально поменялось в данном коде? в чем была ошибка? я вот пока что вижу лишнюю скобку в header
    – teran
    25 дек 2017 в 9:08
  • Спасибо, teran. Переделал.
    – Danny
    25 дек 2017 в 9:13
  • пожалуйста, постарайтесь оставлять чуть более развёрнутые ответы. дополнить ответ можно, нажав править 25 дек 2017 в 9:25

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.