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

[akagane99]

概要

• [サイト内検索] 隠しページ（ページ管理＞メニュー非表示のページ）は検索に含めない設定を追加したい connect-cms-ideas#183
  • こちらの実装です。
• Searchsモデルの narrow_down_label は既に削除済みカラムだったため、$fillable から消しました。

修正後画面

サイト内検索＞設定変更（新規作成）

レビュー完了希望日

急ぎません

関連Pull requests/Issues

なし

参考

なし

DB変更の有無

あり

チェックリスト

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

[masaton0216]

ご対応ありがとうございます。
下記、ご確認お願いいたします。

• パフォーマンス観点で２つ
• enum化の検討依頼１つ
• スペース続きの軽微な修正１つ

[masaton0216]

下記部分はSQLにした方が走査処理としては効率的かと思いましたが、いかがでしょう？

      $pages = $pages->filter(function ($page) {                                                                                                                 
          return $page->base_display_flag == 1;                                                                                                                  
      });

全体を示すと下記のようなコードを想定しています。

      // ページの選択「ページ管理のメニュー表示条件に従う」
      if ($searchs_frame->page_select == 1) {                                                                                                                    
          // 表示ページのみをDBレベルで絞り込む
          $pages = Page::where('base_display_flag', 1)->get();                                                                                                   
                  
          // フレームの選択「選択したものだけ表示する」                                                                                                          
          if ($searchs_frame->frame_select == 1) {
              // 選択したフレームに紐づくページ を追加取得してマージ                                                                                             
              $frame_page_ids = Frame::whereIn('frames.id', explode(',', $searchs_frame->target_frame_ids))                                                      
                  ->pluck('page_id')->toArray();                                                                                                                 
              $pages_frame = Page::whereIn('pages.id', $frame_page_ids)->get();                                                                                  
                                                                                                                                                                 
              $pages = $pages->merge($pages_frame)->unique('id');
          }                                                                                                                                                      
      } else {    
          // 全ページを検索対象とする
          $pages = Page::get();
      } 

[masaton0216]

2回のDBクエリを1回に統合できそうです。

  $pages_frame = Page::whereIn('id', function ($query) use ($searchs_frame) {                                                                                    
      $query->select('page_id')                                                                                                                                  
          ->from('frames')
          ->whereIn('id', explode(',', $searchs_frame->target_frame_ids));                                                                                       
  })->get();  

[masaton0216]

マジックナンバーがコントローラ／viewに散在してしまっているので、enum定数化をお願いすること可能でしょうか？

（一案）
SearchsPageSelect.php
～略～
const ALL_PAGES = 0;           // 全て表示する                                                                                                                 
const MENU_VISIBLE_ONLY = 1;   // ページ管理のメニュー表示条件に従う

[masaton0216]

イコール演算子の右側にスペースが２つ続いています。

$frame_page_ids =  Frame::whereIn(...)