Что не так с 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; ?>

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

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))
это действительно не работает. (иногда видимо нужно перегружать свои мозги).
Исправил на то, что нужно.

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

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

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

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

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

WordPress

+1 ещё

Простой
Как из Wordpress обратиться к внешнему PostgreSQL-серверу?
- 1 подписчик
- 8 часов назад
- 32 просмотра
2

ответа
WordPress

Средний
Как в Elementor создать шаблоны для Custom Post Type?
- 1 подписчик
- вчера
- 20 просмотров
0

ответов
WordPress

Простой
Если это то покажи это, а если нет, то вот это, как в Wordpress?
- 1 подписчик
- 20 нояб.
- 63 просмотра
2

ответа
WordPress

+1 ещё

Простой
Локализация страницы в вордпресс, как исправить?
- 1 подписчик
- 20 нояб.
- 37 просмотров
1

ответ
WordPress

+1 ещё

Сложный
Почему не срабатывает простейший код перенаправления и происходит перебрасывание на старый URL?
- 1 подписчик
- 19 нояб.
- 58 просмотров
0

ответов
JavaScript

+1 ещё

Простой
Как настроить отправку ClientID яндекс метрики в форме обратной связи wordpress?
- 2 подписчика
- 18 нояб.
- 186 просмотров
0

ответов
WordPress

Простой
Как в Wp создать шорткод и вывести, который будет показывать url текущей записи или страницы?
- 1 подписчик
- 18 нояб.
- 35 просмотров
0

ответов
WordPress

Простой
Как при применении фильтров перейти на первую страницу пагинации Wordpress?
- 2 подписчика
- 18 нояб.
- 49 просмотров
0

ответов
WordPress

Простой
На каком бесплатном хостинге можно создать wordpress сайт для теста?
- 1 подписчик
- 18 нояб.
- 569 просмотров
2

ответа
WordPress

Средний
Как убрать в рассылке от плагина Contact Form 7 уведомление о спаме?
- 1 подписчик
- 18 нояб.
- 137 просмотров
1

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

WordPress-разработчик

div. • Ставрополь

от 50 000 до 120 000 ₽

Администратор сайта (Wordpress)

Private-Team

от 700 $

Fullstack Developer под Wordpress

Future Search

от 90 000 до 150 000 ₽

Телеграмм бот на Python

22 нояб. 2024, в 22:26

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

Сделать несколько гиф анимаций

22 нояб. 2024, в 21:47

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

Доделать сайт на Revolution 3

22 нояб. 2024, в 21:44

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

Говнокод. Поэтому и не работает.
Как победить:
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?

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

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

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