Laravel Update Method Error

Question

i have a method to update image path to data base but i'm getting this error

ErrorException in DriversController.php line 296:
Attempt to assign property of non-object

in my controller

public function updateDriver(Request $request, $id)
{
    $driver = Driver::find($id)->update($request->all());

    if($request->hasFile('profile-photo')) {

            $image_file = $request->file('profile-photo');

            $get_image_name = $request['first_name'].$request['phone_number'].'.'.$image_file->getClientOriginalExtension();
            $image_name = preg_replace('/\s+/', '', $get_image_name);

            $s3 = \Storage::disk('s3');
            $filePath = '/drivers/' . $image_name;
            $s3->put($filePath, file_get_contents($image_file), 'public');

            $driver->profile_photo = $image_name;   //this is ware the error line 296 
            $driver->save();
    }

    return redirect()->back()->with('message', 'Driver updater successfully');
}

Thank you

@RiggsFolly thanks for your time, the line is $driver->profile_photo = $image_name; — Mr Robot
– Mr Robot, Commented May 8, 2017 at 10:40
$driver = Driver::find($id)->update($request->all()); this line failed.. you $driver variable is null or false.. that's why you can't assign anything to it.. — Mihai Matei
– Mihai Matei, Commented May 8, 2017 at 10:43
$driver->profile_photo? how will you get it? as there is no object thus that error. use something like this by removing these two lines $driver->profile_photo = $image_name; $driver->save(); to this $driver = Driver::find($id)->update(array('profile_photo'=>$image_name)); — Exprator
– Exprator, Commented May 8, 2017 at 10:54

Patrik · Accepted Answer · 2017-05-08 10:48:40Z

2

$driver will be a Boolean (I believe) after the update.

Try:

$driver = Driver::find($id);
$driver->update($request->all());

answered May 8, 2017 at 10:48

Patrik

712 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

AddWeb Solution Pvt Ltd · Accepted Answer · 2017-05-08 10:49:11Z

1

You can update your code like :

public function updateDriver(Request $request, $id)
{
    $driver = Driver::find($id)->update($request->all());

   if($request->hasFile('profile-photo')) {

           $image_file = $request->file('profile-photo');

           $get_image_name = $request['first_name'].$request['phone_number'].'.'.$image_file->getClientOriginalExtension();
            $image_name = preg_replace('/\s+/', '', $get_image_name);

           $s3 = \Storage::disk('s3');
            $filePath = '/drivers/' . $image_name;
            $s3->put($filePath, file_get_contents($image_file), 'public');

           $driver = Driver::find($id)->update(array('profile_photo'=>$image_name));

   }

   return redirect()->back()->with('message', 'Driver updater successfully');
}

answered May 8, 2017 at 10:49

AddWeb Solution Pvt Ltd

21.8k5 gold badges30 silver badges61 bronze badges

1 Comment

Mr Robot Over a year ago

thank you for your time really appreciate it, it worked

Collectives™ on Stack Overflow

Laravel Update Method Error

2 Answers 2

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related