fix crud users
This commit is contained in:
@@ -38,20 +38,20 @@ class UsersController extends Controller
|
||||
return response()->json(['message' => 'logged out successfully']);
|
||||
}
|
||||
public function store(UsersRequest $request){
|
||||
$validate_data = $request->validated();
|
||||
$validate_data = $request->validated();
|
||||
|
||||
DB::beginTransaction();
|
||||
try{
|
||||
$user = User::create([
|
||||
'name' => $validate_data->name,
|
||||
'email' => $validate_data->email,
|
||||
'password' => Hash::make($validate_data->password),
|
||||
'firstname' => $validate_data->firstname,
|
||||
'lastname' => $validate_data->lastname,
|
||||
'position' => $validate_data->position
|
||||
'name' => $validate_data['name'],
|
||||
'email' => $validate_data['email'],
|
||||
'password' => Hash::make($validate_data['password']),
|
||||
'firstname' => $validate_data['firstname'],
|
||||
'lastname' => $validate_data['lastname'],
|
||||
'position' => $validate_data['position'],
|
||||
]);
|
||||
|
||||
$user->roles()->attach($request->role_id);
|
||||
$user->roles()->attach((int) $validate_data['role_id']);
|
||||
|
||||
DB::commit();
|
||||
return response()->json(['message' => 'Successfully created'],201);
|
||||
@@ -60,4 +60,27 @@ class UsersController extends Controller
|
||||
return response()->json(['message' => $e->getMessage()],500);
|
||||
};
|
||||
}
|
||||
public function update(UsersRequest $request, $id){
|
||||
try{
|
||||
$validate_data = $request->validated();
|
||||
$user = User::findOrFail($id);
|
||||
|
||||
DB::beginTransaction();
|
||||
$user->update([
|
||||
'name' => $validate_data['name'],
|
||||
'email' => $validate_data['email'],
|
||||
'firstname' => $validate_data['firstname'],
|
||||
'lastname' => $validate_data['lastname'],
|
||||
'position' => $validate_data['position']
|
||||
]);
|
||||
|
||||
$user->roles()->sync($request->role_id);
|
||||
|
||||
DB::commit();
|
||||
return response()->json(['message' => 'Successfully updated'], 200);
|
||||
}catch(\Exception $e){
|
||||
DB::rollBack();
|
||||
return response()->json(['message' => $e->getMessage()],500);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@@ -22,7 +22,7 @@ class UsersRequest extends FormRequest
|
||||
*/
|
||||
public function rules(): array
|
||||
{
|
||||
$userId = $this->route('user'); // Get user ID from route (used in update)
|
||||
$userId = $this->route('users'); // Get user ID from route (used in update)
|
||||
|
||||
return [
|
||||
'name' => ['required', 'string', 'max:255'],
|
||||
@@ -31,7 +31,7 @@ class UsersRequest extends FormRequest
|
||||
'string',
|
||||
'email',
|
||||
'max:255',
|
||||
Rule::unique('users')->ignore($userId), // Ignore the user's own email when updating
|
||||
Rule::unique('users')->ignore($userId)
|
||||
],
|
||||
'password' => [$this->isMethod('post') ? 'required' : 'nullable', 'confirmed', 'max:255'],
|
||||
'firstname' => ['required', 'string', 'max:255'],
|
||||
|
||||
Reference in New Issue
Block a user