ループ中にカスタムフィールドが入力されているかで条件分岐したいです。

Question

前提・実現したいこと

ループ中に投稿のカスタムフィールドが入力されているかで条件分岐したいです。

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

何も表示されないです。

該当のソースコード

念の為functions.phpも書きます

  <?php if (have_posts()) : ?>
    <div>
      <?php global $post;
      $args = array(
          'posts_per_page' => 10
      );
      $myposts = get_posts( $args );
      foreach ( $myposts as $post ) : setup_postdata( $post ); ?>
        <section class="post">
          <?php
            if if(!get_post_meta($post->ID, 'img', true)) {
              // カスタムフィールド判別
              the_post_thumbnail('post-thumbnail');
            } else {
              echo "<img src=" .get_post_meta($post->ID, "img", true). ">";
            } ?>
          <div class="post-title">
            <h3><a href="<?php the_permalink() ?>"><?php the_title(); ?></a></h3>
          </div>
        </section>
      <?php endforeach; ?>
    </div>
  <?php else: endif; ?>

functions.php

  // 固定カスタムフィールドボックス
  function add_url_fields() {
    add_meta_box( 'custom_meta_setting', 'img', 'insert_meta_fields', 'post', 'normal');
  }
  add_action('admin_menu', 'add_url_fields');

  // カスタムフィールドの入力エリア
  function insert_meta_fields() {
    global $post;
    echo '<input type="text" name="img" value="'.get_post_meta($post->ID, 'img', true).'" size="50" /><br>';
  }

  // カスタムフィールドの値を保存
  function save_meta_fields( $post_id ) {
    if(!empty($_POST['img'])){
      update_post_meta($post_id, 'img', $_POST['img'] );
    }else{
      delete_post_meta($post_id, 'img');
    }
  }
  add_action('save_post', 'save_meta_fields');

---

修正テキスト
functions.phpに記載

  // small_thum_chk
  function small_thum_chk() {
    if ( get_post_meta($post->ID, 'img', true) ){
      $thumb = '<img src='.get_post_meta($post->ID, 'img', true).'>';
    } elseif ( has_post_thumbnail() ) {
      $thumb = the_post_thumbnail( 'small_thumbnail', array('title' => get_the_title()) );
    } else {
      $thumb = '<img src="https://placehold.jp/eee/aaa/64x64.jpg?text=no-img">';
    }
    echo $thumb;
  }

small_thum_chk()を呼び出した際にのみカスタムフィールドに記載されているURLを記述したimgタグを挿入できるようにしたいです。
発生している事象ですが,
if ( get_post_meta($post->ID, 'img', true) )を認識せず、

elseif ( has_post_thumbnail() ) {
  $thumb = the_post_thumbnail( 'small_thumbnail', array('title' => get_the_title()) );
}

のみ読み込んでいます。
imgのカスタムフィールドは記述しています。

追記

  <?php if (have_posts()) : ?>
    <section class="sidebar_area">
      <p>最近の投稿</p>
      <?php global $post;
      $args = array(
          'posts_per_page' => 4
      );
      $myposts = get_posts( $args );
      foreach ( $myposts as $post ) : setup_postdata( $post ); ?>
      <div class="sidebar_post clearfix">
        <a href="<?php the_permalink() ?>" class="float-left"><?php small_thum_chk(); ?></a>
        <h4 class="float-left"><a href="<?php the_permalink() ?>"><?php the_title(); ?></a></h4>
      </div>
      <?php endforeach; ?>
    </section>
  <?php else: endif; ?>

functions.php

  // small_thum_chk
  function small_thum_chk() {
    if ( get_post_meta($post->ID, 'img', true) ){
      $thumb = '<img src='.get_post_meta($post->ID, 'img', true).'>';
    } elseif ( has_post_thumbnail() ) {
      $thumb = the_post_thumbnail( 'small_thumbnail', array('title' => get_the_title()) );
    } else {
      $thumb = '<img src="https://placehold.jp/eee/aaa/64x64.jpg?text=no-img">';
    }
    echo $thumb;
  }

if ( get_post_meta($post->ID, 'img', true) )がtrueになりません。

---

表示箇所のスクリーンショット

Answer 1

 if if(!get_post_meta($post->ID, 'img', true)) {

ifが重複してます。

Comments

[makoto-n]

回答ありがとうございます。
ifの重複気づきませんでした。

けれど、編集してもカスタムフィールドを挿入できません。
修正を本文に追記します。

[yukikp]

よく見たら、

<?php if (have_posts()) : ?>
<div>
<?php global $post;

ここら辺ですね。このif( have_posts() ): endif;
と、
global $post;
を消してみてください。

[yukikp]

<?php if (have_posts()) : ?>

これって、WordPressが、urlから自動的に判断してデータベースから取ってきたデータの判定しているのですが、あなたの場合、WordPressが自動的に準備したデータは使わず、$argsの条件を元にget_posts()でデータを取得しているので不要です

[makoto-n]

front-pageやサイドバーでそれぞれカテゴリーごとのループを回しています。
なのでそのうちの一つですは
<!-- sidebar-foods -->
<?php if (have_posts()) : ?>
<section class="sidebar_area">
<p>「食材」</p>
<?php global $post;
$args = array(
'posts_per_page' => 4,
'category_name' => 'foods'
);
$myposts = get_posts( $args );
foreach ( $myposts as $post ) : setup_postdata( $post ); ?>
と書いていたりもします。

[yukikp]

テンプレートファイルのコードは後で確かめてみますが、small_thumb_chkの方は、

・has_post_thumbnail()はループ内でしか機能しない
・global $post;が抜けている

と言う点で動かないと思います。

[yukikp]

global $post;
と、関数の最初で宣言して、その後でhas_post_thumbnail( $post->ID )と引数を与えてあげてみてください。

また、the_post_thumbnail()は、引数に投稿IDに取らないので、ループ以外では使いません。get_the_post_thumbnail()を使ってみてください。使い方は

https://wpdocs.osdn.jp/テンプレートタグ/get_the_post_thumbnail

こちらから。

[makoto-n]

small_thumb_chk、ループ内でサムネイルを表示・非表示きりわける関数です。
global $postが設定されたループ内で使用し、特に異常は無いのでているのでこの関数自体には問題は無いと思います。

[yukikp]

それと、small_thumb_chk関数って、問題のコードで使ってないようですが、なぜ急に出てきたのでしょう？？

質問と関係あるのでしょうか？

[makoto-n]

最初に記述したphpコードは不適切でした。
なるべく必要最小限にと思ったのですが、
なので使用している箇所のコードをすべて追記します。

[yukikp]

読み返してみると、こちはの質問も散漫で。なんだかいちゃもんつけてるように見えますね。。。
気を悪くされてたらごめんなさい。

[makoto-n]

いえ、こちらの提示したコードが不適切でした。
気を悪くさせてしまい申し訳ないです。

[YukiYamashina]

> global $postが設定されたループ内で使用し、特に異常は無いのでているのでこの関数自体には問題は無いと思います。

グローバル変数の $post を参照するのであれば、どこで呼び出すかに関わらず small_thumb_chk 関数内に global $post; がないと期待通りに動きません。

[makoto-n]

該当箇所のスクリーンショットを追記しました。
function small_thum_chk()は問題なく動いているんです。
if ( get_post_meta($post->ID, 'img', true) )がtrueにならないんです。

[YukiYamashina]

> if ( get_post_meta($post->ID, 'img', true) )がtrueにならないんです。

global $post; を書いていないからです。global $post; を書かないと $post->ID は NULL となり、get_post_meta 関数は false しか返しません。

[yukikp]

まあまあ、そう固いこと言わずに、global $post;を関数内の最初で宣言してみましょ！

[makoto-n]

> $global $post; を書いていないからです。
small_thumb_chk()内に記述したところ反映できました！
呼び出し元で記述しているなら大丈夫と思っていました...
ありがとうございます。

[yukikp]

よかったですね〜

[makoto-n]

お二方本当にありがとうございました。
質問も理解も未熟で申し訳なかったです。