Form multiple, если при submit оставить поле пустым, то вываливается ошибка, кто подскажет почему?

Question

[Flasher]

Есть форма:

<form action="" method="post" enctype="multipart/form-data">
    <input type="file" name="cover[]" multiple/>
    <input type="submit" value="Отправить">
</form>

В контроллере:
$files = $request->file('cover');

Я хочу чекнуть, если поле пустое, то ловлю exception:

Argument 1 passed to Illuminate\Http\UploadedFile::createFromBase() must be an instance of Symfony\Component\HttpFoundation\File\UploadedFile, null given, called in C:\webserver\data\htdocs\topnews\vendor\laravel\framework\src\Illuminate\Http\Request.php on line 415 and defined

Если пытаюсь проверить:
if ($request->hasFile('cover'))
то все равно данная ошибка

Answer 1

[Flasher]

Решение: откатиться до версии 5.1.*

Answer 2

[Andrzej Wielski]

if($request->hasFile('cover')){
 //тут обрабатывай файл
}

Если файлов много - name инпута все равно должен быть cover (а не cover[]).
Обработка нескольких файлов в Laravel:

if($request->hasFile('cover')){
  foreach($request->file('cover') as $file){
    // $file
  }
}

Comments

[Flasher]

пробовал и такой вариант, все тот же exception

[Andrzej Wielski]

Flasher: попробуйте второй вариант. Только что изменил ответ.

[Flasher]

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

[look2009]

Сделай print_r ($request); посмотри что там, если уж if не отрабатывает.

[Andrzej Wielski]

Flasher: только что проверил. hasFile выдает false, если файлы не выбраны.
Если выбраны - идет их обработка через foreach.

Вы явно делаете что-то неправильно. Можете показать код функции-контроллера, и html код формы?

[Flasher]

look2009: если в контроллере есть $request->file('cover') то ошибка, даже при print_r, Andrzej Wielski, вот контроллер pastebin.com/76Kr49tW, а вот HTML pastebin.com/rzEX2VbC

[Andrzej Wielski]

Flasher: Какая версия Laravel?

[Flasher]

Andrzej Wielski: поставил 5.2 и уже на это грешу

[Andrzej Wielski]

Flasher: Решение не очень, но...

try {
       if ($request->hasFile('cover')) {
            $files = $request->file('cover');
            foreach ($files as $file) {
                $image_name = time() . $file->getClientOriginalName();
                $arr_files[] = $image_name;
                $file->move('uploads', $image_name);
                Image::make(sprintf('uploads/%s', $image_name))->resize(500, null)->save();
            }
        }
} catch (Exception $e){
  // Не получилось, что поделать...
}

[Flasher]

Andrzej Wielski: попробую поставить 5.1, так как exception мне не надо отлавливать, мне надо разрешать оставить это поле пустым по желанию.

[Andrzej Wielski]

Flasher: В данном случае если вывалится exception - значит файл не выбран. Поле будет пустым.
catch можно оставить пустым - дальнейший код выполнится спокойно.

Кстати, вы не забыли поместить все роуты в группу с middleware web? Обычно все проблемы на L5.2 возникает именно из-за этой оплошности. Проверить нет возможности - пока на проектах сижу на 5.1, многие используемые пекейджы не поддерживают 5.2

[Flasher]

Andrzej Wielski: тут все просто с роутами

Route::get('/', function () {
return view('welcome');
});

Route::post('/', 'PhotoController@upload');

[Andrzej Wielski]

Flasher: в Laravel 5.2 надо все оборачивать в группу:

Route::group(['middleware' => ['web']], function () {
    Route::get('/', function () {
        return view('welcome');
    });

    Route::post('/', 'PhotoController@upload');
});