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

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

目次の表示条件(nつ以上見出しがあると...
 
共有:
通知
すべてクリア

[解決済] 目次の表示条件(nつ以上見出しがあるとき)通りとならない

8 投稿
3 ユーザー
10 Reactions
340 表示
大門未知子
(@chu-ya)
Illustrious Member Registered
結合: 4年前
投稿: 4024
トピックスターター  

----------------------------------------------
テーマ名:Cocoon
バージョン:2.8.8.2
カテゴリー数:18
タグ数:123
ユーザー数:1
----------------------------------------------

●設定

[Cocoon設定]→[目次]→目次の表示
「目次を表示する」オン
「本文全ての目次を表示する」オン


●現象

本文の見出しは3つしかない。
しかし、表示条件「4つ以上見出しがあるとき」としても目次が表示される。

<!-- wp:heading -->
<h2 class="wp-block-heading">H2</h2>
<!-- /wp:heading -->

<!-- wp:heading {"level":3} -->
<h3 class="wp-block-heading">H3</h3>
<!-- /wp:heading -->

<!-- wp:heading -->
<h2 class="wp-block-heading">H2</h2>
<!-- /wp:heading -->

●原因

分割ページのとき、全ページの見出し数をカウントする。
その際、変数$counterとした。

https://github.com/xserver-inc/cocoon/blob/9b442ccb83e32dfd19ef42613329695f6744628d/lib/toc.php#L101-L130

しかし、見出し数を求める際、変数$counterとしており、重複する。

https://github.com/xserver-inc/cocoon/blob/9b442ccb83e32dfd19ef42613329695f6744628d/lib/toc.php#L186

結果、見出し数が不正となってしまう(図参照)。


●対策

以下で使用する変数を$toc_countに変更する。

https://github.com/xserver-inc/cocoon/blob/9b442ccb83e32dfd19ef42613329695f6744628d/lib/toc.php#L101-L130

    $toc_counter = 0;
    // 分割ページごとに見出しを取得
    $page_num = 1;
    $past_page_num = 1;
    foreach ($pages as $page_content) {
      if (preg_match_all('/<([hH][1-6]).*?>(.*?)<\/[hH][1-6]>/us', $page_content, $matches, PREG_SET_ORDER)) {
        foreach ($matches as $m) {
          // 違うページになったらカウンターをリセットする
          if ($page_num !== $past_page_num) {
            $past_page_num = $page_num;
            $toc_counter = 0;
          }

          // 目次の深さ取得
          $now_depth = intval(substr(strtolower($m[1]), 1, 1));

          // $set_depth より深い見出しは目次に追加しない
          if ($now_depth <= $set_depth) {
            $toc_counter++;
            $headers[] = array(
              'tag'  => $m[1],
              'text' => $m[2],
              'id'   => 'toc' . $toc_counter,
              'page' => $page_num,
            );
          }
        }
      }
      $page_num++;
    }
    $header_count = count($headers);

 



エックスサーバー 高澤
(@s_takazawa)
Estimable Member Moderator
結合: 1年前
投稿: 155
 

@chu-ya さん

ご指摘いただき、ありがとうございます。

私の方でも先日ご指摘いただきました以下PRの対応にて、同様の不具合を確認しておりました。

https://github.com/xserver-inc/cocoon/pull/315

いただきましたコードを参考にさせていただきつつ、対応いたします。


この投稿は1か月前 2回ずつエックスサーバー 高澤に変更されました

   
わいひら reacted
大門未知子
(@chu-ya)
Illustrious Member Registered
結合: 4年前
投稿: 4024
トピックスターター  

@s_takazawa さん

Cocoon 2.8.9がリリースされ、分割ページサポートが公開されています。
本問題の対応願います。



エックスサーバー 高澤
(@s_takazawa)
Estimable Member Moderator
結合: 1年前
投稿: 155
 

@chu-ya さん

長らくお待たせいたしました。
コードのご提供も誠にありがとうございます。
本件に関して対応が完了いたしましたので、わいひらさんへご共有できればと思います。
少々お待ちいただけましたら幸いです。



   
わいひら reacted
エックスサーバー 高澤
(@s_takazawa)
Estimable Member Moderator
結合: 1年前
投稿: 155
 

@yhira

わいひらさん

大変長らくお待たせしております。
本件に関して、PRを作成し対応いたしましたので、お手すきでご確認のほどよろしくお願いいたします。

PR:
https://github.com/xserver-inc/cocoon/pull/318

(対応内容)
・lib\toc.phpのget_toc_tag()関数にて、「// 分割ページごとに見出しを取得」の処理の箇所の変数名を $counter から $toc_counter に変更

何か修正点等ありましたら、お気軽にお申しつけください。
よろしくお願いいたします。


この投稿は2週間前ずつエックスサーバー 高澤に変更されました

   
わいひら reacted
大門未知子
(@chu-ya)
Illustrious Member Registered
結合: 4年前
投稿: 4024
トピックスターター  

----------------------------------------------
テーマ名:Cocoon
バージョン:2.8.9.1
カテゴリー数:18
タグ数:123
ユーザー数:1
----------------------------------------------
修正されていることを確認しました。
本件をクローズとします。



わいひら
(@yhira)
Illustrious Memberサイト Admin
結合: 8年前
投稿: 18177
 

ご対応いただきありがとうございます!
プルリクエストをマージさせて頂きました。



わいひら
(@yhira)
Illustrious Memberサイト Admin
結合: 8年前
投稿: 18177
 

すでに確認済みでしたか!
ご確認いただきありがとうございます。



共有:

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

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

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

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

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

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

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

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