Laravel 5.3 - image upload not working

Level 1

Try :

$imageName = $request->file('image')->getClientOriginalName();

$request->file('image')->move(
    base_path() . '/public/uploads/', $imageName
);

Hope this helps.

1 like

Level 22

You have a column in your table named image that is not nullable and you're never passing any data to it. Assuming you wanted the path to the image you need to add $model->update(['image' => $path]); line to your script above (where $model is whatever instance variable you use)

1 like

Level 4

@Indemnity83 But the image never gets uploaded in the first place, I was wondering mostly about that, I am aware of the error, I will set the field in the DB to nullable, but I wonder why nothing gets uploaded to the server?

Level 22

Sorry. Typing on phone and got carried away. The update won't work now that I think about it.

You must have a Model::create call in this code somewhere; you need to add the 'image' => $path in there; not in an update.

What does the rest of your controller method look like.

Level 22

public function store(Request $request)
{
    // Do your validation

    $path = $request->file('image')->store('uploads/', 'public');

    $thing = Thing::create([
        'field1' => $request->input('field1'),
        'field2' => $request->input('field2'),
        'field3' => $request->input('field3'),
        'image' => $path,
    ]);
    
}

Level 4

@Indemnity83 Sorry, for the late reply, but I also have the setup like you have shown, in my controller, only the image is not being uploaded.

Level 22

The error your listing points to a problem in your Thing::create() call though; not the image upload. What does your controller method look like? It can't be identical to what I listed as an example.

Level 4

I got rid of the error, not sure why did it even show in the first place, by uploading again, but still nothing gets uploaded, I get a value for $path, that looks like this:

uploads/d87478f7c4717c356353a760fab9addc.jpeg

This is my controller:

public function store(Request $request)
    { 
        $path = $request->file('image')->store('uploads', 'public');

        $club = Club::create([
          'image' => $path,
          'name' => $request->input('name'),
          'adress' => $request->input('adress'),
          'country' => $request->input('country'),
          'city' => $request->input('city'),
          'telephone' => $request->input('telephone'),
          'email' => $request->input('email'),
          'manager' => $request->input('manager'),
          'established' => $request->input('established'),
        ]);

        return view('clubs.show', compact('club'));
    }

I can use the old way of uploading an image and then it works:

        $file = $request->file('image');
        $original_name = $file->getClientOriginalName();
        $imageName = uniqid(true) . $original_name;
        $path = public_path('uploads/' . $imageName);
        Image::make($file)->save($path);

But I wonder why this in 5.3 version recommended way won't work?

Level 22

so you're saying that the store method is returning a value of uploads/d87478f7c4717c356353a760fab9addc.jpeg but there is no file there?

have you looked in /storage/app/ and /storage/app/public?