Laravel required_if について

Question

前提・実現したいこと

私は今、laravelでバリデーションて引っかかった事があります。
まず、カードの貸し借り+使用記録データのテーブルがあります。

テーブル中身はこんな感じ
table :card_using_records

名前	データ型
id	int(10)
card_id	int(10)
receipt_path	varchar(255)
is_using	tinyint(1)　laravelのboolean type
//他は関係ないのでSKIP

で、私の欲しいバリデーションはis_usingは1( true )の時にreceipt_pathは必須項目、これは
https://readouble.com/laravel/5.7/ja/validation.html#rule-url
を参考に

receipt_path => ['required_if:is_using,1', 'nullable', 'string']

これで問題ないです。

逆に、is_usingは0（ false ）の時に receipt_path は空でなければならない。
これのやり方がよくわかりません。

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

該当のソースコード

//App\Http\Requests\StoreCardUsingRecordRequest

    public function rules()
    {
        return [
            'card_id'           => ['required', 'exists:' . (new Card)->getTable() . ',id'],
            'is_using'          => ['required', 'boolean'],
            'receipt_path'      => ['required_if:is_using,1', 'nullable', 'string'],
        ];
    }

試したこと

'receipt_path'      => ['required_if:is_using,1', 'nullable', 'string',
                        Rule::requiredIf(function () {
                                return ( $this->get('is_using') === '0' && empty( $this->get('receipt_path') ) );
                            })],

↑うまく行きません、is_usingは1( true )の時にreceipt_pathは必須項目しかチェックのみチェックしています。

補足情報（FW/ツールのバージョンなど）

Laravel 5.7
PHP 7.2

Answer 1

ルールの順番は変えてみましたか？

'receipt_path'      => ['nullable', 'required_if:is_using,1', 'string'],

あとはis_usingの場合はreceipt_path項目をPOSTさせないようにdisabledかけたり、バリデーションをかける前に値の処理（is_usingが0の場合はreceipt_path項目の値を空にする 等）行ったりもできるかと思います

Comments

[KenTse]

ご返答ありがとうございます。

ルールの順番色々変えて見たけど、変わらないですね。
POSTさせないのはveiwのjavascriptの所やっていますが、クライアント側だからね、
サーバー側もやる必要があります。

で、一々バリデーションをかける前に値を処理するのはあんまりやりたくないですね、
ここ一つのフォームでなら問題ないですけど、後にCSVアップロードの大量データ保存のバリエーションも同じことをやるのでもう一回ループでやる分、遅くなります。
やっぱりバリデーションのルールでやりたいです。

[sola-msr]

んーそうですね、、、is_usingの値はポストされてるか、中身の値の確認されましたか？
（ポストされてないとか、0じゃないとか）

あとはrequired_withを使うかでしょうか・・・

[KenTse]

今、カスタマイズのバリデーションルールでの方向で考えています。
https://qiita.com/komatzz/items/c64c83a004b7f2c63ce6

[sola-msr]

Laravel5.3の情報みたいなので、5.7での方法を張っておきますhttps://readouble.com/laravel/5.7/ja/validation.html

[sola-msr]

カスタムバリデーションルールの項を参考にしてください（php artisan make:rule Uppercase）

[KenTse]

ご協力ありがとうございました、解決しました。

Answer 2

最終的には
https://qiita.com/komatzz/items/c64c83a004b7f2c63ce6
と
Illuminate\Validation\Concerns\ValidatesAttributes
のvalidateRequiredIfのfunctionを参考に
カスタマイズのバリデーションルール "null_if" を作りますた。

//App\Validator\CustomValidator


    /**
     * 他のフィールドが値のどれかと一致している場合、このフィールドが空であることをバリデートします。
     * eg. nullable_if:is_using,0
     * @param  string  $attribute
     * @param  mixed   $value
     * @param  mixed   $parameters
     * @return bool
     */
    public function validateNullIf($attribute, $value, $parameters)
    {
        $this->requireParameterCount(2, $parameters, 'null_if');

        $other = Arr::get($this->data, $parameters[0]);

        $values = array_slice($parameters, 1);

        if (is_bool($other)) {
            $values = $this->convertValuesToBoolean($values);
        }

        if (in_array($other, $values) && !empty($value) ) {
            return false;
        }

        return true;
    }

//App\Http\Requests\StoreCardUsingRecordRequest

    public function rules()
    {
        return [
            'card_id'           => ['required', 'exists:' . (new Card)->getTable() . ',id'],
            'is_using'          => ['required', 'boolean'],
            'receipt_path'      => [ 'required_if:is_using,1', 'null_if:is_using,0', 'numeric', 'nullable'],
        ];
    }