Почему не работает update данных в базе laravel 5.4?

Question

[bpGusar]

контроллер загрузки страницы редактирования

public function edit($id)
    {
        $post = Posts::find($id);
        $category = DB::table('post_category')->pluck('cat_name', 'id_cat');

        return view('admin.posts.editpost', compact('post','category'));
    }

котроллер update

public function update(Request $request, $id)
    {
        $this->validate($request, [
            'title' => 'required|max:255',
            'short_story' => 'required|unique:posts',
            'full_story' => 'required',
            'id_cat' => 'required',
            'poster' => 'image',
        ]);
        $post = Posts::find($id);
        $post->title=$request->title;
        $post->short_story=$request->short_story;
        $post->full_story=$request->full_story;
        $post->id_cat=$request->id_cat;
        if($request->hasFile('poster')){
            $posterimg=$request->file('poster');
            $postername=time().'.'.$posterimg->getClientOriginalExtension();
            $posterlocation=public_path('img/' . $postername);
            Image::make($posterimg)->save($posterlocation);
            $oldPosterName=$post->image;
            $post->image=$postername;
            Storage::delete($oldPosterName);
        }
        $post->save();
        Session::flash('success', 'Новость опубликована! ');
        return view('admin.posts.msgGlobal');
    }

view

{!! Form::model($post, ['route' => ['admin.update', $post->id], 'method'=>'PUT', 'files'=>true]) !!}

            {!! Form::label('title', 'Название поста') !!}
            {!! Form::text('title', null, ['class'=>'form-control', 'required'=>'']) !!}

            {!! Form::label('short_story', 'Краткая новость, предисловие' ) !!}
            {!! Form::textarea('short_story', null, ['class'=>'form-control', 'required'=>'']) !!}

            {!! Form::label('full_story', 'Полная новость') !!}
            {!! Form::textarea('full_story', null, ['class'=>'form-control', 'required'=>'']) !!}

                    {!! Form::label('id_cat', 'Категория') !!}
                    {!! Form::select('id_cat', $category, null, ['class' => 'form-control', 'required'=>'']) !!}


                    {!! Form::label('poster', 'Постер') !!}
                    {!! Form::file('poster',['class' => 'form-control']) !!}
                    <img src="{{ asset('img/' . $post->poster)}}" class="rounded img-fluid"/>

                    {!! Form::label('created_at', 'Дата публикации') !!}
                    {!! Form::date('created_at', \Carbon\Carbon::now(), ['class' => 'form-control', 'disabled'=>'']) !!}

                {!! Form::submit('Опубликовать', ['class'=>'btn btn-primary']) !!}

    {!! Form::close() !!}

уже 2 дня бьюсь об камень головой но не пойму что не так!

Answer 1

[First Last]

Ошибку пишет?
Во-первых:

public function edit($id)
    {
        $post = Posts::find($id);
        $category = DB::table('post_category')->pluck('cat_name', 'id_cat');

        return view('admin.posts.editpost', compact('post','category'));
    }

Это что за запрос для $category? Да и почему в pluck названия ячеек из 'posts'?
Во-вторых:

public function update(Request $request, $id)
    {
        $this->validate($request, [
            'title' => 'required|max:255',
            'short_story' => 'required|unique:posts',
            'full_story' => 'required',
            'id_cat' => 'required',
            'poster' => 'image',
        ]);

У Вас обновление и стоит unique:posts
Это меня смущает, уберите и попробуйте обновить запись
$post->save();
Можно использовать update()

Comments

[bpGusar]

$category = DB::table('post_category')->pluck('cat_name', 'id_cat'); это выборка категорий из таблицы post_category, с таблицей posts она связана через поле id_cat, оно есть и в той и в той таблице

[First Last]

Павел Бондаренко: прочтите что делает pluck
Или же, Вам нужно взять все категории из таблицы:
И еще: почему такие названия ячеек?
Почему не : id | name ?

[First Last]

Павел Бондаренко: мне кажется, что Вы хотите выбрать категорию для определенного поста, но вместо этого берете все

[bpGusar]

First Last: этот запрос загружает категории в select {!! Form::select('id_cat', $category, null, ['class' => 'form-control', 'required'=>'']) !!}, и показывает какая сейчас выбрана

[First Last]

Павел Бондаренко: и все же, я не пойму: Вам нужны все категории из таблицы или категория связанная с этим постом?
И что насчет ошибок при обновление? Есть? Пробовали убрать unique?

[bpGusar]

First Last: убрал unique и всё заработало :facepalm: . смысл того запроса такой - на странице с редактированием новости есть select куда загружаются все категории из таблицы post_category, при этом в селетке уже выбрана та категория, которая уже стоит у редактируемого поста

[First Last]

Павел Бондаренко: теперь понял.

[bpGusar]

First Last: теперь не удаляет старую картинку при замене ее на новую, не посмотришь?

[First Last]

Павел Бондаренко: сделайте dd($request->file('poster')) после условия hasFile('poster') и посмотрите что там

[bpGusar]

First Last:

UploadedFile {#218 ▼
  -test: false
  -originalName: "UHSGbnr5.jpg"
  -mimeType: "image/jpeg"
  -size: 80571
  -error: 0
  #hashName: null
  path: "D:\OSP527\OSPanel\userdata\temp"
  filename: "php6570.tmp"
  basename: "php6570.tmp"
  pathname: "D:\OSP527\OSPanel\userdata\temp\php6570.tmp"
  extension: "tmp"
  realPath: "D:\OSP527\OSPanel\userdata\temp\php6570.tmp"
  aTime: 2017-08-06 19:32:49
  mTime: 2017-08-06 19:32:49
  cTime: 2017-08-06 19:32:49
  inode: 0
  size: 80571
  perms: 0100666
  owner: 0
  group: 0
  type: "file"
  writable: true
  readable: true
  executable: false
  file: true
  dir: false
  link: false
  linkTarget: "D:\OSP527\OSPanel\userdata\temp\php6570.tmp"
}

[First Last]

Павел Бондаренко: не так прочел.
Не удаляет старый, но новый сохраняет, так?
Есть ошибки?

$oldPosterName=$post->image;
$post->image=$postername;
Storage::delete($oldPosterName);

До этого удаляло?
Посмотрите $oldPosterName (dd(..))
Еще попробуйте в config/filesystem.php в 'public' => []
заменить root на вот это:
'root' => storage_path('app/public'),

[bpGusar]

First Last: в config/filesystem.php так и было

'public' => [
            'driver' => 'local',
            'root' => storage_path('app/public'),
            'url' => env('APP_URL').'/storage',
            'visibility' => 'public',
        ],

сделал так

$oldPosterName=$post->image;
$post->image=$postername;
Storage::delete($oldPosterName);

выдало
SQLSTATE[42S22]: Column not found: 1054 Unknown column 'image' in 'field list' (SQL: update `posts` set `updated_at` = 2017-08-06 19:41:39, `image` = 1502048499.jpg where `id` = 1)

это изза того что нет колонки image, у меня есть колонка Poster но если сменить image на poster то не удаляет

[First Last]

Павел Бондаренко:
Storage::delete('img/' + $oldPosterName);

[First Last]

Павел Бондаренко: и еще:
Вы при новом постере сохраняете только в fs,а в ячейке имя не обновляете

[bpGusar]

First Last: прописал это Storage::delete('img/' + $oldPosterName); выдает

(1/1) ErrorException
A non-numeric value encountered

[First Last]

Павел Бондаренко: а что у Вас в $oldPosterName?

[bpGusar]

First Last: $oldPosterName=$post->poster;

[First Last]

Павел Бондаренко: dd($oldPosterName);

[bpGusar]

First Last: выдало "1502049176.jpg", это название того постера который щас в базе в колонке poster

[First Last]

Павел Бондаренко: попробуйте вместо 'img/' + $oldPosterName -> "img/" + $oldPosterName;

[bpGusar]

First Last: нет ((
(1/1) ErrorException
A non-numeric value encountered

[First Last]

Павел Бондаренко:

$oldPosterName = 'img/' + $post->image;
dd($oldPosterName);

Попробуйте так, если dd сработает, уберите и попробуйте обновить постер.

[bpGusar]

First Last: выдало 'img/1236512.jpg' я сменил Storage::delete('img/' . $oldPosterName); на Storage::delete($oldPosterName); и один фиг не работает, че блин за нафиг, может из public_path нельзя ничего удалять а только записывать?

[First Last]

Павел Бондаренко: ну Вы же картинку сохраняете в /public/img/%name.%format%%
А пытаетесь удалить из /public/%name.%format%%

[bpGusar]

First Last: ну. так как $oldPosterName = 'img/' . $post->poster; на выходе получается img/1236512.jpg, то есть если сделать Storage::delete($oldPosterName); то будет так Storage::delete(/public/img/1236512.jpg); или я не так понимаю?

[First Last]

Павел Бондаренко: хм, точно, забыл совсем про public
Добавьте еще и его
'public/img/' + $oldPosterName;

[bpGusar]

First Last: сделал Storage::delete('img/' . $oldPosterName); и всё равно не работает

[bpGusar]

First Last: добавлял public не помогло

[First Last]

Павел Бондаренко: тогда без понятия, увы

[bpGusar]

First Last: ладно, спасибо за помощь