Что не так с elseif?

Question

Legal Adviser @Legal2019

Всё в имени моём... и радость и печаль...

WordPress

Что не так с elseif?

Всем привет, ни как не пойму, что не так с кодом... Глаза уже замылились видимо...
Почему-то не хочет он подгружать is_page() - template-parts/content-page/content-page
Всё останавливается на /template-parts/content-page/content-court-msk-jurisdiction
Сам код:

<?php if (is_page('kontakty')): ?>
		<?php
		// Start the loop.
		while ( have_posts() ) : the_post();
                    //echo '<div style="border: 1px dotted gray; background-color: #ffffd975; margin: 0; padding: 0;">
                    //    <p style="color: black;">Это страница - page.php</p>';
			// Include the page content template.
                        get_template_part( 'template-parts/content-page/content-kontakt-page', 'page' );
                        //  echo '***********';
                        //  get_template_part( 'template-parts/content/content', 'page' );
                        //  echo '***********';
                        //  get_template_part( 'template-parts/content', 'page' );

			// If comments are open or we have at least one comment, load up the comment template.
			if ( comments_open() || get_comments_number() ) {
				comments_template();
			}
                    //    echo '</div>';
			// End of the loop.
		endwhile;
		?>
    <?php//   elseif (is_page(array('notary'=>$post->ID))): ?>
    <?php  elseif (is_singular('notary')): ?>
		<?php
		// Start the loop.
		while ( have_posts() ) : the_post();
                    //echo '<div style="border: 1px dotted gray; background-color: #ffffd975; margin: 0; padding: 0;">
                    //    <p style="color: black;">Это страница - page.php</p>';
			// Include the page content template.
                        get_template_part( 'template-parts/content-page/content-page-notary', 'page-notary' );
                        //  echo '***********';
                        //  get_template_part( 'template-parts/content/content', 'page' );
                        //  echo '***********';
                        //  get_template_part( 'template-parts/content', 'page' );

			// If comments are open or we have at least one comment, load up the comment template.
			if ( comments_open() || get_comments_number() ) {
				comments_template();
			}
                    //    echo '</div>';
			// End of the loop.
		endwhile;
		?>
    <?php   elseif (is_page(array('court-msk-jurisdiction'=>$post->ID))): ?>
    <?php   //elseif (is_page(array('laws'=>$post->ID))): ?>
		<?php
		// Start the loop.
		while ( have_posts() ) : the_post();
                    //echo '<div style="border: 1px dotted gray; background-color: #ffffd975; margin: 0; padding: 0;">
                    //    <p style="color: black;">Это страница - page.php</p>';
			// Include the page content template.

  get_template_part( '/template-parts/content-page/content-court-msk-jurisdiction', 'page' );
//  get_template_part( 'template-parts/content-page/content-page-tests', 'page' );
                        //  get_template_part( 'template-parts/content-page/content-laws-page', 'page' );
                        //  echo '***********';
                        //  get_template_part( 'template-parts/content/content', 'page' );
                        //  echo '***********';
                        //  get_template_part( 'template-parts/content', 'page' );

			// If comments are open or we have at least one comment, load up the comment template.
			if ( comments_open() || get_comments_number() ) {
				comments_template();
			}
                    //    echo '</div>';
			// End of the loop.
		endwhile;
		?>
<?php   elseif (is_page(array('zakony'=>$post->ID))): ?>
            <?php //   elseif (is_page(array('zakony'=>$post->ID))): ?>
            <?php
		// Start the loop.
		while ( have_posts() ) : the_post();

			// Include the single post content template.
			get_template_part( 'template-parts/content-laws/content-zakon', 'page' );

			// If comments are open or we have at least one comment, load up the comment template.
			if ( comments_open() || get_comments_number() ) {
				comments_template();
			}

			if ( is_singular( 'attachment' ) ) {
				// Parent post navigation.
				the_post_navigation( array(
					'prev_text' => _x( '<span class="meta-nav">Published in</span><span class="post-title">%title</span>', 'Parent post link', 'mylbo' ),
				) );
			} elseif ( is_singular( 'post' ) ) {
				// Previous/next post navigation.
				the_post_navigation( array(
					'next_text' => '<span class="meta-nav" aria-hidden="true">' . __( 'Next', 'mylbo' ) . '</span> ' .
						'<span class="screen-reader-text">' . __( 'Next post:', 'mylbo' ) . '</span> ' .
						'<span class="post-title">%title</span>',
					'prev_text' => '<span class="meta-nav" aria-hidden="true">' . __( 'Previous', 'mylbo' ) . '</span> ' .
						'<span class="screen-reader-text">' . __( 'Previous post:', 'mylbo' ) . '</span> ' .
						'<span class="post-title">%title</span>',
				) );
			}

			// End of the loop.
		endwhile;
		?>
            <?php   elseif (is_page(array('kodeksy-rf'=>$post->ID))): ?>
            <?php
		// Start the loop.
		while ( have_posts() ) : the_post();

			// Include the single post content template.
			get_template_part( 'template-parts/content-laws/content-kodeks', 'page' );

			// If comments are open or we have at least one comment, load up the comment template.
			if ( comments_open() || get_comments_number() ) {
				comments_template();
			}

			if ( is_singular( 'attachment' ) ) {
				// Parent post navigation.
				the_post_navigation( array(
					'prev_text' => _x( '<span class="meta-nav">Published in</span><span class="post-title">%title</span>', 'Parent post link', 'mylbo' ),
				) );
			} elseif ( is_singular( 'post' ) ) {
				// Previous/next post navigation.
				the_post_navigation( array(
					'next_text' => '<span class="meta-nav" aria-hidden="true">' . __( 'Next', 'mylbo' ) . '</span> ' .
						'<span class="screen-reader-text">' . __( 'Next post:', 'mylbo' ) . '</span> ' .
						'<span class="post-title">%title</span>',
					'prev_text' => '<span class="meta-nav" aria-hidden="true">' . __( 'Previous', 'mylbo' ) . '</span> ' .
						'<span class="screen-reader-text">' . __( 'Previous post:', 'mylbo' ) . '</span> ' .
						'<span class="post-title">%title</span>',
				) );
			}

			// End of the loop.
		endwhile;
		?>
            <?php   elseif (is_page(array('plenumy-vs-rf'=>$post->ID))): ?>
            <?php
		// Start the loop.
		while ( have_posts() ) : the_post();

			// Include the single post content template.
			get_template_part( 'template-parts/content-laws/content-plenum', 'page' );

			// If comments are open or we have at least one comment, load up the comment template.
			if ( comments_open() || get_comments_number() ) {
				comments_template();
			}

			if ( is_singular( 'attachment' ) ) {
				// Parent post navigation.
				the_post_navigation( array(
					'prev_text' => _x( '<span class="meta-nav">Published in</span><span class="post-title">%title</span>', 'Parent post link', 'mylbo' ),
				) );
			} elseif ( is_singular( 'post' ) ) {
				// Previous/next post navigation.
				the_post_navigation( array(
					'next_text' => '<span class="meta-nav" aria-hidden="true">' . __( 'Next', 'mylbo' ) . '</span> ' .
						'<span class="screen-reader-text">' . __( 'Next post:', 'mylbo' ) . '</span> ' .
						'<span class="post-title">%title</span>',
					'prev_text' => '<span class="meta-nav" aria-hidden="true">' . __( 'Previous', 'mylbo' ) . '</span> ' .
						'<span class="screen-reader-text">' . __( 'Previous post:', 'mylbo' ) . '</span> ' .
						'<span class="post-title">%title</span>',
				) );
			}

			// End of the loop.
		endwhile;
		?>
    <?php   elseif (is_page(array('notary-msk'=>$post->ID))): ?>
    <?php   //elseif (is_page(array('laws'=>$post->ID))): ?>
		<?php
		// Start the loop.
		while ( have_posts() ) : the_post();
                    //echo '<div style="border: 1px dotted gray; background-color: #ffffd975; margin: 0; padding: 0;">
                    //    <p style="color: black;">Это страница - page.php</p>';
			// Include the page content template.
                        get_template_part( 'template-parts/content-page/content-notary-page', 'page' );
                        //  echo '***********';
                        //  get_template_part( 'template-parts/content/content', 'page' );
                        //  echo '***********';
                        //  get_template_part( 'template-parts/content', 'page' );

			// If comments are open or we have at least one comment, load up the comment template.
			if ( comments_open() || get_comments_number() ) {
				comments_template();
			}
                    //    echo '</div>';
			// End of the loop.
		endwhile;
		?>

    <?php   elseif (is_page()): ?>
		<?php
		// Start the loop.
		while ( have_posts() ) : the_post();
                    //echo '<div style="border: 1px dotted gray; background-color: #ffffd975; margin: 0; padding: 0;">
                    //    <p style="color: black;">Это страница - page.php</p>';
			// Include the page content template.
                        get_template_part( 'template-parts/content-page/content-page', 'page' );
                        //  echo '***********';
                        //  get_template_part( 'template-parts/content/content', 'page' );
                        //  echo '***********';
                        //  get_template_part( 'template-parts/content', 'page' );

			// If comments are open or we have at least one comment, load up the comment template.
			if ( comments_open() || get_comments_number() ) {
				comments_template();
			}
                    //    echo '</div>';
			// End of the loop.
		endwhile;
		?>
    <?php endif; ?>

Вопрос задан более трёх лет назад
64 просмотра

6 комментариев

Подписаться 1 Простой 6 комментариев

Дмитрий @dimasmagadan
Говнокод. Поэтому и не работает.
Как победить:
1. конструкции вида endwhile; endif; - зло. Замени на скобочки.
Некоторые редакторы умеют подсвечивать парные скобочки, сразу видно блоки кода.
По стандартам, впрочем, так писать можно.
2. код дублируется
посчитай, сколько раз в этой портянке у тебя вызывается while ( have_posts() ) : the_post(); ?
я посчитал. 8 раз. Зачем так?
лучше вызвать 1 раз, а все эти ифы запихать внутрь типо так:

while ( have_posts() ) { the_post(); if ( is_singular('notary') ) { // грузим один шаблон } elseif ( is_page() ) { // грузим другой шаблон } // и тп }

все аккуратненько лежит внутри цикла, код не дублируется
3. почитай про иерархию шаблонов в WP
у тебя вся логика напихана в один файл, некоторые вещи проще разнести по разным файлам. Как минимум станет читабельнее
Написано более трёх лет назад

Legal Adviser @Legal2019 Автор вопроса

Дмитрий, Да я и не против с вашим заключением, я понять не могу почему не работает-то...
Вот тот же код (почистил для удобочитаемости), но он всё равно не пашет. Обрабатывается лишь самое первое условие и всё.

<?php
    // Start the loop.
    while ( have_posts() ) : the_post();{
            if (is_page()){
            get_template_part( 'template-parts/content-page/content-page', 'page' );
            }   elseif (is_page('contacts')){
            get_template_part( 'template-parts/content-page/content-contacts-page', 'page' );
            }   elseif (is_singular('notary')){
            get_template_part( 'template-parts/content-page/content-page-notary', 'page-notary' );
            }   elseif (is_page(array('court-msk-jurisdiction'=>$post->ID))){
            get_template_part( '/template-parts/content-page/content-court-msk-jurisdiction', 'page' );
            }   elseif (is_page(array('zakony'=>$post->ID))){
            get_template_part( 'template-parts/content-laws/content-zakon', 'page' );
            }   elseif (is_page(array('kodeksy-rf'=>$post->ID))){
            get_template_part( 'template-parts/content-laws/content-kodeks', 'page' );
            }   elseif (is_page(array('plenumy-vs-rf'=>$post->ID))){
            get_template_part( 'template-parts/content-laws/content-plenum', 'page' );
            }   elseif (is_page(array('notary-msk'=>$post->ID))){
            get_template_part( 'template-parts/content-page/content-notary-page', 'page' );
    }
    // If comments are open or we have at least one comment, load up the comment template.
    if ( comments_open() || get_comments_number() ) {
            comments_template();
	}
    if ( is_singular( 'attachment' ) ) {
    // Parent post navigation.
    the_post_navigation( array(
                        'prev_text' => _x( '<span class="meta-nav">Published in</span><span class="post-title">%title</span>', 'Parent post link', 'mylbo' ),
			) );
    } elseif ( is_singular( 'post' ) ) {
    // Previous/next post navigation.
    the_post_navigation( array(
                        'next_text' => '<span class="meta-nav" aria-hidden="true">' . __( 'Next', 'mylbo' ) . '</span> ' .
			'<span class="screen-reader-text">' . __( 'Next post:', 'mylbo' ) . '</span> ' .
			'<span class="post-title">%title</span>',
			'prev_text' => '<span class="meta-nav" aria-hidden="true">' . __( 'Previous', 'mylbo' ) . '</span> ' .
			'<span class="screen-reader-text">' . __( 'Previous post:', 'mylbo' ) . '</span> ' .
			'<span class="post-title">%title</span>',
			) );
			}
    }
    endwhile;
?>

Т.е. если поменять скажем (соответственно меняя elseif + if ):

if (is_page()){
            get_template_part( 'template-parts/content-page/content-page', 'page' );

elseif (is_page('contacts')){
            get_template_part( 'template-parts/content-page/content-contacts-page', 'page' );

То выводится будет лишь первое условие.

Написано более трёх лет назад

Дмитрий @dimasmagadan

Legal Adviser,

То выводится будет лишь первое условие.

именно по этому более специфичные условия стоит ставить выше

is_page('contacts') - должно идти первым
is_page() - ниже

так же не понятно, что по вашему должна делать вот такая конструкция
is_page(array('notary-msk'=>$post->ID))
в виде массива в нее можно передавать список страниц, которые нужно проверить
а у вас там внутри что-то странное
https://wp-kama.ru/function/is_page

Написано более трёх лет назад
Legal Adviser @Legal2019 Автор вопроса

Дмитрий,
Про:
is_page(array('notary-msk'=>$post->ID))

Есть страница (главная) у неё есть дочки (до 3-4 уровня) пробовал разные варианты вывода шаблона для главной и её дочек... самое забавное, но почему-то в этом виде выводится стало так, как мне нужно.

Написано более трёх лет назад
Дмитрий @dimasmagadan
Legal Adviser, по ссылке выше есть примеры, как правильно должно быть
например так

if ( is_page( 'about' ) || '2' == $post->post_parent ) { // Это страница "О сайте" или её дочерняя страница $bannerimg = 'about.jpg'; }
Написано более трёх лет назад
Legal Adviser @Legal2019 Автор вопроса

Дмитрий,
Сейчас пересмотрел... по всей видимости был некий глюк после активации плагина, т.к. теперь
is_page(array('notary-msk'=>$post->ID))
это действительно не работает. (иногда видимо нужно перегружать свои мозги).
Исправил на то, что нужно.

Написано более трёх лет назад

Пригласить эксперта

Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации

Похожие вопросы

JavaScript

+1 ещё

Простой
Как при мощи js создать поле дополнительных ингридиентов?
- 1 подписчик
- 3 часа назад
- 23 просмотра
0

ответов
WordPress

+1 ещё

Простой
Какой есть плагин для регистрации и добавлением дополнительных полей и с поддержкой Woocommerce?
- 1 подписчик
- 17 часов назад
- 24 просмотра
2

ответа
WordPress

+1 ещё

Простой
Woocommerce rest api, как реализовать филтрацию с учетом вариаций?
- 1 подписчик
- 19 часов назад
- 19 просмотров
1

ответ
WordPress

Средний
Каким способом лучше сделать натяжку верстки на Wordpress? ACF либо другие способы?
- 1 подписчик
- вчера
- 51 просмотр
1

ответ
WordPress

Простой
Как сделать pagination в WordPress для Custom Post Type?
- 1 подписчик
- 23 апр.
- 30 просмотров
0

ответов
WordPress

Простой
Как создать шаблон вывода записей с произвольным типом?
- 1 подписчик
- 23 апр.
- 31 просмотр
1

ответ
WordPress

+2 ещё

Средний
Как сделать автоматический выбор региона по IP в Wordpress?
- 1 подписчик
- 22 апр.
- 62 просмотра
2

ответа
WordPress

+1 ещё

Простой
Как добавить RSS widget на wordpress?
- 1 подписчик
- 22 апр.
- 29 просмотров
1

ответ
WordPress

Средний
Как увеличить стоимость товара при помощи чекбоксов?
- 1 подписчик
- 21 апр.
- 53 просмотра
2

ответа
WordPress

Средний
Как мне настроить слудеющую иерархию ссылок WordPress?
- 1 подписчик
- 21 апр.
- 32 просмотра
0

ответов
Показать ещё Загружается…

WordPress developer (Contractor, Remote)

Hicaliber

До 2 300 $

Разработчик на Wordpress

Nocodered

от 40 000 до 140 000 ₽

Контент-менеджер, редактор (web3, крипто, финансы)

Bitbanker.ru

от 80 000 до 100 000 ₽

Разработать электронику для весов с Wi-Fi

26 апр. 2024, в 01:22

1000 руб./в час

Очень срочно нужно помочь запустить программу с UI

26 апр. 2024, в 00:13

1000 руб./за проект

Создание бота/скрипта для сайта забронирование мест

26 апр. 2024, в 00:10

30000 руб./за проект

Говнокод. Поэтому и не работает.
Как победить:
1. конструкции вида endwhile; endif; - зло. Замени на скобочки.
Некоторые редакторы умеют подсвечивать парные скобочки, сразу видно блоки кода.
По стандартам, впрочем, так писать можно.
2. код дублируется
посчитай, сколько раз в этой портянке у тебя вызывается while ( have_posts() ) : the_post(); ?
я посчитал. 8 раз. Зачем так?
лучше вызвать 1 раз, а все эти ифы запихать внутрь типо так:

while ( have_posts() ) { the_post(); if ( is_singular('notary') ) { // грузим один шаблон } elseif ( is_page() ) { // грузим другой шаблон } // и тп }

все аккуратненько лежит внутри цикла, код не дублируется
3. почитай про иерархию шаблонов в WP
у тебя вся логика напихана в один файл, некоторые вещи проще разнести по разным файлам. Как минимум станет читабельнее
Legal Adviser,

То выводится будет лишь первое условие.

именно по этому более специфичные условия стоит ставить выше

is_page('contacts') - должно идти первым
is_page() - ниже

так же не понятно, что по вашему должна делать вот такая конструкция
is_page(array('notary-msk'=>$post->ID))
в виде массива в нее можно передавать список страниц, которые нужно проверить
а у вас там внутри что-то странное
https://wp-kama.ru/function/is_page
Дмитрий,
Про:
is_page(array('notary-msk'=>$post->ID))

Есть страница (главная) у неё есть дочки (до 3-4 уровня) пробовал разные варианты вывода шаблона для главной и её дочек... самое забавное, но почему-то в этом виде выводится стало так, как мне нужно.
Legal Adviser, по ссылке выше есть примеры, как правильно должно быть
например так

if ( is_page( 'about' ) || '2' == $post->post_parent ) { // Это страница "О сайте" или её дочерняя страница $bannerimg = 'about.jpg'; }
Дмитрий,
Сейчас пересмотрел... по всей видимости был некий глюк после активации плагина, т.к. теперь
is_page(array('notary-msk'=>$post->ID))
это действительно не работает. (иногда видимо нужно перегружать свои мозги).
Исправил на то, что нужно.

Что не так с elseif?

Войдите, чтобы написать ответ

Минуточку внимания

Войдите на сайт