updateOrCreate Function in laravel

Question

Code is fine and I'll double check my code but I didn't find anything wrong in my code. This doesn't update or insert address or new user data in any condition.

$user = User::updateOrCreate(
 [
 'id' => Auth::id(),
 'address' => auth()->user()->address
 ],
 [
 'address' => $request->address
 ]
);
if ($user)
 return response()->json([
 'status' => 200,
 'add' => $request->address,
 'msg' => 'Address Updated Successfully'
 ]);
else
 return response()->json([
 'status' => 200,
 'msg' => 'Error Occured'
 ]);

Same Code here just taking user_id in parameter

$user = User::updateOrCreate(
 [
 'id' => $request->id
 ],
 [
 'address' => $request->address
 ]
);
if ($user)
 return response()->json([
 'status' => 200,
 'add' => $request->address,
 'msg' => 'Address Updated Successfully'
 ]);
else
 return response()->json([
 'status' => 200,
 'msg' => 'Error Occured'
 ]);

• parth: Can you show what's coming in Auth::id()?
• jef: Try adding address to your User Model $fillable ?
• linktoahref: Check if the controller action is within the auth middleware to obtain Auth::id(). For user_id in parameter, shouldn't it be $request->user_id ?
• Beast69: Authenticated user id is coming up in Auth::id(). @parth
• Beast69: yeah let me try model $filable thing for address that may work for me @jef

Answer 1

If you use "updateOrCreate" method in UserController, this is not a good practice to make them in the same endpoint. Normally, "updateOrCreate" should be used for secondary type of data. For main data, creating and updating should hit the separate endpoints thus separate methods.

Furthermore, If you have Auth::id(), then you have a user. So there is nothing to create. To update the user, just use update method and get the user with id.

To create a user, use the "store" method pass the values and create it there.