Ошибка загрузки файла в папку, как решить?

Question

[Дмитрий Кузнецов]

Доброго времени суток! То ли я туповат, то ли что...Делаю загрузку изображения в нужную папку, предварительно создавая её. Вот код:

if($request->hasFile('cover_news')){
				// Если мы загружаем обложку
				$storagePath = Storage::allDirectories($path);
				if(!$storagePath){
					// Если нету папки, то создаём её
					Storage::makeDirectory($path);
				}

				// Создаём название картинки для сохранения
				$imageCover = $addNews->id_news . 'cover.' . $request->cover_news->extension();

				// Загружаем картинку в папку
				$uploadCover = Image::make($request->cover_news)->save($path . '/id' . $addNews->id_news . '/' . $imageCover);
			}

Именно эта часть кода выводит следующую ошибку:

Can't write image data to path (/public/news/id1/1cover.png)

PS: Делал линк для папки storage.
PSS: Работаю на Windows, Open Server.

Помогите решить эту маленькую проблему.

Comments

[zorca]

Подробно опишите, как создаете папку. Хотя нашел, отбой. :-)

[Дмитрий Кузнецов]

Windows, Open Server. Так что права не важны)

Answer 1

[pantagruel964]

File::makeDirectory($path, 0755, true, true)

Comments

[Дмитрий Кузнецов]

Windows, Open Server. Так что права не важны)

[zorca]

Да и Лара 5.4 :-)

[Дмитрий Кузнецов]

zorca, не 5,4, а 5,5,*)

[pantagruel964]

Дмитрий Кузнецов, Да хоть win95, denwer. Права нужно явно указать, чтобы перейти к 3-му аргументу, который по-умолчанию false. И зачем вы используете фасад Storage??

[Дмитрий Кузнецов]

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

[zorca]

Дмитрий Кузнецов, хотя в принципе все понятно. Вы пытаетесь писать в некую относительную директорию public. А должны писать в директорию с полным путем. Добавьте перед всем public_path().

[Дмитрий Кузнецов]

pantagruel964, Да и раньше такого не требовалось. Хотя может это из-за того, что я сохраняю в папку storage, а раньше, когда всё работало, сохранял в public.

[Дмитрий Кузнецов]

Дмитрий Кузнецов, теперь пишет: "mkdir(): No such file or directory"

[Дмитрий Кузнецов]

$path = storage_path('/public/news/');
		$imageName = 'cover_' . md5($path . time());
		$addNews = true;

		if($addNews){
			if($request->hasFile('cover_news')){
				// Если мы загружаем обложку
				$storagePath = Storage::allDirectories($path);
				if(!$storagePath){
					// Если нету папки, то создаём её
//					File::makeDirectory($path, 0755, true, true);
					Storage::makeDirectory($path);
				}

				// Создаём название картинки для сохранения
				$imageCover = $imageName . '.' . $request->cover_news->extension();

				// Загружаем картинку в папку
				$uploadCover = Image::make($request->cover_news)->save($path . '/id' . $addNews->id_news . '/' . $imageCover);
			}

			return redirect()->route('news.getNews', $addNews->id_news)->with('successMessages', 'Вы успешно добавили новость!');
		}

[zorca]

Дмитрий Кузнецов, через хелпер сделайте запись: https://laravel.com/docs/5.5/helpers#method-public-path , путь у вас должен быть не /public/fbcdhfdhfchfch , а С:/OpenServer и так далее.

[Дмитрий Кузнецов]

zorca, так и есть. Полный путь...

D:\OpenServer\domains\netrone.dev\storage

D:\OpenServer\domains\netrone.dev\public

[zorca]

Дмитрий Кузнецов, сейчас в переменной $path что именно? Сделайте var_dump. Ошибка говорит о том, что путь не полный, а относительно корня проекта. Должен быть от диска C:, или где у вас стоит локальный сервер.

[pantagruel964]

$dir = File::dirname($path);

if (!File::isDirectory($dir)) {
    File::makeDirectory($dir, 0755, true, true);
}

[zorca]

Дмитрий Кузнецов, интересно, а если заранее создать папку с id и так далее? Тот случай, когда подключившись по удаленке проблему решить можно в 1000 раз быстрее.

[Дмитрий Кузнецов]

zorca, путь: "D:\OpenServer\domains\netrone.dev\storage\/public/news/cover/".
Папки создал, ошибка не изменилась.
pantagruel964, фасад File какой?

[zorca]

Дмитрий Кузнецов, ахах. \/ в пути не смущает? :-) Слеш один лишний, видимо при генерации $path.

[pantagruel964]

Дмитрий Кузнецов,
use File

[Дмитрий Кузнецов]

zorca, переписывал код просто, пропустил. Ни чего не поменялось.

[Дмитрий Кузнецов]

pantagruel964, zorca, проблема решена. Сделал так:

$path = 'storage/news/cover/';
		$imageName = 'cover_' . md5($path . time());

		$addNews = true;

		if($addNews){
			if($request->hasFile('cover_news')){
				// Если мы загружаем обложку
				$storagePath = Storage::allDirectories($path);
				if(!$storagePath){
					// Если нету папки, то создаём её
//					File::makeDirectory($path, 0755, true, true);
					Storage::makeDirectory($path);
				}

				// Название изображения ('cover_' . md5($path . time()).jpg)
				$imageCover = $imageName . '.' . $request->cover_news->extension();

				// Загружаем картинку в папку
				$uploadCover = Image::make($request->cover_news)->save($path . $imageCover);
			}

			return redirect()->route('news.getNews', $addNews->id_news)->with('successMessages', 'Вы успешно добавили новость!');
		}

Answer 2

[Дмитрий Кузнецов]

Решение проблемы: Прописать в переменную path путь без всяких хелперов.

$path = 'storage/news/cover/';
		$imageName = 'cover_' . md5($path . time());

		$this->validate($request, [
			'title_news' => 'required',
			'short_news' => 'required',
			'full_news' => 'required',
			'cover_news' => 'required|mimes:jpeg,jpg,png',
		]);

		$addNews = News::create([
			'author_news' => Auth::user()->id_users,
			'title_news' => $request->title_news,
			'short_news' => $request->short_news,
			'full_news' => $request->full_news,
			'cover_news' => $request->cover_news,
		]);

		if($addNews){
			if($request->hasFile('cover_news')){
				// Если мы загружаем обложку
				$storagePath = Storage::allDirectories($path);
				if(!$storagePath){
					// Если нету папки, то создаём её
//					File::makeDirectory($path, 0755, true, true);
					Storage::makeDirectory($path);
				}

				// Название изображения ('cover_' . md5($path . time()).jpg)
				$imageCover = $imageName . '.' . $request->cover_news->extension();

				// Загружаем картинку в папку
				$uploadCover = Image::make($request->cover_news)->save($path . $imageCover);
			}

			return redirect()->route('news.getNews', $addNews->id_news)->with('successMessages', 'Вы успешно добавили новость!');
		}