Как добавть в код get_category_link?

Question

[AlextooG]

У меня плагин мобильной версии понаделал дубли типа: ?fdx_switcher=true, ?fdx_switcher=mobile и ?fdx_switcher=desktop

Т.к. rel="canonical" у меня прописан, я не особо переживал, пока не увидел, что PageSpeed Insights ругается на переадресацию и выводит урл всех страницы мобильной темы с ?fdx_switcher=true.

Покапавший в интернете я нашел такой код в статье «Переадресация с дублей на каноническую страницу»

if ( is_singular() ) {
	$check = get_bloginfo('url') . $_SERVER['REQUEST_URI']; // узнаем текущий url
	if ( $check != get_permalink() ){
		header( 'HTTP/1.1 301 Moved Permanently' );
		header( 'Location: ' . get_permalink() );
	} // операция сравнения, если обнаруживаем отличие, делаем перенаправление
}

Этот код замечательно сработал, в мобильной теме посты и страницы показываются с каноническим урл и PageSpeed Insights так же перестал ругаться.

Но данный код не работает, для категорий. Урлы категорий остались, как раньше с ?fdx_switcher=true.
Присмотревшись к коду, я начал думать, что get_permalink() это получение постоянной ссылки на запись, а для категорий надо использовать get_category_link(). Пытался по разному вставить get_category_link() в данный код, но не получается, моих знаний не хватает.

Помогите, как правильно изменить код, чтобы переадресация работала и для категориях.

Answer 1

[Артем Золин]

Потому что вначале у вас условие is_singular(). Для категорий нужно условие is_archive(). И вы можете использовать нативную функцию wp_redirect()

Comments

[AlextooG]

Спасибо, за ответ.
Да, когда экспериментировал с get_category_link() делал условия с is_archive() и добавлял такой код:

<?php
if (is_archive()){
$check = get_bloginfo(‘url’) . $_SERVER[‘REQUEST_URI’]; // узнаем текущий url
if ( $check != get_category_link()){
header(«HTTP/1.1 301 Moved Permanently»);
header(‘Location: ‘ . get_category_link() );
} // операция сравнения, если обнаруживаем отличие, делаем перенаправление
}
?>

Но сайт сразу ломался, видимо два кода вызывают где-то конфликт.

Про wp_redirect() ничего не знаю. Как должен выглядеть код с ним, чтобы сделать редирект на рубрики? Помогите разобраться.

[Артем Золин]

AlextooG, ломался, потому что у вас везде неправильные кавычки, используйте одинарные ' или двойные ", а в функцию get_category_link() нужно передавать $category_id первым параметром

[AlextooG]

Спасибо за кавычки, действительно не заметил. Переписал код так:
<?php
if (is_archive()){
$check = get_bloginfo('url') . $_SERVER['REQUEST_URI'];
if ( $check != get_category_link($category_id)){
header('HTTP/1.1 301 Moved Permanently');
header('Location: ' . get_category_link($category_id) );
}
}
?>

сайт не ломается, но и ридерект не происходит. Не работает этот код.

[Артем Золин]

AlextooG, потому что в $category_id ничего нет, id категории нужно передавать в функцию, чтобы получить на нее ссылку

[AlextooG]

Артем Золин, не могли бы написать подробнее, как передавать в функцию, что конкретно записать в этом коде.
сам я не справлюсь

[AlextooG]

Артем Золин, Ну, я как вижу воду здесь все мастера разводить, а как до дела доходит, то молчок.

[Артем Золин]

AlextooG, я вам ничего не должен, эти претензии совсем не уместны

[AlextooG]

Артем Золин, Извините, если мои слова вас обидели. Но если сказал "А", то надо говорить и "В"
А просто комментировать, что бы поиздеваться над чужими проблемами, может каждый. Лишний раз убедился, что подобные форуму абсолютно бесполезны, только тратишь свое время.

Answer 2

[AlextooG]

нет видимо здесь знающих людей. Бессмысленно, что то спрашивать.