diff --git a/app/Http/Controllers/UserController.php b/app/Http/Controllers/UserController.php index 978a3f5..565fc69 100644 --- a/app/Http/Controllers/UserController.php +++ b/app/Http/Controllers/UserController.php @@ -13,33 +13,33 @@ use function redirect; * @codeCoverageIgnore */ // TODO consider for removal -//class UserController extends Controller -//{ -// /** -// * Update the specified resource in storage. -// */ -// public function update(Request $request, User $user) -// { -// if ($user->id !== Auth::user()->id) { -// return redirect()->route('dashboard')->with('error', 'You can only modify your own profile'); -// } -// dd($request->all()); -// } -// -// public function set_school(Request $request, User $user) -// { -// if ($request->user()->cannot('set_school', $user)) { -// abort(403); -// } -// -// request()->validate([ -// 'school_id' => ['required', 'integer', 'exists:schools,id'], -// ]); -// -// $user->update([ -// 'school_id' => request('school_id'), -// ]); -// -// return redirect('/my_school'); -// } -//} +class UserController extends Controller +{ + /** + * Update the specified resource in storage. + */ + public function update(Request $request, User $user) + { + if ($user->id !== Auth::user()->id) { + return redirect()->route('dashboard')->with('error', 'You can only modify your own profile'); + } + dd($request->all()); + } + + public function set_school(Request $request, User $user) + { + if ($request->user()->cannot('set_school', $user)) { + abort(403); + } + + request()->validate([ + 'school_id' => ['required', 'integer', 'exists:schools,id'], + ]); + + $user->update([ + 'school_id' => request('school_id'), + ]); + + return redirect('/my_school'); + } +} diff --git a/routes/user.php b/routes/user.php index 37db0b8..fc48a90 100644 --- a/routes/user.php +++ b/routes/user.php @@ -7,6 +7,7 @@ use App\Http\Controllers\EntryController; use App\Http\Controllers\PdfInvoiceController; use App\Http\Controllers\SchoolController; use App\Http\Controllers\StudentController; +use App\Http\Controllers\UserController; use Illuminate\Support\Facades\Route; Route::middleware(['auth', 'verified'])->group(function () { @@ -29,10 +30,10 @@ Route::middleware([ //// User Related Routes //// TODO: Consider this block for removal -//Route::middleware(['auth', 'verified'])->controller(UserController::class)->group(function () { -// Route::patch('/users/{user}/set_school', 'set_school')->name('users.set_school'); -// Route::patch('/user_profile/{$user}', 'update')->name('users.update'); -//}); +Route::middleware(['auth', 'verified'])->controller(UserController::class)->group(function () { + Route::patch('/users/{user}/set_school', 'set_school')->name('users.set_school'); + // Route::patch('/user_profile/{$user}', 'update')->name('users.update'); +}); // Student Related Routes Route::middleware([