PetschStudios/KanbanGit
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases 27 Wiki Activity

delete-functions-in-request-arr-get-replacement #24

Merged
Twyco merged 9 commits from delete-functions-in-request-arr-get-replacement into main 2026-04-11 12:51:57 +00:00
Conversation 1 Commits 9 Files Changed 28
+4 -4
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 848f7694e6 - Show all commits
app/Http/Controllers/Admin/UserManagementController.php
+3 -3
View File
@@ -26,9 +26,9 @@ class UserManagementController extends Controller
public function update(UpdateUserAdminStatusRequest $request, User $user): RedirectResponse
{
$validated = $request->validated();
$user->is_admin = $validated['is_admin'];
$user->save();
$user->forceFill([
'is_admin' => Arr::get($validated, 'is_admin'),
Twyco commented 2026-04-04 09:31:51 +00:00
Outdated
Review
Copy Link
Copy Source

hier ist noch nen $request->boolean

hier ist noch nen $request->boolean
Twyco commented 2026-04-11 12:43:23 +00:00
Outdated
Review
Copy Link
Copy Source

das geht so (glaube ich) nicht. Du kannst darfst das forceFill nciht entfernen, weil is_admin ist kein fillable, daher forceFill.

außerdem benutzt du jetzt $validated['is_admin'] statt Arr::get($validated, 'is_admin')

das geht so (glaube ich) nicht. Du kannst darfst das forceFill nciht entfernen, weil is_admin ist kein fillable, daher forceFill. außerdem benutzt du jetzt $validated['is_admin'] statt Arr::get($validated, 'is_admin')
])->save();
$status = $user->is_admin
? "{$user->name} now has admin access."
app/Http/Requests/UpdateTeamMembersRequest.php
+1 -1
View File
@@ -25,7 +25,7 @@ class UpdateTeamMembersRequest extends FormRequest
{
return [
'user_ids' => ['nullable', 'array'],
'user_ids.*' => ['integer', Rule::exists('users', 'id')],
'user_ids.*' => ['integer', 'distinct', Rule::exists('users', 'id')],
];
}