laravelのバリデーションでエラー

Question

前提・実現したいこと

PhP laravelにて画像を投稿できる機能を実装しているのですが、
そのフォームに対して添付写真のエラーが出てしまいます。

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

画像投稿時にエラーが発生。

該当のソースコード

  public function upload(LostRequest $request) {

        $lost = new Lost;
        $lost->user_id = $request->user()->id;
        $lost->pref    = $request->pref;
        $lost->seed    = $request->seed;
        $lost->image   = $request->image->getClientOriginalName(); // アップロードされたファイル名のままテーブルに保存
        $lost->body    = $request->body;
        $lost->save();

        $this->validate($request, [
            'image' => [
                // アップロードされたファイルであること
                'file',
                // 画像ファイルであること
                'image',
                // MIMEタイプを指定
                'mimes:jpeg,png',
            ]
        ]);

        if (empty($request->image) == false) {
            if ($request->file('image')->isValid([])) {
                $file_name = $request->file('image')->getClientOriginalName(); // アップロードされたファイル名の取得
                $path = $request->image->storeAs('public',$file_name);
                return view('home')->with('filename', basename($path));
            } else {
                return redirect()
                    ->back()
                    ->withInput()
                    ->withErrors();
            }
        }
        return view('home');
    }

    <?php

namespace App\Http\Requests;

use Illuminate\Foundation\Http\FormRequest;

class LostRequest 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 [
            'seed'  => 'required',
            'pref'  => 'required',
            'image' => 'nullable'|'image',
            'body'  =>  'required_without:image',
        ];
    }
}

試したこと

getClientOriginalName()on nulがエラーの原因のため、
一度削除してみたがその場合、テーブルにランダムな文字列で画像が保存されてしまうようになった。
getClientOriginalNameがnullでも通る方法を検索したが、解決できず。

Comments

[退会済みユーザー]

form部分に問題がある可能性が高いので、「画像アップロード機能実装時に　enctype="multipart/form-data"　は記載済みになります。 」と伝えるよりもエビデンスを示していただきたい。

[rururu3]

画像なしのときって書いてたのでエビデンスはありますよ

[rururu3]

っておもったら質問から編集で消えてた…

Answer 1

<?php
  $hoge = null;
  $hoge->getClientOriginalName();

これで同じように
PHP Fatal error: Uncaught Error: Call to a member function getClientOriginalName() on null in…というエラーが出ますので、nullに対して関数呼び出ししてるのが原因です(存在しない関数呼び出しをしてるのが原因)。
値が入っているか確認してみたらいかがでしょう。

Comments

[k3114]

ご回答ありがとうございます。
言葉足らずで申し訳ないです。
画像を投稿せずにバリデーションを通過させたい時に出てしまいます。
故意に画像を投稿しない場合に、getClientOriginalName();でエラーを発生させない方法はありますでしょうか？

[rururu3]

$lost->image = empty($request->image) !== false ? 'Lostテーブルのimageに画像がなかったときに入れたい値' : $request->image->getClientOriginalName();
でもしておけばよいのでは？(その後の$this->validate($request, …の処理が不要ならそこも条件分岐で飛ばす

[rururu3]

https://readouble.com/laravel/5.4/ja/requests.html
hasFile使うほうが良さそうかも

[k3114]

ありがとうございます
$lost->image = empty($request->image) !== false ? '' :$request->image->getClientOriginalName();
こちらの記述に変更してみましたところ、追加した写真の別のエラーが発生してしまいました。
コードを変更してみたりしたところ、今回のimageではなく他の問題のように考えたのですが、検討はつきますでしょうか？
質問が派生してしまい大変申し訳ありませんが、ご教授いただければ幸いです

[退会済みユーザー]

form に enctype=multipart-formdata が入っているかを確認しては？

[k3114]

ありがとうございます、画像アップロード機能実装時に　enctype="multipart/form-data"　は記載済みになります。

[k3114]

申し訳ありません、写真に誤りがありました。
現在二番目に添付しているものが現在のエラーになります。
お答えいただいたのに大変失礼しました。

[rururu3]

お・・・２枚めもないしベストアンサーになったから解決でいいのかな

[k3114]

すみません！別のエラーが発生のため、再投稿させていただいております！