【Laravel】emailをnull許容としたい

Question

Laravel初めて1ヶ月経過しておりません。教えていただきたいです。

やりたいこと

Laravelでアプリケーションを作成しています。会社のemailを今まで必須項目だったのですが、それを登録の際に任意項目をしたいです。

<?php

namespace App\Http\Requests;

use Illuminate\Foundation\Http\FormRequest;
use Illuminate\Http\Exceptions\HttpResponseException;
use Illuminate\Contracts\Validation\Validator;

class CreateCompanyRequest extends FormRequest
{
  /**
   * Determine if the user is authorized to make this request.
   *
   * @return bool
   */
  public function authorize()
  {
    return true;
  }

  /**
   * Get the validation rules that apply to the request.
   *
   * @return array
   */
  public function rules()
  {
    return [
      'code' => 'string|unique:facilities|max:255',
      'name' => 'string|max:80',
      'name_kana' => 'string|max:80',
      'email' => ['string', 'unique:facilities', 'email:rfc,dns', 'max:250'],   //ここをnull許容としたい
      'tel' => 'string|max:15',
      'zip_code' => 'string|max:10',
      'address_1' => 'string|max:10',
      'address_2' => 'string|max:80',
      'address_3' => 'string|max:80',

    ];
  }

  protected function failedValidation(Validator $validator)
  {
    throw new HttpResponseException(response()->json([
      'status' => false,
      'messages' => $validator->errors()->all()
    ], 422));
  }
}

やってみたこと

'email' => ['string', 'unique:facilities', 'email:rfc,dns', 'max:250'],
このコードを削除してみましたが、server errorとの表示となってしまいます。

laravel,phpに詳しい方に教えていただきたいです。

必要な情報などございましたら、ぜひ教えてください。

Comments

[m.ts10806]

予測できてるならやってみれば良いのでは。やってみた結果どうなったかで質問するかどうするかだと思うのですが。

[masaosan18]

ありがとうございます！！！

[m.ts10806]

「うまくいかない」は何も情報がない表現です（質問するときのヒント参照）

[masaosan18]

はい！

[K_3578]

エラーになってるならエラー文読む。分からないならコピーして調べる。
それでも分からなかったらここでどう調べてどこが分からなかったか聞いてみよう

Answer 1

バリデーションでいくら空を許容してもDBで許容してないと登録はできませんので、DB側の調整は必至です。
ただ、元々null許容してなかったのであれば、リレーションなど関係するかもしれませんし、元のデータとの整合性もあります。
「やり方」を主とするのではなく影響範囲含めた「要件」をまとめた上で対応されたほうがトラブルなく進むと思います。

Comments

[masaosan18]

migrationファイルを更新してみます！

[m.ts10806]

それ、既存のデータに影響ないですか？まだ全然「やり方」だけにしか目がいってないように見えます。
サーバーエラーとなったのでしたらエラーログを確認するのがプログラミングにおける定石です。

[m.ts10806]

起きてる現象を正確に捉えて適切な対応しないとその場しのぎになって元々動いていたものが動かなくなることだってあります。なので影響範囲調査は必須。
バックアップとってすぐ戻せるような体制も必要です。
あれこれやってるうちに戻せなくなって総動員しなくて済むように配慮してください。

[masaosan18]

ありがとうございます！

[m.ts10806]

結局どのように解決とされたのでしょうか