Smart Custom Fieldsを使った繰り返し画像のスライドショー　foreachでの条件分岐

前提・実現したいこと

Smart Custom Fieldsを使って繰り返し画像のフィールドを作成し、
複数枚画像がある場合はスライド表示、画像がなければアイキャッチ画像を表示させたいです。

WordPressのカスタム投稿記事をモーダルで表示させています。
下記がモーダルの中身部分のコードです。

<table>タグでAdvanced Custom Fieldsで作成したカスタムフィールドを表示させています。

その横に、Smart Custom Fieldsで作成した繰り返しフィールドに投稿された画像を
スライダー表示させ、もし繰り返しフィールドに画像が登録されていなければアイキャッチ画像を表示させたいです。

質問に不備や失礼な点があれば大変申し訳ありません。

▪️ここに質問の内容を詳しく書いてください。
foreachを使って条件分岐させたのですが、表示されません。
foreach文の箇所にエラーがあると思うのですが、原因がわかりません。

発生している問題・エラーメッセージ

foreachを使って条件分岐させたのですが、表示されません。

phpは勉強不足で調べながら記述しましたので、完全に理解しきれておりません。
ご教授いただけますと幸いです。

<?php
      foreach( $catlists as $cat ) :
        $args = array(
          'post_type' => 'test',
          'posts_per_page' => -1,
        );

        $my_posts = get_posts( $args );
        if ( $my_posts ) {
          foreach ( $my_posts as $post ) :
          setup_postdata( $post );
          echo '<div class="remodal" data-remodal-id="'.$post->ID .'" data-remodal-options="hashTracking:false">';
          echo '<button data-remodal-action="close" class="remodal-btn1 _trigger">
            <span></span>
            <span></span>
          </button>';

          echo '<div class="remofal-inr">';
          echo '<div class="wp-slick"">';
          
          $imgGroup = SCF::get('画像');
          $imgurl = wp_get_attachment_image_src($fields['images'] , 'large');
             
          foreach ($imgGroup as $fields){
           if ($fields['images'] === "" ) {
            $image = get_post_meta($post->ID, '_thumbnail_id', true);
            echo wp_get_attachment_image($image, 'medium_large');

           }else {
            echo "<li>";
            echo wp_get_attachment_image($imgurl, 'medium_large');
            echo "</li>";
           }
            }
          echo "</div>";
          echo '<div class="remodal-right">';
          echo '<h2 class="remodal-ttl">';
          echo the_title();
          echo '</h2>';
          echo '<table class="remodal_table">';
          echo "<tr>";
          echo "<th>bbbb</th>";
          echo "<td>";
          the_field( 'bbbbb' );
          echo "</td>";
          echo "</tr>";

          echo "<tr>";
          echo "<th>aaaaa</th>";
          echo "<td>";
          the_field( 'aaaaa' );
          echo "</td>";
          echo "</tr>";
          
          echo "</table>";
          echo "</div>";
          echo "</div>";
          $content = get_the_content();
          echo '<p class="remodal-lead">'. $content . '</p>';
          echo '<button data-remodal-action="close" class="remodal-btn2">
            <span class="remodal-btn2-in">閉じる</span>
          </button>';
          echo "</div>";
          endforeach;
        }
        wp_reset_postdata();
      endforeach;
    ?>

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

php 7.3.5
WordPress 5.6.2
Smart Custom Fields　4.1.5

ここにより詳細な情報を記載してください。

退会済みユーザー

2021/03/09 09:19

コードはバッククォート3つで囲っておくれ ```コード```

moooooun

2021/03/09 09:51

大変申し訳ありません。

退会済みユーザー

2021/03/09 12:54

$catがどこにも使われてないからforeach( $catlists as $cat ) :と最後のendforeach;は不要では？あと、$fieldsはどこから出て来た？あと、echo echo echoっていっぱい書いてるが、別に悪かないけど一旦 ?> で閉じてしまえばHTMLタグが書けるから、そうすりゃecho echoとしなくても良い。あと、アイキャッチはget_the_post_thumbnail()で出力できる。 https://developer.wordpress.org/reference/functions/get_the_post_thumbnail/ とりあえず、よー分からんとこありすぎなのでもう少し整理したらどうだ。

行動規範の内容に同意します

回答1件

ベストアンサー

以下の部分、$imgurl = の行はelse内にあるべきでは？

php
1$imgGroup = SCF::get('画像');
2//$imgurl = wp_get_attachment_image_src($fields['images'] , 'large');#else内に移動
3
4foreach ($imgGroup as $fields){
5
6 if ($fields['images'] === "" ) {
7 
8  $image = get_post_meta($post->ID, '_thumbnail_id', true);
9  echo wp_get_attachment_image($image, 'medium_large');
10
11 }else {
12 
13  $imgurl = wp_get_attachment_image_src($fields['images'] , 'large');#ここに移動
14
15  echo "<li>";
16  echo wp_get_attachment_image($imgurl, 'medium_large');
17  echo "</li>";
18 }
19}