【laravel】画像の更新が反映されない

Question

やりたいこと

・画像保存のアプリを作成したい
・画像の更新（/update/{id})をすると新しい画像をアップロードする

現状

・エラーは出てないけど、更新されない状態

###コード

Controller.php

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use \App\Memo;
use \App\User;
use Illuminate\Support\Facades\Storage;

    public function store(Request $request)
    {
        $data = $request->all();
        //dd($data);
        $image = $request->file('image');
        //dd($image);
        if($request->hasFile('image')){
            $path = \Storage::put('/public', $image);
            $path = explode('/', $path);
        }else{
            $path = null;
        }
        $memo_id = Memo::insertGetId(
            ['title' => $data['title'],
            'content' => $data['content'],
            'user_id' => $data['user_id'],  
            'image' => $path[1],
            'url' => $data['url'],
            'status' => 1]);

        // リダイレクト処理
        return redirect()->route('home');
    }

    public function update(Request $request , $id ,Memo $memos){
        $inputs = $request->all();
        
        if($request->hasFile('image')) {
            Storage::delete('public' . $memos->image); //元の画像を削除
            $path = $request->file('image')->store('public');
            $memos->image = basename($path);
            $memos->save;
        }

        Memo::where('id',$id)->update(['title' => $inputs['title'],
                                        'content' => $inputs['content'],
                                       'url' => $inputs['url']
                                    ]);

        return redirect()->route('home');
    }

web.php

<?php

  Route::group(['middleware' => ['auth', 'can:admin-higher']], function () {
    Route::get('/newpost', 'AccountController@newpost')->name('newpost');
    Route::get('/create', 'AccountController@create')->name('create');
    Route::post('/store', 'AccountController@store')->name('store');
    Route::get('/edit/{id}', 'AccountController@edit')->name('edit');
    Route::post('/update/{id}', 'AccountController@update')->name('update');
    Route::post('/delete/{id}', 'AccountController@delete')->name('delete');
    Route::get('/userlist', 'AccountController@userlist')->name('userlist');
    Route::get('/userdetail/{id}', 'AccountController@userdetail')->name('userdetail');

  });

edit.blade.php

@extends('layouts.app')

@section('content')
<div class="row justify-content-center ml-0 mr-0 h-100">
    <div class="card w-100">
        <div class="card-header">メモ編集</div>

        <!-- メモの内容-->
            <div class="card-body">
            <form method='POST' action="{{route('update',['id' => $memo['id']] ) }}" enctype="multipart/form-data">
                @csrf
                <input type='hidden' name='user_id' value="{{ $user['id'] }}">

                <!-- タイトルの表示 -->
                    <div class="form-group">
                    <input type=”text” name='title' class="form-control"rows="10">{{ $memo['title'] }}</input>
                    </div>

                <div class="form-group">
                     <textarea name='content' class="form-control"rows="10">
                         {{ $memo['content'] }}
                    </textarea>

                    
                    <div class="form-group">
                    <label for="image">画像のupload</label>
                      
                    <input type="file" class="form-control-file" name='image' id="image">
                </div>
                    
                        <!--URLをとってくる-->
                    <div class="mb-3">
                        <label for="disabledTextInput" class="form-label">URLの入力してください</label>  
                        <input type="text" id="disabledTextInput" class="form-control" name='url' placeholder="URL">
                    </div>

                </div>
                <div class="form-group">
                </div>
                <button type='submit' class="btn btn-primary btn-lg">更新する</button>
            </form>
            
        </div>
    </div>
</div>
@endsection

追加コード
Memo.php

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Memo extends Model
{
    /*
    public function admins(){
        return $this->belongsTo('App\Admin');
    }
    */

    protected $fillable = [
        'title',
        'contnet',
        'url',
        'image'
    ];
}

ddで$memoを確認した結果

"image" => "UdvdOkrYjJKGMDPBVAQcVQNJ99VHHpSWl5xV4KIa.jpg"

public/storageの中身を確認⇨"UdvdOkrYjJKGMDPBVAQcVQNJ99VHHpSWl5xV4KIa.jpg"が存在している

DBの中身は更新前のimageになっている

Comments

[skys215]

Memoのfillableに更新する各コラムを書きましたか？
複数代入ー＞　https://readouble.com/laravel/8.x/ja/eloquent.html

[yoshioka01]

記入してみましたがうまくいきませんでした

Answer 1

//$memos->save;じゃなくて
$memos->save(); //です。

save()は関数です。

Comments

[yoshioka01]

見落としていました
ありがとうございます