laravelのvalidationの判定を分けたい。

Question

前提・実現したいこと

今集計表をつくってまして、それには図の通り
「日付指定検索」と「範囲指定検索」がありまして、
それぞれのデータ名はdateとstart,endです。

それぞれコントローラーの関数で判定してます。

発生している問題・エラーメッセージ

図の通り日付が空欄のまま範囲は指定されてる状態で
日付指定の検索をしてしまうとno messageのエラーが出てしまいます。
その逆のパターンも起こります。

この場合はどうvalidationを書けばうまく動くでしょうか。

Symfony \ Component \ HttpKernel \ Exception \ MethodNotAllowedHttpException
No message

該当のソースコード

    // 日付指定検索の関数
    public function date_data(Request $request)
    {
        $this->validate($request, [
            'date' => 'required',
        ]);
      ～略～
    }

    // 範囲指定検索の関数
    public function between_data(Request $request)
    {
        $this->validate($request, [
            'start' => 'bail|required',
            'end' => 'required',
        ]);
　　　～略～
    }

Answer 1

MethodNotAllowedHttpExceptionは、バリデーションエラーがきっかけで発生しているものですが、「バリデーションの設定内容」のせいで発生しているわけではありません。。。

集計ボタンが設置してある画面のルート設定が、postのみになっていませんか？

バリデーションエラーがあった時、送信元URLに自動リダイレクトされますが、これはGETでのアクセスになります。
なので、バリデーションを使うなら、集計ボタンが設置してある画面のルートにはgetの設定も必要になるのです。

コントローラー側も、getでアクセスされることを考慮して、
・バリデーションチェック
・集計処理を
この2点は、post時のみに行われるよう調整した方が良さそうです。

---

以下は「余計なお世話」なお話です。
「今後何かの参考になれば」程度に、、、なんなら、読み飛ばしても大丈夫です。

全く同じ見た目のボタンが二つ並んでいれば、誰でも押し間違える可能性が高いので、
ユーザーに優しくないな、と思いました。

私が、もしこういう画面を作るならば。

入力欄もボタンも、紛らわしいものは設置せず、最少数で賄います。

<h2>集計</h2>
<form action="xxxxxx">
日付<input type="text" name="start">～<input type="text" name="end">
<button type="submit">集計</button>
</form>

コントローラも1つにまとめてしまいます。

バリデーションチェックは、必須はstartのみに。

ついでに日付の正当性と、日付範囲で後ろになるendには「startより後ろの日」を付けると、後々の集計処理が楽かなと思って追加しています。

「日付指定」と「範囲指定」どちらの集計を行うかは、endの入力有無で、自動的に判断する方式に。。。

    public function xxxxxx(Request $request)
    {
        //postなら、入力値チェックと集計の処理
        if ($request->isMethod('post')) {
            $this->validate($request, [
                'start' => 'bail|required|date',
                'end'   => 'bail|nullable|date|after:start'
            ]);

            $start = $request->get('start');
            $end = $request->get('end');

            if($end){
                //endがあるなら、範囲指定集計処理を

            }else{
                //なければ、日付指定の集計処理を

            }
        }else{
            //getで何か処理したければここに
        }

    }

※コードはここで直接書いたものなので、動作検証はしておりません。ご了承を・・