При методе update появляется ошибка, когда пытаешься изменить текст так ещё и удаляется фотография, почему?

Question

[Denys03]

Честно, уже вообще не понимаю что происходит, что я делаю не так, пожалуйста подскажите что с этим storage не так.
Во первых warning в терминале начал появляться:

[404]: GET /storage/products - No such file or directory.

В env указал:

APP_URL=http://127.0.0.1:8000
FILESYSTEM_DISK=public

При попытке изменить текст, не добавляя фотографии выпадает ошибка:
Call to a member function store() on null
На строку:

$path = $request->file('image')->store('categories');

Ошибка именно с методом update. Создать и удалить работают нормально.
Сам метод, он не доработан, но по идеи он не должен себя так вести, типа если не изменяешь фото, то ошибка:

public function update(Request $request, category $category): \Illuminate\Http\RedirectResponse
    {
        Storage::delete($category->image);
        $path = $request->file('image')->store('categories');
        $params = $request->all();
        $params['image'] = $path;
        $category->update($params);
        return redirect()->route('categories.index');
    }

Ну и при попытке отредактировать какой-то текст в поле названии категории например, вызывается ошибка, если вернутся в категории, ранее добавленная фотография исчезает.

Недавно был один момент. Я store убрал, и написал вот так:
$path = $request->file('image', 'categories');
Я знаю, тупо, но для эксперимента пойдёт, и у меня каким-то образом начали сохраняться записи без добавления фотографии, все бы нормально, но, удалив все фотографии из папок, снова пошли ошибки.

Comments

[Дмитрий]

потому что если вы ничего не загружаете в фото - у вас $request->file('image') = null. Интерпретатор вам честно об этом говорит - не могу вызывать метод store у null

[Denys03]

Дмитрий,
А на счёт этой ошибки что скажете?

http://league/Flysystem/Filesystem::delete():%20Argument%20#1%20($location)%20must%20be%20of%20type%20string,%20null%20given,%20called%20in%20G:\Dir\Test\project\vendor\laravel\framework\src\Illuminate\Filesystem\FilesystemAdapter.php%20on%20line%20496

public function delete(string $location): void

[Дмитрий]

Denys03, ну там написано. переменная $location должна быть типа string, а мне дают null - я так не играю.

как бы английский необходим ваще то.

[Denys03]

Дмитрий, Я вот сделал чтобы фотография не удалялась при любом изменении и все равно пишет что нулл что за бред

[Denys03]

Дмитрий, Просто бесит хрень, вчера чисто случайно заработало, хотя я ничего не менял, а сейчас не работает.
почему тогда с create всё нормально, именно редактировать самым для меня оказался таким себе

[Дмитрий]

Denys03, что вы хотите? ну проверяйте

if($request->hasFile('image')){
удаляем старую если есть
загружаем новую
}

чего вы херней страдаете.

почему тогда с create всё нормально, именно редактировать самым для меня оказался таким себе

я не вижу кода create - не могу сказать, мой персональный телепат в отпуске

[JhaoDa]

Denys03, да-да, ты ничего не менял, оно само...

[Denys03]

JhaoDa, мда, смысл писать, если ничего подсказать не можешь, гений?

[Denys03]

Дмитрий, Я хочу понять как это работает, вот вы кинули метод, есть ресурсы, где можно понять более детально как это работает. Я взял этот пример что скинул из видеоурока, у автора он работал нормально, возможно на более старой версии позволялось допускать небольшие мелочи, сейчас видимо пофиксили

[Дмитрий]

Denys03, нет. Попытка использовать метод у null всегда вызывала ошибку. Попытка обратиться к файлу который не загружали - всегда возвращала null. Что там в уроке я не знаю. Но с ларавел работаю с 4 версии. Сие не менялось

Ну а детали - господи кликаете мышкой на методе нужном и смотрите его код. И все. Как бы нет ничего лучше чем посмотреть код который вы вызывает

[Denys03]

Дмитрий, Странно, я точь в точь из коммита всё взял, в комментариях тоже не кто за ошибку не писал, хотя до этого когда ошибки были, то все писали, блин, буду знать тогда, я просто новичок так сказать в этом, и очень многого не знаю, даже тот метод что вы скинули, мне не особо был знаком, так что я 0.

[Дмитрий]

Denys03, ссылку на коммит?

[Denys03]

Дмитрий, Держите, может что-то упустил, но по идеи я каждый файл пересмотрел.

<a href="https://github.com/warlight/internet-shop-course-youtube/commit/5c6c74961f2fc9d3b4c6db8cd93edd77f1d3fe9a"></a>

По этому курсу и учусь, решил прям из коммита скопировать чтобы точно не допустить ошибку, думал я что-то делаю не так.

[Дмитрий]

Denys03, ссылку дайте на код который вы смотрите в интернете? в ответе его нет?

[Denys03]

Дмитрий, Вот видео, можете посмотреть, у него работает, бывает что я не внимательный, но он же коммиты правильно ведет как минимум.

<a href="https://www.youtube.com/watch?v=_OatPW8NQKc&list=PL5RABzpdpqAlSRJS1KExmJsaPFQc161Dy&index=16&ab_channel=rdavydov"></a>

[Дмитрий]

Denys03, ну этот код будет ровно так же будет не работать если нет картинки.

а вот окончательный код -
https://github.com/warlight/internet-shop-course-y...

public function store(CategoryRequest $request)
    {
        $params = $request->all();
        unset($params['image']);
        if ($request->has('image')) {
            $params['image'] = $request->file('image')->store('categories');
        }

        Category::create($params);
        return redirect()->route('categories.index');
    }

    public function update(CategoryRequest $request, Category $category)
    {
        $params = $request->all();
        unset($params['image']);
        if ($request->has('image')) {
            Storage::delete($category->image);
            $params['image'] = $request->file('image')->store('categories');
        }

        $category->update($params);
        return redirect()->route('categories.index');
    }

будет работать. Хотя я бы посоветовал использовать подготовку параметров в реквестах если уж завели, и удаление файла делать не хрена не в контроллере - не место там таким вещам, равно как и открыть для себя $request->validated();. Для создателя курсов как то косяков много в таком минимуме кода.

[Denys03]

Дмитрий, Он на оборот говорил так правильно :D Я понял, хорошо, спасибо, буду разбираться дальше. В след уроке про валидацию пойдёт как раз реквесты

[Дмитрий]

Denys03, что именно? подготовка параметров в реквестах? ну а для чего она там есть? удаление файлов в контроллере - ну откройте любую статью "MVC для чайников" и там будет написано что где должно лежать. А как только человек откроет для себя более менее сложные приложения - когда нужно пользовательский интерфейс, какое нибудь api, разборка очереди, какие нибудь импорты и интеграции, и ему один раз придется это удаление исправить в 10 местах - он сам изобретет правило тонкие контроллеры толстые модели. validated не правильно? ну это вообще смешно - пихать в модель не валидированные данные. за такое можно в конторе и по лбу получить.