Laravel 5.8で配列のリクエストのバリデーションができません

Question

前提・実現したいこと

Laravel5.8で勤怠システムを作成しています。
テーブルの形で、出勤時間、退勤時間、合計時間、作業内容、プロジェクトコードに入力欄があります。

formは下記のようになっています。(inputが入力欄の想定)

出勤時間	退勤時間	合計時間	作業内容	プロジェクトコード
input	input	input	input	input
input	input	input	input	input
input	input	input	input	input

行は複数行同時登録可能な仕様です。
それを、登録のためPOSTした際のバリデーションで
1行目は必須、2行目は出勤時間があれば、退勤時間必須...のようにバリデーションをかけたいです。

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

POSTした際に、何も入力していない場合、nullの配列が送られ、思うようにバリデーションがかかりません。

該当のソースコード

ここでは10個決め打ちしていますが実際は何個になるか不明です。

Blade

<table class="table white-space-nowrap">
    <thead>
        <tr>
            <th scope="col">出勤時間</th>
            <th scope="col">退勤時間</th>
            <th scope="col">合計</th>
            <th scope="col">作業内容</th>
            <th scope="col">pj番号</th>
        </tr>
    </thead>
    <tbody>
        @for ($i = 0; $i < 10; $i++) <tr>
            <td>
                <input type="text" placeholder="00:00" name="start_time[]">
            </td>
            <td>
                <input type="text" placeholder="00:00" name="end_time[]">
            </td>

            <td class="input-size-s">
                <input type="text" name="sum_time[]" disabled="disabled" placeholder="合計">
            </td>
            <td>
                <input type="text" placeholder="作業内容" name="work_content[]">
            </td>
            <td>
                <input type="text" disabled="disabled" name="pj_num[]" placeholder="pj番号">
            </td>
            </tr>
            @endfor
    </tbody>
</table>

FormRequest

 public function rules()
    {
        return [
            'start_time' => 'required|array|min:1|',
            'start_time.*' => 'nullable|date_format:H:i|',
            'end_time' => 'required|array|min:1',
            'end_time.*' => 'required_with:start_time.*|date_format:H:i|after:start_time.*',
            'sum_time' => 'required|array|min:1|',
            'sum_time.*' => 'required_with:start_time.*|date_format:H:i|',
        ];
    }

試したこと

バリデーションがかかる前に、array_filterでnullを削除しようとしましたが、全部からだとエラーが出てうまくできていません。

大変申し訳ありませんが、お力を貸していただけますと助かります。
初めての質問で不備等ありましたら追記しますので、宜しくお願い致します。

Answer 1

start_timeとend_timeをについて、

・1行目が必須
・2行目以降は、start_timeがあれば、end_timeも

なら下記でいけるはず・・。

'start_time.0' => 'required',
'end_time.0'   => 'required',
'start_time.*' => 'nullable|date_format:H:i',
'end_time.*'   => 'required_with:start_time.*|nullable|date_format:H:i|after:start_time.*',

提示のbladeを見る限り、inputのnameが配列形式（***[]）なので、1行目の必須は「0」指定で書いてしまいます。

なお、「0」もそれ以外「.」も共通でチェックしたいもの（時間の形式など）は、
「.」の方に書いておけば「0」もチェックされるので、「0」はrequiredのみで問題ないとおもいます。（※）

（※）動作確認をlaravel6.0で行ったので、もし挙動が違った場合はすみません^^;

---

それと、質問の本筋とは関係ありませんが、
nullを許容する可能性のある項目には、全てnullableも並記しておいた方が良いですよ。

質問に提示されたコードより、下記のルールを例にあげますと、

'sum_time.*' => 'required_with:start_time.*|date_format:H:i|',

sun_time[1]を、空のまま送信したとします。

start_time[1]がなければ、required_withのバリデーションチェックは通りますが、nullableが設定されていない為、次のdate_formatが空の送信値nullに対して実行され、バリデーションエラーになってしまいます。

以上、ご参考までに・・。

Comments

[natsume2233]

できました！有難うございました
とても勉強になりました。
補足事項つけて頂き有難うございます、まさにそこにも引っかかっていたようです
助かりました。有難うございます

Answer 2

ある項目に入力あった場合は他の項目も必要
みたいなのがしたいならバリデーションのrequired_withを駆使したらいけそうな気がします

バリデーション 5.8 Laravel

Comments

[natsume2233]

回答ありがとうございます
既に使用していますが、POSTしたデータがnullでも配列として存在しているため機能していません...

回答有難うございました

Answer 3

出勤時間があれば、退勤時間必須...

以下のサイトが参考になるかと思います。
※Laravelのバージョンは違いますが、大丈夫かと…

https://www.larajapan.com/2016/10/23/バリデーション-8-配列をバリデーションする/

Comments

[natsume2233]

回答ありがとうございます
こちらを参考に実装したものなのですが、うまくいかず...
また考えてみます

回答有難うございました

Answer 4

自動バリデートではなく、カスタムバリデート組む形になると思います。
カスタムというか、行個々にバリデートするかしないか決める形。
つまり、「その行でひとつでも入力があれば」を条件にしてバリデート通すイメージですね。