質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.62%

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

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,080

rude_rockers

score 32

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']]);
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • kazto

    2019/01/08 17:22

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

    キャンセル

  • rude_rockers

    2019/01/08 17:34

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

    キャンセル

回答 1

checkベストアンサー

+1

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/01/09 05:42

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

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

    echo $errors;

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


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


    と出ます。

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

    キャンセル

  • 2019/01/09 05:46

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

    キャンセル

  • 2019/01/09 05:49

    大変申し訳ありません。

    enctype="multipart/form-data"


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

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

    キャンセル

  • 2019/01/09 15:58

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.62%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る