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カスタマイズ依頼

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

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

エントリーカード(tmp/entry-...
 
通知
すべてクリア

エントリーカード(tmp/entry-card.php)における変数 $count の競合と未定義エラーの改善提案

6 投稿
3 ユーザー
4 Reactions
214 表示
(@shame-stamp)
New Member Registered
結合: 2か月前
投稿: 1
トピックスターター  

わいひら様
いつも素晴らしいテーマをありがとうございます。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
引用
大門未知子
(@chu-ya)
Illustrious Member Registered
結合: 4年前
投稿: 4405
 

言われている通りですね。

エントリーカードの何枚目を処理しているか以下でset_query_varを用い、$countをテンプレートに引き渡しており。

https://github.com/xserver-inc/cocoon/blob/cb686489a00fcf85e5e33f40f5d88e3958e996a1/tmp/list-tab-index.php#L92

https://github.com/xserver-inc/cocoon/blob/cb686489a00fcf85e5e33f40f5d88e3958e996a1/tmp/entry-card.php#L14

さらに、エントリーカードのコメント数を$countに格納し、上書きし壊してしまい。

ユニークな変数名とする必要がありますね


●余談

以前話しましたが、今後を考え、set_query_varを止め、 WordPress 5.5でサポートされた、get_template_part($slug, $name, $args)の第三パラメータを用い、テンプレートに安全にパラメータを渡す(修正多いですが)。



   
わいひら reacted
返信引用
わいひら
(@yhira)
Illustrious Memberサイト Admin
結合: 8年前
投稿: 18466
 

@shame-stamp さん

ご連絡いただきありがとうございます。
次のように修正させていただきました。
https://github.com/xserver-inc/cocoon/commit/a68f012663a8750fd1b760e4fdc7c1124ea6e275

投稿者:: @chu-ya

以前話しましたが、今後を考え、set_query_varを止め、 WordPress 5.5でサポートされた、get_template_part($slug, $name, $args)の第三パラメータを用い、テンプレートに安全にパラメータを渡す(修正多いですが)。

ご提案ありがとうございます。
第三引数を用いる方法は影響範囲が広い上に破壊的な変更となるため、僕として行うのはかなりリスクが高いと考えています。ひとまず変数名の競合を回避する形の安全な対策にさせていただきました。

 



   
返信引用
大門未知子
(@chu-ya)
Illustrious Member Registered
結合: 4年前
投稿: 4405
 

投稿者:: @yhira

第三引数を用いる方法は影響範囲が広い上に破壊的な変更となるため、僕として行うのはかなりリスクが高いと考えています。ひとまず変数名の競合を回避する形の安全な対策にさせていただきました。

set_query_varを全部変更するのは影響があることは理解しています。

今からでもcocoon_template_partの口だけ準備し、徐々にset_query_varを廃止するのが望ましいかと思います。

●ご参考

https://notepad.pw/share/3rp63535g



   
わいひら reacted
返信引用
わいひら
(@yhira)
Illustrious Memberサイト Admin
結合: 8年前
投稿: 18466
 

投稿者:: @chu-ya

今からでもcocoon_template_partの口だけ準備し、徐々にset_query_varを廃止するのが望ましいかと思います。

それはそうですね。
次回からこういったテンプレート追加するときはset_query_varは使わず引数を使用していくようにします。
https://github.com/xserver-inc/cocoon/commit/b36d99006020c7a9f699eac0cc5a770a81dcfb6d



   
返信引用
大門未知子
(@chu-ya)
Illustrious Member Registered
結合: 4年前
投稿: 4405
 

@yhira さん

以下の件でスキンソースを変更しており。
今回気付きましたが、変更の際は連絡ください。

https://github.com/xserver-inc/cocoon/commit/d9d94c1c4e7345836cc785a44bf6c42985d0d70e#diff-3138c46940a19d6f701e3d5a0c7d382f74a640f47633beea9dad5e7eec0469ef



   
わいひら reacted
返信引用
共有:

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

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

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

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

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

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

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

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