php「Notice: Undefined variable」エラーの解決方法

Question

実現したいこと

phpの知識が全くない初心者です。

こちらのサイト（ https://propagandes.info/wordpress-news/ ）を参考に、HP内にお知らせ欄を作りたいと考えているのですが、「Warning: Undefined variable $html in ○○ on line 547」というエラーが発生してしまいました。

なお一緒に紹介されているプラグインの方は現在インストールできない状態になっており、プラグインなしの方を実装するつもりで考えております。

発生している問題・分からないこと

該当サイトのコードをコピーペーストしたところ、以下のようなエラーが発生しました。

エラーメッセージ

Warning: Undefined variable $html in ○○ on line 547
（該当ソースコードの「$html .= '<li>';」が547行目に当たります）

該当のソースコード

function create_post_type_news(){
 register_post_type( 
  'news',
  array(
   'labels' => array(
    'name' => 'お知らせ'
   ),
   'public' => true,
   'has_archive' => true,
   'supports' => array('title','editor','thumbnail','author'),
   'show_in_rest' => true,
  )
 );
}
add_action( 'init', 'create_post_type_news' );

function shortcode_news_list() {
 global $post;
 $args = array(
  'posts_per_page' => 3,  // 一覧に表示させる件数
  'post_type' => 'news',  // お知らせのスラッグ
  'post_status' => 'publish'
 );
 $the_query = new WP_Query( $args );
 // お知らせ一覧用HTMLコード作成
 if ( $the_query->have_posts() ) {
  $html .= '<ul>';
  while ( $the_query->have_posts() ) :
  $the_query->the_post();
  $url = get_permalink();
  $title = get_the_title();
  $date = get_the_date('Y/m/d');
  $html .= '<li>';
  $html .= '<a href="'.$url.'">';
  $html .= '<p class="news_date">'.$date.'</p>';
  $html .= '<h3 class="news_title">'.$title.'</h3>';
  $html .= '</a></li>';
  endwhile;
  $html .= '</ul>';
 }
 return $html;
}
add_shortcode("news_list", "shortcode_news_list");

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

php7からphp8へのアップデートによるエラーであることまでは分かったのですが、具体的な解決方法が分かりませんでした。isset()という関数を使用するのだと思いますが、phpの記述の仕方、またどこを修正すべきなのかが分かりませんでした。

補足

特になし

Answer 1

Warning: Undefined variable $html in ○○ on line 547
（該当ソースコードの「$html .= '<li>';」が547行目に当たります）

件の行は、`$html = $html . '<li>'; と同等です。
なので、未定義の変数が右辺に使われていると判断されたのでは？

その直前の if 以前で初期化しておけば良いだろうと思います。

---

phpの知識が全くない初心者

謙遜だとは思いますが、本当に「初心者」でしたら、WordPressの改造は控えるべきだと思います。
WordPressを教材に学習をしようとするのも避けるべきだと思います。

Answer 2

$htmlという変数を初期化してください

if ( $the_query->have_posts() ) {
  $html .= '<ul>';

↓↓↓

$html="";
if ( $the_query->have_posts() ) {
  $html .= '<ul>';

Comments

[aya_56]

回答ありがとうございます。
こちらで試したところ問題が解決しました。本当にありがとうございました。