固定ページのカスタムフィールドに設定した複数画像をトップページでも出したい

Question

###前提・実現したいこと
固定ページで並べて表示した画像をトップページのスライドと連動するようにしたいです。かつ、固定ページの編集ページで写真を増やせるようにしたいです。

宜しくお願い致します。

出力されるソースとしては写真の数だけ<li>の中に<img>としたいです。

<li><img src="xxxx"></li>
<li><img src="xxxx"></li>

###試したこと
カスタムフィールドに写真を登録し、どちらもショートコードで呼び出そうとしましたが、うまくいきません。

###発生している問題・エラーメッセージ
１枚の写真を出すことはできましたが、複数枚を出すことができません。

###該当のソースコード
functions.phpに下記ショートコードを作りました。
現状<li>タグは無視して進めてます。

//写真呼出し用
function imgFunc() {
    $size = 'full';
    $img = get_post_meta(get_the_ID(325), '写真', true);
    return 
    wp_get_attachment_image($img, $size);
}
add_shortcode('img', 'imgFunc');

修正後1

//写真呼出し用
function imgFunc() {
    $img_ids = get_post_meta(get_the_ID(325), '写真', false);
    // echo '<pre>'.var_dump($img_ids).'</pre>';
    for($i=0;$i<$img_ids;$i++){
        $img = wp_get_attachment_image_src($img_ids[$i], 'full');
        echo '<li><img src="'.$img[0].'"></li>';
    }
}
add_shortcode('img', 'imgFunc');

修正後2

//写真呼出し用
function imgFunc() {
    $img_ids = get_post_meta(get_the_ID(325), '写真', false);
    // echo '<pre>'.var_dump($img_ids).'</pre>';
    for($i=0;$i<$img_ids;$i++){
        $img = wp_get_attachment_image_src($img_ids[$i], 'full');
        if($img){
            echo '<li><img src="'.$img[0].'"></li>';
        } else {
            return false;
        }
    }
}
add_shortcode('img', 'imgFunc');

修正後3

//写真呼出し用
function imgFunc() {
    $img_ids = get_post_meta(get_the_ID(325), '写真', false);
    // echo '<pre>'.var_dump($img_ids).'</pre>';
    for($i=0;$i<$img_ids;$i++){
        $img = wp_get_attachment_image_src($img_ids[$i], 'full');
        if($img){
            $output .= '<li><img src="'.$img[0].'"></li>';
        } else {
            return false;
        }
    } 
    return $output;
}
add_shortcode('img', 'imgFunc');

Answer 1

配列で取得すればよいのでは？

【get_post_meta:WordPress私的マニュアル】
http://elearn.jp/wpman/function/get_post_meta.html

返り値 パラメータ$singleがtrueの場合は、投稿データにパラメータ$keyの名前で登録されている1番目のカスタムフィールドの値を文字列として返す。パラメータ$singleがfalseの場合は、投稿データにパラメータ$keyの名前で登録されているすべてのカスタムフィールドの値を配列で返す。

Comments

[raidomaru]

ありがとうございます。falseにして修正後のようにしたところ、画像は表示されるのですが、画像がなくても延々と繰り返されてしまいます。

[kei344]

$imgに画像が入っているか確認すればよいのでは？

[raidomaru]

判定を入れて修正後2にしました。
表示される写真の数は正常になったのですが、なぜか表示位置が固定ページにショートコードを書いた場所ではなく、header.phpのソース直後になってしまいます。

アドバイスいただけますと幸いです。

[kei344]

なぜショートコードにこだわるかわかりませんが、使う場所で「imgFunc();」とすればよいと思います。「なぜ／どうして」に答えられるだけの情報が無いのでこれ以上はちょっとわかりません。

[raidomaru]

ショートコードなくても固定ページで設定した画像をトップページのスライドと連動できればいいのですが、ショートコードでできるのではと思ったためです。
もっと簡単な方法などありますでしょうか？

[kei344]

使う場所で「imgFunc();」

[raidomaru]

固定ページだと使えないですよね？

[kei344]

functions.php に書いた関数が固定ページで動かないというのは聞いたことがありませんが･･･。

[raidomaru]

http://wp-technique.com/php-short/
こちらを見てphpで作っているトップと固定ページ両方で使えるショートコードを作ろうと考えたのですが。

[kei344]

ああ、トップのテンプレートで書いているものだとばかり思っていました。現在書かれているコードは特定のページのみ読み出すものなので。

すみませんが、どういったコードをかかれているかがわからないので、お役に立てる気がしません。

[raidomaru]

色々とありがとうございました。やり方も含めて考えてみます。

[退会済みユーザー]

横やり失礼します。
ショートコードなので、echoで出力するのではなく、returnしなければいけないと思います。
https://wpdocs.osdn.jp/%E3%82%B7%E3%83%A7%E3%83%BC%E3%83%88%E3%82%B3%E3%83%BC%E3%83%89_API#.E5.87.BA.E5.8A.9B

[raidomaru]

ありがとうございます。自分でもちょうど調べてました。returnで書き換えてみます。
http://my-dictionary.info/internet/wordpres-%E3%82%B7%E3%83%A7%E3%83%BC%E3%83%88%E3%82%B3%E3%83%BC%E3%83%89%E3%82%92%E8%A1%A8%E7%A4%BA%E3%81%95%E3%81%9B%E3%82%8B%E3%81%A8%E3%81%8D%E3%81%AFecho%E3%81%AF%E4%BD%BF%E3%81%A3%E3%81%A6/

[raidomaru]

returnにしたら1つしか出力されなくなってしまいました。returnを使うとそこで繰り返し処理が終わってしまうのでしょうか。

[退会済みユーザー]

はい、そうです。(http://www.phpbook.jp/tutorial/function/index4.html)

なので、関数の途中では、$output .= '<li><img src="'.$img[0].'"></li>';として、
関数の最後で、return $output;としてください。（変数名はなんでもいいですが）

[raidomaru]

修正後3の通りにしたところ、画像が１つも表示されなくなってしまいました。

[退会済みユーザー]

return false;の分岐に入ってしまっていないか確認してみてください。
それから気になっていたのですが、`for($i=0;$i<$img_ids;$i++)`って`for($i=0;$i<count($img_ids);$i++)`としたほうがいいような……。

[raidomaru]

ありがとうございます。ご指摘の通りcountを使用したらできました。
あとはindex.phpでショートコードを実行するとArrayになってしまうので、こちらを考えてみます。

[退会済みユーザー]

> ショートコードを実行するとArrayになってしまうので、こちらを考えてみます。
ショートコードの中で配列をreturnしてませんか？ショートコードの中でreturnしてよいのは、文字列だけです。

[raidomaru]

してます。複数枚の写真を出力したいのですが、配列以外でできるのですか？

[退会済みユーザー]

> してます。
配列は Echo しても Array と表示されるだけです。これは WordPress 関係なく PHP の仕様です。add_shortcode フックのコールバック関数内で return に指定できるのは文字列だけです。なので、img タグをリターンするようにしてみてください。