matt
/
auditionadmin
1
0
Fork 0
Code Issues 16 Pull Requests Packages Projects Releases Wiki Activity

Correct issue where users were unable to choose their school.

This commit is contained in:
Matt Young 2025-08-17 10:03:52 -05:00
parent 3dc5ae1793
commit 5f360c7d3a
2 changed files with 35 additions and 34 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

60
app/Http/Controllers/UserController.php
View File

@ -13,33 +13,33 @@ use function redirect;
* @codeCoverageIgnore * @codeCoverageIgnore
*/ */
// TODO consider for removal // TODO consider for removal
//class UserController extends Controller class UserController extends Controller
//{ {
// /** /**
// * Update the specified resource in storage. * Update the specified resource in storage.
// */ */
// public function update(Request $request, User $user) public function update(Request $request, User $user)
// { {
// if ($user->id !== Auth::user()->id) { if ($user->id !== Auth::user()->id) {
// return redirect()->route('dashboard')->with('error', 'You can only modify your own profile'); return redirect()->route('dashboard')->with('error', 'You can only modify your own profile');
// } }
// dd($request->all()); dd($request->all());
// } }
//
// public function set_school(Request $request, User $user) public function set_school(Request $request, User $user)
// { {
// if ($request->user()->cannot('set_school', $user)) { if ($request->user()->cannot('set_school', $user)) {
// abort(403); abort(403);
// } }
//
// request()->validate([ request()->validate([
// 'school_id' => ['required', 'integer', 'exists:schools,id'], 'school_id' => ['required', 'integer', 'exists:schools,id'],
// ]); ]);
//
// $user->update([ $user->update([
// 'school_id' => request('school_id'), 'school_id' => request('school_id'),
// ]); ]);
//
// return redirect('/my_school'); return redirect('/my_school');
// } }
//} }

9
routes/user.php
View File

@ -7,6 +7,7 @@ use App\Http\Controllers\EntryController;
use App\Http\Controllers\PdfInvoiceController; use App\Http\Controllers\PdfInvoiceController;
use App\Http\Controllers\SchoolController; use App\Http\Controllers\SchoolController;
use App\Http\Controllers\StudentController; use App\Http\Controllers\StudentController;
use App\Http\Controllers\UserController;
use Illuminate\Support\Facades\Route; use Illuminate\Support\Facades\Route;
Route::middleware(['auth', 'verified'])->group(function () { Route::middleware(['auth', 'verified'])->group(function () {
@ -29,10 +30,10 @@ Route::middleware([
//// User Related Routes //// User Related Routes
//// TODO: Consider this block for removal //// TODO: Consider this block for removal
//Route::middleware(['auth', 'verified'])->controller(UserController::class)->group(function () { Route::middleware(['auth', 'verified'])->controller(UserController::class)->group(function () {
// Route::patch('/users/{user}/set_school', 'set_school')->name('users.set_school'); Route::patch('/users/{user}/set_school', 'set_school')->name('users.set_school');
// Route::patch('/user_profile/{$user}', 'update')->name('users.update'); // Route::patch('/user_profile/{$user}', 'update')->name('users.update');
//}); });
// Student Related Routes // Student Related Routes
Route::middleware([ Route::middleware([