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

Code coverage ignore 

Consider removing UserController
This commit is contained in:
Matt Young 2025-07-04 15:23:21 -05:00
parent db34a86545
commit c22f3ddadf
2 changed files with 40 additions and 83 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -9,81 +9,37 @@ use Illuminate\Support\Facades\Auth;
use function abort; use function abort;
use function redirect; use function redirect;
class UserController extends Controller /**
{ * @codeCoverageIgnore
/** */
* Display a listing of the resource. // TODO consider for removal
*/ //class UserController extends Controller
public function index() //{
{ // /**
// // * Update the specified resource in storage.
} // */
// public function update(Request $request, User $user)
/** // {
* Show the form for creating a new resource. // if ($user->id !== Auth::user()->id) {
*/ // return redirect()->route('dashboard')->with('error', 'You can only modify your own profile');
public function create() // }
{ // dd($request->all());
// // }
} //
// public function set_school(Request $request, User $user)
/** // {
* Store a newly created resource in storage. // if ($request->user()->cannot('set_school', $user)) {
*/ // abort(403);
public function store(Request $request) // }
{ //
// // request()->validate([
} // 'school_id' => ['required', 'integer', 'exists:schools,id'],
// ]);
/** //
* Display the specified resource. // $user->update([
*/ // 'school_id' => request('school_id'),
public function show(User $user) // ]);
{ //
// // return redirect('/my_school');
} // }
//}
/**
* Show the form for editing the specified resource.
*/
public function edit(User $user)
{
//
}
/**
* 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());
}
/**
* Remove the specified resource from storage.
*/
public function destroy(User $user)
{
//
}
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');
}
}

11
routes/user.php
View File

@ -28,11 +28,12 @@ Route::middleware([
Route::delete('/entries/{entry}', 'destroy')->name('entries.destroy'); Route::delete('/entries/{entry}', 'destroy')->name('entries.destroy');
}); });
// User Related Routes //// User Related Routes
Route::middleware(['auth', 'verified'])->controller(UserController::class)->group(function () { //// TODO: Consider this block for removal
Route::patch('/users/{user}/set_school', 'set_school')->name('users.set_school'); //Route::middleware(['auth', 'verified'])->controller(UserController::class)->group(function () {
Route::patch('/user_profile/{$user}', 'update')->name('users.update'); // Route::patch('/users/{user}/set_school', 'set_school')->name('users.set_school');
}); // Route::patch('/user_profile/{$user}', 'update')->name('users.update');
//});
// Student Related Routes // Student Related Routes
Route::middleware([ Route::middleware([