Как это упростить?

Question

[olya_097]

РЕчь про дублиюрущий php код
а конкретно этот кусок

while ( have_posts() ) : the_post();
		  if ( get_post_gallery() ) :
			$gallery = get_post_gallery( get_the_ID(), false );

https://jsfiddle.net/cp4et3La/

Comments

[Павел]

а что у вас get_post_gallery() делает в первый раз без параметров? и в чем тогда дублирование, если get_post_gallery() и get_post_gallery( get_the_ID(), false ) это разные вызовы

[olya_097]

Павел, дублирование есть
строки 2-4 и 22-24

[Павел]

olya_097, а где там у вас дублирование, если вы выводите разную разметку. Просто не инициализируйте $gallery 2 раза. вынесите $gallery = get_post_gallery( get_the_ID(), false ); в начало скрипта и проверяйте уже саму переменную в условии if($gallery) или if(count($gallery)). И все равно не ясно, что вы делаете в 3 и 23 строке, что делает get_post_gallery БЕЗ параметров.

[Павел]

olya_097, и что это за шаблон? Одна страница/пост или архив? Если это одиночная запись, то зачем вы два раза выводите цикл WP. Можно весь шаблон от начала и до конца обернуть в <?php while ( have_posts() ) : the_post(); ..... endwhile;?>

[olya_097]

Павел, Одна страница

Answer 1

[Павел]

Если это шаблон одиночной страницы/записи, то можно вот так

<?php while ( have_posts() ) : the_post();
    $gallery = get_post_gallery( get_the_ID(), false );?>
    <div class="content-catalog">
        <?php if ( $gallery ) :
            foreach( $gallery['src'] as $src ) : ?>
                <div class="item-img">
                    <div class="item-img-title"><span>look 1</span></div>
                    <img src="<?php echo $src; ?>" alt="">
                </div>
            <?php
            endforeach;
        endif; ?>
    </div>

    <div class="overlay overlay-img">
        <div class="inner-overlay inner-img">
            <div class="overlay-close"></div>
            <div class="overlay-slider owl-carousel">
                <?php if ( $gallery ) :
                    foreach( $gallery['src'] as $src ) : ?>
                        <div class="slide-item">
                            <img src="<?php echo $src; ?>" alt="#" />
                        </div>
                    <?php
                    endforeach;
                endif; ?>
            </div>
        </div>
    </div>
<?php endwhile; ?>

Comments

[olya_097]

Павел, ваше решение помогло
Спасибо!

Answer 2

[Пашенька]

м?

while ( have_posts() ) : the_post();
      $gallery = get_post_gallery( $post->ID, false );
      if ( !empty($gallery) )  {
            // do something
      }

Comments

[olya_097]

like-a-boss, как понять? вы ж убрали многое

[Пашенька]

olya_097, я убрал ненужное, а не многое

Answer 3

[Антон Р.]

Вынеси в отдельную функцию и вызывай её там где надо.