[ページ管理] ページ一覧で一部のページが表示されない

[masaton0216]

概要

ページ管理のページ一覧（/manage/page）で、一部のページ行が画面に表示されません。同じページがページ権限一覧（/manage/page/roleList）には正常に表示されるため、データが失われているわけではなく表示側の問題です。顧客問い合わせから発覚しました。

混入元: PR #2417「[ページ管理] ページ一覧で親ページごとに折り畳めるようにしました」（マージコミット 7040aa20f）
影響バージョン: v1.43.0 以降すべて（v1.44.0 を含む）

原因

PR #2417 で追加された resources/js/manage/page/index.js の isRowVisible() が、行の表示可否を data-parent-id（pages.parent_id カラム）を1段ずつ遡る方式で判定しています。

function isRowVisible(row, rowMap) {
    let parentId = row.dataset.parentId;
    while (parentId) {
        const parentRow = rowMap.get(parentId);
        if (!parentRow || parentRow.dataset.treeExpanded !== '1') {
            return false;   // ← 親行が見つからないだけで非表示になる
        }
        parentId = parentRow.dataset.parentId;
    }
    return true;
}

pages.parent_id は既に存在しないページIDを指している場合があります（根本原因は別Issueで報告）。その場合 rowMap.get(parentId) が undefined となり、!parentRow で即 false（非表示）になります。

一方、同じファイルの applyPreloadStyle() は Nested Set（_lft/_rgt）由来の data-ancestor-ids を使っており、こちらは正しく動作します。同一ファイル内で祖先情報のデータソースが2種類混在している点が問題の本質です。

補足：折り畳み状態とは無関係に常時発生します

init() は保存済みの折り畳み状態の有無にかかわらず updateVisibility() を無条件に実行するため、以下いずれの条件でも再現します。

• sessionStorage をクリアしても再現
• 別ブラウザ・シークレットウィンドウでも再現
• 全ユーザー・全端末で恒常的に発生

影響範囲

• ページ一覧に該当ページが出ないため、編集・権限設定・ページ移動などの操作導線が完全に断たれます
• ページ権限一覧など他の画面は影響を受けません
• 該当ページのフロント表示（一般利用者から見たサイト）には影響しません

実行環境

OS: 影響なし（クライアントサイドJSの問題のため全環境で再現）
ブラウザ: 影響なし
Connect-CMS: v1.43.0 以降（v1.44.0 で確認）

再現手順

1. 階層構造を持つページを用意する（親ページA > 子ページB）
2. ページ管理から親ページAを削除する（子ページBは削除されず残り、parent_id が存在しないIDを指した状態になる）
3. ページ一覧（/manage/page）を開く
4. 子ページBが一覧に表示されない
5. ページ権限一覧（/manage/page/roleList）を開くと、子ページBは表示される

修正方針

isRowVisible() の判定を、applyPreloadStyle() と同じ data-ancestor-ids ベースに統一します。data-ancestor-ids は page.blade.php で既に出力済みのため、サーバ側の変更は不要です。

function isRowVisible(row, rowMap) {
    const ancestorIds = (row.dataset.ancestorIds || '').split(' ').filter(Boolean);

    // 祖先が一覧内に無い場合は隠さず表示する（表示漏れを防ぐ）。
    return ancestorIds.every(function (ancestorId) {
        const ancestorRow = rowMap.get(ancestorId);
        return !ancestorRow || ancestorRow.dataset.treeExpanded === '1';
    });
}

祖先行が見つからない場合は「表示する」側（fail-safe）に倒します。本Issueは「祖先が引けないと消える」ことそのものが問題のため、同種の不整合が別経路で起きても表示漏れにならないようにする意図です。

副次的に、parent_id に循環参照があった場合の while 無限ループリスクも構造的に解消されます。

残課題（本Issueのスコープ外）

page.blade.php の $page_children = $pages->groupBy('parent_id') による has_children 判定も同じく parent_id ベースです。ある親の子が全員不整合な parent_id を持つ場合、その親に折り畳みボタンが表示されません。表示消失そのものは上記修正で解消するため別途対応とします。

[masaton0216]

修正リリースまでの運用回避策

コード修正を待たずに実施できる回避策を整理します。回避策2は parent_id の恒久的な修復にもなります。

回避策1：ブラウザのコンソールで行を再表示する（即効性・最も現実的）

ページ一覧を開いた状態で、ブラウザの開発者ツールのコンソールで以下を実行します。

document.querySelectorAll('[data-page-manage-tree] tbody tr[data-page-id]').forEach(r => r.hidden = false);

行は hidden 属性で隠されているだけで DOM には存在するため、これで全ページが表示され、編集・ページ移動・並び替えの各ボタンもそのまま正常に動作します。

注意点

• ページを再読み込みするたびに再実行が必要です
• 実行後に折り畳みボタンを押すと updateVisibility() が走って再び非表示になります

回避策2：一度「ページ移動」を実行して parent_id を正す（恒久対応）

PageManage::movePage() は削除処理と異なり Eloquent 経由（saveAsRoot() / appendToNode() / insertBeforeNode()）で実装されているため、NodeTrait が parent_id を正しく書き直します。

手順

1. 回避策1で対象行を表示させる
2. 対象ページのページ移動アイコンから移動を実行する
3. parent_id が実体に合った値へ更新され、以後は回避策1なしで通常表示される

注意点

• 現在地と同じ場所へ「最上位へ移動」した場合、saveAsRoot() はルート階層の末尾に追加するため表示順が変わります。並び順は移動後に上下移動で調整してください
• permanent_link（ページURL）は movePage() で変更されないため、移動してもページのURLは変わりません
• 子を持つページを移動すれば配下は Nested Set ごと移動するため、子側の parent_id も併せて正されます

回避策3：ページ権限一覧から編集画面へ入る

ページ権限一覧（/manage/page/roleList）には各ページの編集アイコン（role_list.blade.php → /manage/page/edit/{id}）と権限設定リンクがあります。ページ一覧に表示されないページでも、編集と権限設定にはここから到達できます。

限界: ページ移動・並び替えのUIはページ一覧側にしかないため、この経路では階層操作はできません。

回避策4：URL直打ち

ページIDがわかれば /manage/page/edit/{page_id} へ直接アクセスできます。ページIDはページ権限一覧のHTML（tr の id 属性）から取得できます。回避策3と同様、編集系の操作のみ可能です。

使い分け

状況	回避策
すぐに表示・操作したい	回避策1
恒久的に解消したい	回避策1 → 回避策2
編集・権限設定だけできればよい	回避策3 / 回避策4

開発者ツールを使えない利用者への案内としては回避策3が現実的です。