[サイト内検索] ページの選択「ページ管理のメニュー表示条件に従う」がうまく機能していない不具合を修正

[akagane99]

概要

• [サイト内検索] 隠しページ（ページ管理＞メニュー非表示のページ）は検索に含めない設定を追加 #2386

前述の不具合修正です。

SearchsPlugin::fetchSearchablePageIds() 内の以下で、親子ページを加味してページ表示　対応が出来ていると思い込みましたが、違っていました。

            // 親子ページを加味してページ表示できるか
            if (!$page->isVisibleAncestorsAndSelf($page_tree)) {
                continue;
            }

このメソッドは権限に基づいて見れる・見れないを判定するメソッドで、メニューの表示・非表示 page->base_display_flag は判定していなかったため、親子のpage->base_display_flagを判定する処理を追加しました。

レビュー完了希望日

急ぎません

関連Pull requests/Issues

• [サイト内検索] 隠しページ（ページ管理＞メニュー非表示のページ）は検索に含めない設定を追加したい connect-cms-ideas#183

参考

なし

DB変更の有無

なし

チェックリスト

• プルリクエストにわかりやすいタイトルとラベルを付けました。https://github.com/opensource-workshop/connect-cms/wiki/Pull-requests-Rule

[masaton0216]

@akagane99
永らく放置してしまい誠に申し訳ございません。
今週中に確認させて頂きます🙇

[masaton0216]

ご対応ありがとうございます。
3点コメント入れさせていただきました。

[akagane99]

@masaton0216 さん
レビュー指摘事項をすべて修正しました。
お時間あるときにレビュー及び、よければマージをお願いします<(_ _)>