WPでショートコード用に作った記述（内容）を、single.php などでも使いたい

Question

ショートコード用に作った記述（内容）を、single.php などでも使いたい

WordPressで作成しているサイトにて、商品の通常価格と販売価格を簡単に出力しようと、
投稿記事の管理画面にオリジナルのカスタムフィールドを作り、そこへ数字（価格）だけを入力し、
その情報を投稿の管理画面内にショートコードで出力できるように、functions.phpに以下の記述を作り、
投稿ページでは出力できるようになりました。
###functions.php

<?php
// ショートコード作成
function regular_price_shortcode(){
  $price1 = get_post_meta(get_the_ID(), 'regular_price', true);
  if (empty($price1)){//数値がない場合は何もいれない
    return null;
  }else{
    return '<span class="Price1">¥' . $price1 . ' </span>';
  }
}
add_shortcode( 'regularPrice', 'regular_price_shortcode' );

function current_price_shortcode(){
  $price2 = get_post_meta(get_the_ID(), 'current_price', true);
  if (empty($price2)){//数値がない場合は何もいれない

    return null;
  }else{
    return '<span class="Price2">→ ¥' . $price2 . '</span>';
  }
}
add_shortcode( 'currentPrice', 'current_price_shortcode' );
?>

ですが、この情報を投稿の管理画面外で使いたく（サイドバーやフッターなどでも）、
single.phpなどに以下の記述をしたのですが、
全ての画像が読み込まれなくなり、出力もされてくれません。
（html,cssの文字情報等は問題なし）

###single.php

〜〜〜
<figcaption>
  <?php $price1 = get_post_meta(get_the_ID(), 'regular_price', true);
  if (empty($price1)){//数値がない場合は何もいれない
    return null;
  }else{
    return '<span class="Price1">¥' . $price1 . ' </span>';
  }?>
  <?php $price2 = get_post_meta(get_the_ID(), 'current_price', true);
  if (empty($price2)){//数値がない場合は何もいれない
    return null;
  }else{
    return '<span class="Price2">→ ¥' . $price2 . '</span>';
  }?>
</figcaption>
〜〜〜

だいぶ色々試してみたのですが、
もうお手上げ状態となりましたので、
何が間違い、原因かお分かりになる方おられましたら、
教えていただけないでしょうか？

よろしくお願いいたします。

補足

html,cssに関しては問題なく使用できるのですが、PHPに関してはほぼ初心者です。
見よう見まねで作っていますので、無駄な記述をしているかもしれません。

ですので、もし効率的な書き方があれば、
そちらも合わせて教えていただければ幸いです。

Comments

[CHERRY]

全ての画像というのは、ショートコード以外の画像も含めて表示されないということでしょうか？

[j123]

CHERRYさん、
全ての画像はそのページ全体に使われている画像（SVG以外）がLoadingの状態になって表示されませんでした。
（ショートコードでは数字情報のみを引っ張っています。）

よろしくお願いいたします。

[CHERRY]

遅延読み込みしている画像が表示されないような感じでしょうか？

そのページのHTML は、最後まで読み込まれていますか？　
具体的には、Webブラウザで、htmlソースを確認すると 最後の</html> までありますか？

[j123]

そうです。画像は遅延読み込みさせていたため、その画像が表示されなかったようです。
htmlの閉じタグはありました。

今回は遅延読み込み等の問題ではなかったようですが、
検証の際の目の付け所？を教えていただきありがとうございます！

Answer 1

テンプレート上で普通に出力するならreturnじゃなくecho

emptyだったらnullじゃなく、!emptyだったらechoすれば良いんじゃないか

Comments

[j123]

Space-Monkeyさん、
ご回答ありがとうございます！

「return」を「echo」に変更したところ、
無事表示されてくれました。
ありがとうございます！

ただ、
>emptyだったらnullじゃなく、!emptyだったらechoすれば良いんじゃないか
これの意味が少しわからないのですが、

emptyを使う時は nullを使えない？ということでしょうか？
!emptyとemptyの違いがイマイチ、、、なのですが、
emptyではなく、
!emptyとechoを使えば大丈夫ということでしょうか？

今後のために追加で教えていただけないでしょうか？

よろしくお願いいたします！

[退会済みユーザー]

変数が空か否かで表示を変えるならelseを使えば良いが、そうでないならelseでの分岐は不要。
if (empty($price1)){
echo '$price1は空です'
}else{
echo '$price1は空ではありません'
}
この場合、$price1が空だと表示は『price1は空です』になり、そうでなければ『price1は空ではありません』になる。

だから変数が空ではない場合のみ出力するだけで良いなら
if (!empty($price1)){
echo '$price1は空ではありません'
}
だけで良い。

結局こういうことになる。

<figcaption>
<?php $price1 = get_post_meta(get_the_ID(), 'regular_price', true);
if (!empty($price1)){
echo '<span class="Price1">¥' . $price1 . ' </span>';
}?>
<?php $price2 = get_post_meta(get_the_ID(), 'current_price', true);
if (!empty($price2)){
echo '<span class="Price2">→ ¥' . $price2 . '</span>';
}?>
</figcaption>

これはショートコードの方も同じこと。

尚、独自にincludeしない限りfunction.phpではなくfunctions.phpだろうな。

[j123]

なるほど。そういうことだったんですね。
全てクリアになりました。

かなりスッキリしますね。
まさに求めていた答えです！

教えていただくとわかる（理解できる）のですが、
自力ではなかなか辿り着けないので、
すごく助かります！

ありがとうございました！

P.S. functions.phpですね。何も意識せず書いて間違えていました^^; 訂正しますね。