LaravelでCRUDの更新機能。バリデーションが成功しません。

Question

LaravelでCRUDの練習中です。

画像をpublic配下のimagesフォルダに保存しつつ、ファイルネームをデータベースに保存する感じで作りました。

新規投稿の方はうまくいったのに、更新がうまくいきません。

「dd();」で見てみるとバリデーションが通っていません。

新規投稿と同じ画像を使ってもダメなのですが、原因がわかりません。

よろしくお願いします。

環境
さくらレンタルサーバー
laravel5.7
apache
php7.2
mysql5.7

成功するstoreメソッド

class PostsController extends Controller
{
...
public function store(Request $request)
  {
      $request->validate([
          'title' => 'required|max:50',
          'body' => 'required|max:2000',
          'image'=>'image|mimes:jpg,jpeg,png|max:5000',
      ]);

      $file = null;
      $fileName = '';

      if($request->file('image') != null){

      $file=$request->file('image');
      $fileName=str_random(20).'.'.$file->getClientOriginalExtension();
      Image::make($file)->save(public_path('images/'.$fileName));
      }

      $post=new Post;
      $post->title=$request->title;
      $post->body=$request->body;
      $post->image=$fileName;
      unset($post['_token']);
      $post->save();

      return redirect()->route('top');
  }
...

失敗するupdateメソッド

  public function update($post_id, Request $request)
  {
    // dd($request);
    // バリデート
    $request->validate([
        'title' => 'required|max:50',
        'body' => 'required|max:2000',
        'image'=>'image|mimes:jpg,jpeg,png|max:5000',
    ]);

    dd($request);

    $post = Post::findOrFail($post_id);

    $oldImage = $post->image;

    // dd($oldImage);
    // \Debugbar::info($oldImage);

    $file = null;
    $fileName = '';

    if($request->file('image') != null){

    $file=$request->file('image');
    $fileName=str_random(20).'.'.$file->getClientOriginalExtension();
    Image::make($file)->save(public_path('images/'.$fileName));
    }

    $post->title=$request->title;
    $post->body=$request->body;
    $post->image=$fileName;
    unset($post['_token']);
    $post->save();

    \File::delete(public_path('images/'.$oldImage));

    return redirect()->route('posts.show', ['post' => $post]);
  }

なぜ同じバリデーションなのに通過できないのでしょうか？

追記
route/web.php

<?php

/*
|--------------------------------------------------------------------------
| Web Routes
|--------------------------------------------------------------------------
|
| Here is where you can register web routes for your application. These
| routes are loaded by the RouteServiceProvider within a group which
| contains the "web" middleware group. Now create something great!
|
*/
Route::get('/', 'PostsController@index')->name('top');
Route::resource('comments', 'CommentsController', ['only' => ['store']]);
Route::resource('posts', 'PostsController', ['only' => ['create', 'store', 'show', 'edit', 'update', 'destroy']]);

Comments

[kazto]

route/web.php の内容はどうなっていますか？追記をお願いします。

[rude_rockers]

ありがとうございます。
追記致しました。

Answer 1

「dd();」で見てみるとバリデーションが通っていません。

Laravel5.5 バリデーションの「バリデーションエラー表示」を見て、具体的になぜvalidationが通っていないのか確認するとよいと思います。

Comments

[rude_rockers]

ありがとうございます。
エラーの見方がわかりました。

titleとbodyのバリデーションは通るのですが、やはり、バリデーションにひっかからない画像データをアップロードしても、エラーが出ます。

echo $errors;

で画面上部にエラーをだしてみました。
タイトルもボディも画像も問題のないモノをアップロードしても、

「{"image":["The image must be an image.","The image must be a file of type: jpg, jpeg, png."]}」

と出ます。

ちゃんと画像であり、5000以下であり、jpgなのですが。。。

[rude_rockers]

バリデーションをとったら、通過できましたが、画像ファイルが保存されていませんでした。
ということはinputがおかしいかもですね。

[rude_rockers]

大変申し訳ありません。

enctype="multipart/form-data"

が抜けておりました！！！！！！

これもひとえにエラーの見方がわかったから気づきました。
ありがとうございます。

[takg]

無事解決できたようで、よかったです^^