Как правильно вывести строку через условные теги WordPress и условные операторы?

Question

[Biff]

Пытаюсь вывести для разметки OpenGraph строки.

Учитывая:

• главную страницу
  
• 404-ю
  
• страницу поиска
  

Конструкция мета тега такая сейчас:

<meta property="og:description" content="

<?php 

if ( is_front_page() ) {
echo 'Главная страница';
} 

if ( is_404() ) {
echo 'Ошибка 404';
} 

if ( is_search() ) {
echo 'Странца поиска';
} 

else {
echo the_title();
} ?>

">

Проблема следующая :)
На 404-й, странице поиска и остальных страницах всё показывается верно. А вот на главной выводится не то, что нужно.

На главной показывается мета тег, который содержит и "Главная страница", как надо, и название страницы:

<meta property="og:description" content="Главная страницаКомпания по производству теплоизоляции">

Важно - главная страница сайта у меня статическая (т.е. не блог на главной, а другая страница, выбранная в админке).
Где я неправильно сделал?

Answer 1

[Denkuwus]

Похоже, вы хотите отобразить строку «Главная страница» для главной страницы и заголовок страницы для всех остальных страниц.

Одна проблема с вашим кодом заключается в том, что оператор else не связан ни с одним из операторов if. Это означает, что он всегда будет выполняться, независимо от того, верны ли какие-либо из предыдущих операторов if.

Чтобы исправить это, вы можете использовать оператор elseif вместо оператора else. Оператор elseif будет выполнен только в том случае, если предыдущий оператор if ложен, а условие в операторе elseif истинно.

Вот как вы можете изменить свой код для достижения желаемого поведения:

<meta property="og:description" content="

<?php

if ( is_front_page() ) {
    echo 'Main page';
} elseif ( is_404() ) {
    echo 'Error 404';
} elseif ( is_search() ) {
    echo 'Search page';
} else {
    echo the_title();
} ?>

">

Это должно отображать правильную строку для каждой страницы.

Comments

[Дмитрий]

echo the_title(); - делать плохо
достаточно the_title();

[Biff]

Спасибо. То, что нужно!

[Biff]

Дмитрий, почему echo the_title плохо?

[Biff]

Дмитрий, и также вопрос. Конструкция для opengraph также имеет meta property="og:url" content="//site.ru/". Проблема в том, что на странице 404 и странице поиска meta og:url пустой. Я пробовал использовать get_permalink() и get_page_link(), всегда пусто в мета теге. Как это исправляется?

[Denkuwus]

Biff, wp_get_canonical_url пробуй эт

$canonical_url = wp_get_canonical_url();
echo '<meta property="og:url" content="' . esc_attr( $canonical_url ) . '" />';

[Дмитрий]

Biff, ну если прям хочется делать echo the_title(); нужно делать вот так
echo the_title('', '', 0);
но зачем? если можно сразу сделать вот так:
the_title();

Проблема в том, что на странице 404 и странице поиска meta og:url пустой

а какой по вашему должен быть урл для 404й страницы? зачем вообще вам нужна og разметка для 404й страницы? вы планируете ее индексировать или шарить в соц.сетях?
обычно, если была страница и ее удалили, ставят редирект на что-то другое. чтоб не терять ссылку

совет из этой дискусси зря отметили ответом, оне не полный и с ошибками

проблема с вашим кодом заключается в том, что оператор else не связан ни с одним из операторов if.

вот это

if ( is_search() ) {
echo 'Странца поиска';
} 

else {
echo the_title();
}

равно вот этому

if ( is_search() ) { // проверяем, это страница поиска?
// если да, выводим это
echo 'Странца поиска';
}  else {
// но это не страница поиска, поэтоум выводим вот это
echo the_title();
}

последнее условие всегда будет срабатывать, тк это не страница поиска. Утверждение "оператор else не связан ни с одним из операторов if." - неправильно

[Biff]

Denkuwus,

wp_get_canonical_url пробуй эт

Не сработало :) Также - пусто на 404 и поиске

[Biff]

Дмитрий, про echo the_title так и не понял. Без указания параметров может вылезти лишнее в названии?

зачем вообще вам нужна og разметка для 404й страницы?

Да просто, чтобы было. Чтобы были заполнены основные поля. Вроде как og:url является обязательным по валидации.
Я не просто так боролся и с elseif ( is_404() ). В случае, если будет отправлена неверная ссылка в мессенджер или соцсеть, чтобы получатель увидел "Ошибка 404". Чисто удобство для юзеров.

Совет из этой дискусси зря отметили ответом, оне не полный и с ошибками

Не совсем понял.
С elseif работает нормально.

[Дмитрий]

Biff,
Утверждение "оператор else не связан ни с одним из операторов if." - неправильно.
Код в вопросе был с ошибкой (всегда выполнялась проверка "это страница поиска или нет"), предложенный Denkuwus вариант исправляет, но то, что он пишет, в ответе, неправильно

[Denkuwus]

Дмитрий, :(

[Biff]

Дмитрий, работает она, как надо)

Answer 2

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

Забирайте проверки по всем условным тегам

$separator = '|';

// Title prefixes.
if ( is_post_type_archive() ) {
	$title[] = _x( 'Archives:', 'post type archive title prefix', 'wpgen' );
} elseif ( is_category() ) {
	$title[] = _x( 'Category:', 'category archive title prefix', 'wpgen' );
} elseif ( is_tag() ) {
	$title[] = _x( 'Tag:', 'tag archive title prefix', 'wpgen' );
} elseif ( is_author() ) {
	$title[] = _x( 'Author:', 'author archive title prefix', 'wpgen' );
} elseif ( is_date() ) {
	if ( is_year() ) {
		$title[] = _x( 'Year:', 'date archive title prefix', 'wpgen' );
	} elseif ( is_month() ) {
		$title[] = _x( 'Month:', 'date archive title prefix', 'wpgen' );
	} elseif ( is_day() ) {
		$title[] = _x( 'Day:', 'date archive title prefix', 'wpgen' );
	}
} elseif ( is_tax( 'post_format' ) ) {
	if ( is_tax( 'post_format', 'post-format-aside' ) ) {
		$title[] = _x( 'Asides', 'post format archive title', 'wpgen' );
	} elseif ( is_tax( 'post_format', 'post-format-gallery' ) ) {
		$title[] = _x( 'Galleries', 'post format archive title', 'wpgen' );
	} elseif ( is_tax( 'post_format', 'post-format-image' ) ) {
		$title[] = _x( 'Images', 'post format archive title', 'wpgen' );
	} elseif ( is_tax( 'post_format', 'post-format-video' ) ) {
		$title[] = _x( 'Videos', 'post format archive title', 'wpgen' );
	} elseif ( is_tax( 'post_format', 'post-format-quote' ) ) {
		$title[] = _x( 'Quotes', 'post format archive title', 'wpgen' );
	} elseif ( is_tax( 'post_format', 'post-format-link' ) ) {
		$title[] = _x( 'Links', 'post format archive title', 'wpgen' );
	} elseif ( is_tax( 'post_format', 'post-format-status' ) ) {
		$title[] = _x( 'Statuses', 'post format archive title', 'wpgen' );
	} elseif ( is_tax( 'post_format', 'post-format-audio' ) ) {
		$title[] = _x( 'Audio', 'post format archive title', 'wpgen' );
	} elseif ( is_tax( 'post_format', 'post-format-chat' ) ) {
		$title[] = _x( 'Chats', 'post format archive title', 'wpgen' );
	}
}

// Main title.
if ( is_single() || is_page() ) {
	$title[] = get_the_title();
} elseif ( is_404() ) {
	$title[] = __( 'Page not found', 'wpgen' );
} elseif ( is_search() ) {
	$title[] = sprintf( __( 'Search Results for “%s”', 'wpgen' ), get_search_query( false ) );
} elseif ( is_post_type_archive() ) {
	$title[] = get_queried_object()->label;
} elseif ( is_author() ) {
	$title[] = get_queried_object()->display_name ?? '';
} elseif ( is_date() ) {
	if ( is_year() ) {
		$title[]  = get_the_date( 'Y' );
	} elseif ( is_month() ) {
		$title[]  = get_the_date( 'F Y' );
	} elseif ( is_day() ) {
		$title[]  = get_the_date( 'F j, Y' );
	}
} elseif ( is_archive() ) {
	$title[] = get_queried_object()->name;
}

// Add site info in the title.
if ( ! isset( $title ) ) {
	$title[] = get_bloginfo( 'name' );
	$title[] = $separator;
	$title[] = get_bloginfo( 'description' );
} else {

	if ( is_paged() ) {
		$title[] = $separator;
		$title[] = __( 'Page', 'wpgen' ) . ' ' . $paged;
	}

	$title[] = $separator;
	$title[] = get_bloginfo( 'name' );
}

$title = implode( ' ', $title );

Comments

[Biff]

Спасибо. Полезнейшая информация!

[Biff]

Артем Золин, и также вопрос. Конструкция для opengraph также имеет meta property="og:url" content="//site.ru/". Проблема в том, что на странице 404 и странице поиска meta og:url пустой. Я пробовал использовать get_permalink() и get_page_link(), всегда пусто в мета теге. Как это исправляется?

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

Biff, функция get_permalink() используется для получения ссылки на запись (post) в цикле, а get_page_link() для получения ссылки на страницы (page) тоже в цикле

Для архивных страниц можете использовать get_term_link(), а вообще текущий запрос находится в global $wp; и получить текущую ссылку можно так

$url = trailingslashit( home_url( $wp->request ) );

Если вам нужно добавить в ссылку страницы пагинации или другие гет-параметры, это можно сделать с помощью add_query_arg()

[Biff]

Артем Золин, я поступил иначе.

Сделал так:

$cururl = ((!empty($_SERVER['HTTPS'])) ? 'https' : 'http') . '://' . $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI'];

<meta property="og:url" content="<?php echo $cururl; ?>">

Работает, как нужно, в принципе