Call to a member function getClientOriginalExtension() on null what should do? - laravel

Question

public function store(Request $request)
{
$data = new product1();
$file = $request->file;
$filename= time().'.'.$file->getClientOriginalExtension();
$request->file->move('assets', $filename);
$data->file = $filename;
$data->name = $request->name;
$data->description = $request->description;
$data->author = $request->author;
$data->comment = $request->comment;
$data->save();
return redirect()->back();
}
View:
<form action="{{ url('uploadproduct') }}" method="post" enctype="multipart/form-data">
#csrf
<div class="form-group">
<input class="form-control" name="comment" placeholder="Write comment" type="text" style=" width: 50%;">
<input class="btn btn-primary" type="submit" value="Done" style=" width: 20%;">
</div>
</form>

Answer 1

$file is null when trying to get its properties.
You can either condition the process to the existence of $file (in case that the 'file' input is not required), or simply check what is happening that input value is null when requesting it.
Maybe there's no <input type="file" name="file"> in your form...
Or maybe your form has no enctype="multipart/form-data" property that allows you to submit files.
I DO also recommend you to validate your request before processing the data: https://laravel.com/docs/8.x/validation
public function store(Request $request)
{
$validator = Validator::make($request->all(), [
'file' => 'required'
]);
if ($validator->fails()) {
return Redirect::back()->withErrors($validator)->withInput();
}
// Here you start processing your inputs
}