サイト内検索
Cocoonフォーラム
書き込みの前に以下の3点をご確認ください。
何を書き込んだら良いか分からない場合は、以下のテンプレートをコピペしてご利用ください。
不具合・カスタマイズ対象ページのURL:
相談内容:
不具合の発生手順:
解決のために試したこと:
※文字だけでは正しく伝わらない可能性があるため、スクショ画像の添付もお願いします。
※高速化設定をしている場合は無効にしてください。
環境情報:※↑こちらに「Cocoon設定 → テーマ情報」にある「環境情報」を貼り付けてください。
環境情報の取得方法はこちら。
→ https://wp-cocoon.com/theme-report/
高速化設定を無効にするにはこちら。
→ https://wp-cocoon.com/theme-trouble/
フォーラム利用ガイドリンク
- フォーラムガイドライン
- よくある質問と答え(FAQ)
- サポート対象外のケース
- 原因不明の不具合用トラブルシューティング
- トピックにHTMLを貼り付ける方法(推奨ツール:notepad.pw)
- 真っ白画面でのエラーメッセージの確認方法
- ブラウザ環境チェックツール
- Cocoonカスタマイズ依頼
フォーラム質問後、問題等が解決した場合は結果を書き込んでいただけると幸いです。同様の問題で調べている方には、結果が一番気になる部分となります。
このフォーラムには初心者の方も多く参加されています。質問に不慣れな方も安心して相談できるよう、できるだけ穏やかな言葉でご案内いただけますと幸いです。
通知
すべてクリア
トピックスターター 2026年3月21日 23:29
わいひら様
いつも素晴らしいテーマをありがとうございます。Cocoonを愛用させていただいております。
現在配布されているCocoonの tmp/entry-card.php において、PHPのエラーログに Undefined variable $count が出力されるケースを確認いたしました。原因を調査したところ、変数のバグと未定義参照が混在しているようですので、修正のフィードバックをさせていただきます。
【発生している問題】
変数の競合(上書き):
tmp/entry-card.php の後半(コメント数表示セクション)で $count = get_comments_number(); と記述されており、ループの順序管理用である $count がコメント数で上書きされています。これにより、後続の処理やフックで「何番目の記事か」という情報が破壊されます。
未定義変数の参照:
関連記事や特定のウィジェットからカードが呼び出される際、呼び出し元で $count が set_query_var されていないと、27行目の get_entry_card_thumbnail_size($count) 等で Undefined variable エラーが発生します。
【改善案のコード】
変数の衝突を避け、かつ未定義時にもデフォルト値を担保する以下の修正を提案いたします。
PHP
// ファイル冒頭で安全に変数を取得(優先順位: query_var > global > 0)
$current_loop_count = get_query_var('count');
if ($current_loop_count === '') {
global $count;
$current_loop_count = (isset($count) && is_numeric($count)) ? (int)$count : 0;
}
// (中略:$count を $current_loop_count に差し替えて利用)
// 後半のコメント数取得:変数名を変更して競合を避ける
$comment_num = get_comments_number();
if(is_entry_card_post_comment_count_visible() && ...):
// $comment_num を使用
endif;
このように修正することで、どのテンプレートやウィジェットから呼び出されてもエラーログが出なくなり、テーマの堅牢性が高まると考えられます。
お忙しいところ恐縮ですが、次回のアップデート等でご検討いただけますと幸いです。
よろしくお願いいたします。
このトピックは2か月前からshame-stampに変更されました
わいひら reacted
2026年3月22日 01:31
言われている通りですね。
エントリーカードの何枚目を処理しているか以下でset_query_varを用い、$countをテンプレートに引き渡しており。
さらに、エントリーカードのコメント数を$countに格納し、上書きし壊してしまい。
ユニークな変数名とする必要がありますね。
●余談
以前話しましたが、今後を考え、set_query_varを止め、 WordPress 5.5でサポートされた、get_template_part($slug, $name, $args)の第三パラメータを用い、テンプレートに安全にパラメータを渡す(修正多いですが)。
わいひら reacted
2026年3月22日 23:55
@shame-stamp さん
ご連絡いただきありがとうございます。
次のように修正させていただきました。
https://github.com/xserver-inc/cocoon/commit/a68f012663a8750fd1b760e4fdc7c1124ea6e275
以前話しましたが、今後を考え、set_query_varを止め、 WordPress 5.5でサポートされた、get_template_part($slug, $name, $args)の第三パラメータを用い、テンプレートに安全にパラメータを渡す(修正多いですが)。
ご提案ありがとうございます。
第三引数を用いる方法は影響範囲が広い上に破壊的な変更となるため、僕として行うのはかなりリスクが高いと考えています。ひとまず変数名の競合を回避する形の安全な対策にさせていただきました。
2026年3月23日 00:19
第三引数を用いる方法は影響範囲が広い上に破壊的な変更となるため、僕として行うのはかなりリスクが高いと考えています。ひとまず変数名の競合を回避する形の安全な対策にさせていただきました。
set_query_varを全部変更するのは影響があることは理解しています。
今からでもcocoon_template_partの口だけ準備し、徐々にset_query_varを廃止するのが望ましいかと思います。
●ご参考
https://notepad.pw/share/3rp63535g
わいひら reacted
2026年3月23日 23:38
今からでもcocoon_template_partの口だけ準備し、徐々にset_query_varを廃止するのが望ましいかと思います。
それはそうですね。
次回からこういったテンプレート追加するときはset_query_varは使わず引数を使用していくようにします。
https://github.com/xserver-inc/cocoon/commit/b36d99006020c7a9f699eac0cc5a770a81dcfb6d
問題の解決に至った場合には、トピック冒頭の「解決済み」をクリックしていただけますと幸いです。
また、有用な回答があった場合は返信右下にある「いいね!」もご活用ください。回答者の励みになります。
(CC BY-ND 2.1)準じていれば(リンクを貼っていただければ)転載も自由です。カスタマイズ記事を書く際にコード等をコピペ利用していただいて構いません。
フォーラムの使い方がよくわからない場合は、テストトピックで自由にテストしていただいて構いません。
最近の書き込みはこちら。
詳細なカスタマイズ依頼をするならこちら。


