Cocoonフォーラム

サイト内検索
書き込みの前に以下の3点をご確認ください。
  1. 1つのトピックにつき1つの質問を書き込んでください
  2. 不具合・カスタマイズ対象ページのURLを提示高速化を無効にしてください
  3. 該当部分のキャプチャ・環境情報とともに書き込んでいただけると助かります

何を書き込んだら良いか分からない場合は、以下のテンプレートをコピペしてご利用ください。

不具合・カスタマイズ対象ページのURL:

相談内容:

不具合の発生手順:

解決のために試したこと:

※文字だけでは正しく伝わらない可能性があるため、スクショ画像の添付もお願いします。
※高速化設定をしている場合は無効にしてください。
環境情報:

※↑こちらに「Cocoon設定 → テーマ情報」にある「環境情報」を貼り付けてください。

環境情報の取得方法はこちら。
https://wp-cocoon.com/theme-report/
高速化設定を無効にするにはこちら。
https://wp-cocoon.com/theme-trouble/

フォーラム利用ガイドリンク

  1. フォーラムガイドライン
  2. よくある質問と答え(FAQ)
  3. サポート対象外のケース
  4. 原因不明の不具合用トラブルシューティング
  5. トピックにHTMLを貼り付ける方法(推奨ツール:notepad.pw
  6. 真っ白画面でのエラーメッセージの確認方法
  7. ブラウザ環境チェックツール
  8. Cocoonカスタマイズ依頼

フォーラム質問後、問題等が解決した場合は結果を書き込んでいただけると幸いです。同様の問題で調べている方には、結果が一番気になる部分となります。

このフォーラムには初心者の方も多く参加されています。質問に不慣れな方も安心して相談できるよう、できるだけ穏やかな言葉でご案内いただけますと幸いです。

存在しないカテゴリーIDを指定して検索...
 
通知
すべてクリア

存在しないカテゴリーIDを指定して検索した際のPHP 8.1エラーについて

1 投稿
1 ユーザー
0 Reactions
12 表示
大門未知子
(@chu-ya)
Illustrious Member Registered
結合: 4年前
投稿: 4526
トピックスターター  

一見、過剰なチェックにも思え、以下の対応有無と判断願いたく。

----------------------------------------------
テーマ名:Cocoon
バージョン:2.9.3.3
カテゴリー数:18
タグ数:123
投稿数:1752
固定ページ数:12
ユーザー数:1
----------------------------------------------

●関連

Xwriteフォーラムの横展開です。

https://users.xwrite.jp/community/postid/5284/


●現象

URLに ?s=検索キーワード&cat=99999 のように「存在しないカテゴリーID」を指定してサイト内検索を行った際、以下のエラー(WarningおよびDeprecated)が発生する。

Warning: Attempt to read property "name" on null in /lib/content-category.php on line 90
Deprecated: htmlspecialchars(): Passing null to parameter #1 ($string) of type string is deprecated in /lib/seo.php on line 450

 


●原因

存在しないカテゴリーIDが指定された際、get_category($cat_id) が null を返す。
しかし、コード内で戻り値のチェックを行わずに直接 ->name プロパティを参照しているため、Warningが発生する。

https://github.com/xserver-inc/cocoon/blob/af985e04cd052c93a8d8226aff1485fcad3f2284/lib/content-category.php#L89-L91

 

メタキーワードを生成する際、上記の影響でカテゴリー名(またはタグ名)が取得できず、変数 $res が null になる。
PHP 8.1以降では htmlspecialchars() に null を渡すことが非推奨となったため、Deprecated警告が発生する。

https://github.com/xserver-inc/cocoon/blob/af985e04cd052c93a8d8226aff1485fcad3f2284/lib/seo.php#L450

https://github.com/xserver-inc/cocoon/blob/af985e04cd052c93a8d8226aff1485fcad3f2284/lib/seo.php#L628


●対策

get_category() の戻り値オブジェクトの存在確認を挟んでから name を取得するように変更する。

  //タイトルが存在しない場合はカテゴリー名を利用する
  if (!$res && $is_cat_name) {
    $category = get_category($cat_id);
    // カテゴリが正常に取得できた場合のみ name を取得
    if ($category && !is_wp_error($category)) {
      $res = $category->name;
    }
  }

 

htmlspecialchars() に渡す前に、変数を型キャストして null を空文字 "" に変換する。

  $res = htmlspecialchars((string)$res);

このトピックは2時間前 2回から大門未知子に変更されました

   
引用
共有:

問題の解決に至った場合には、トピック冒頭の「解決済み」をクリックしていただけますと幸いです。

また、有用な回答があった場合は返信右下にある「いいね!」もご活用ください。回答者の励みになります。

「いいね!」機能はフォーラム登録者のみが利用できる機能です。

CC BY-ND 2.1)準じていれば(リンクを貼っていただければ)転載も自由です。カスタマイズ記事を書く際にコード等をコピペ利用していただいて構いません。

フォーラムの使い方がよくわからない場合は、テストトピックで自由にテストしていただいて構いません。

最近の書き込みはこちら。

詳細なカスタマイズ依頼をするならこちら。

タイトルとURLをコピーしました