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')
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
hab auch lokale variable genommen wenn öfters $request->user() genommen wurde oder andere ähnlcihe zugriffe da wo es sich gedopppelt hat
@@ -26,3 +27,4 @@{$user->forceFill(['is_admin' => $request->boolean('is_admin'),hier ist noch nen $request->boolean
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')
@@ -115,1 +122,3 @@$this->projectPermissionService->syncUserPermissions($project, $user, $request->permissions());$validated = $request->validated();$permissions = collect(Arr::get($validated, 'permissions'))->map(fn (mixed $permission): string => (string) $permission)das kann alles raus.
einfach distinct in die request packen und dann einfach nur:
$permissions = Arr::get($validated, 'permissions', [])
@@ -132,0 +147,4 @@->map(fn (mixed $permission): string => (string) $permission)->unique()->values()->all();selbe wie oben
@@ -23,1 +25,4 @@{$user = $request->user();$session = $request->session();hier brauchst du keine lokalen vars, beides wird ja nur einmal genutzt
@@ -45,0 +51,4 @@$apiToken = Arr::get($validated, 'api_token');$baseUrl = Arr::get($validated, 'base_url');$provider = GitProviderKey::from((string) Arr::get($validated, 'provider'));(string) cast kann weg, das validiert die request ja schon
@@ -78,0 +91,4 @@$validated = $request->validated();$apiToken = Arr::get($validated, 'api_token');$baseUrl = Arr::get($validated, 'base_url');$provider = GitProviderKey::from((string) Arr::get($validated, 'provider'));selbe hier mit cast
@@ -53,0 +53,4 @@$validated = $request->validated();$userIds = collect(Arr::get($validated, 'user_ids', []))->map(fn (mixed $userId): int => (int) $userId)hier auch einfach $userIds = Arr::get($validated, 'user_ids', []) und distinct ind request dazu
das collect->map->unique->values->all mach gar nichts.
hier hast du vergessen distinct in die request reinzumachen
@@ -181,18 +181,6 @@ class UpdateProjectTicketRequest extends FormRequest* value_user_id?: int|string|null* }>*/php docs auch mit löschen
@@ -32,12 +32,4 @@ class UpdateTeamMembersRequest extends FormRequest/*** @return array<int, int>*/hier auch php docs vergessen