mirror of
https://github.com/Einundzwanzig-Podcast/einundzwanzig-portal.git
synced 2025-12-11 06:46:47 +00:00
Add PHP type hints for Laravel 10
This commit is contained in:
committed by
HolgerHatGarKeineNode
parent
ad55c69b51
commit
98a430c91c
@@ -2,6 +2,7 @@
|
||||
|
||||
namespace App\Actions\Jetstream;
|
||||
|
||||
use Closure;
|
||||
use Illuminate\Support\Facades\Gate;
|
||||
use Illuminate\Support\Facades\Validator;
|
||||
use Laravel\Jetstream\Contracts\AddsTeamMembers;
|
||||
@@ -21,7 +22,7 @@ class AddTeamMember implements AddsTeamMembers
|
||||
* @param string|null $role
|
||||
* @return void
|
||||
*/
|
||||
public function add($user, $team, string $email, string $role = null)
|
||||
public function add($user, $team, string $email, string $role = null): void
|
||||
{
|
||||
Gate::forUser($user)->authorize('addTeamMember', $team);
|
||||
|
||||
@@ -46,7 +47,7 @@ class AddTeamMember implements AddsTeamMembers
|
||||
* @param string|null $role
|
||||
* @return void
|
||||
*/
|
||||
protected function validate($team, string $email, ?string $role)
|
||||
protected function validate($team, string $email, ?string $role): void
|
||||
{
|
||||
Validator::make([
|
||||
'email' => $email,
|
||||
@@ -63,7 +64,7 @@ class AddTeamMember implements AddsTeamMembers
|
||||
*
|
||||
* @return array
|
||||
*/
|
||||
protected function rules()
|
||||
protected function rules(): array
|
||||
{
|
||||
return array_filter([
|
||||
'email' => ['required', 'email', 'exists:users'],
|
||||
@@ -80,7 +81,7 @@ class AddTeamMember implements AddsTeamMembers
|
||||
* @param string $email
|
||||
* @return \Closure
|
||||
*/
|
||||
protected function ensureUserIsNotAlreadyOnTeam($team, string $email)
|
||||
protected function ensureUserIsNotAlreadyOnTeam($team, string $email): Closure
|
||||
{
|
||||
return function ($validator) use ($team, $email) {
|
||||
$validator->errors()->addIf(
|
||||
|
||||
@@ -12,7 +12,7 @@ class DeleteTeam implements DeletesTeams
|
||||
* @param mixed $team
|
||||
* @return void
|
||||
*/
|
||||
public function delete($team)
|
||||
public function delete($team): void
|
||||
{
|
||||
$team->purge();
|
||||
}
|
||||
|
||||
@@ -32,7 +32,7 @@ class DeleteUser implements DeletesUsers
|
||||
* @param mixed $user
|
||||
* @return void
|
||||
*/
|
||||
public function delete($user)
|
||||
public function delete($user): void
|
||||
{
|
||||
DB::transaction(function () use ($user) {
|
||||
$this->deleteTeams($user);
|
||||
@@ -48,7 +48,7 @@ class DeleteUser implements DeletesUsers
|
||||
* @param mixed $user
|
||||
* @return void
|
||||
*/
|
||||
protected function deleteTeams($user)
|
||||
protected function deleteTeams($user): void
|
||||
{
|
||||
$user->teams()->detach();
|
||||
|
||||
|
||||
@@ -2,6 +2,7 @@
|
||||
|
||||
namespace App\Actions\Jetstream;
|
||||
|
||||
use Closure;
|
||||
use Illuminate\Support\Facades\Gate;
|
||||
use Illuminate\Support\Facades\Mail;
|
||||
use Illuminate\Support\Facades\Validator;
|
||||
@@ -23,7 +24,7 @@ class InviteTeamMember implements InvitesTeamMembers
|
||||
* @param string|null $role
|
||||
* @return void
|
||||
*/
|
||||
public function invite($user, $team, string $email, string $role = null)
|
||||
public function invite($user, $team, string $email, string $role = null): void
|
||||
{
|
||||
Gate::forUser($user)->authorize('addTeamMember', $team);
|
||||
|
||||
@@ -47,7 +48,7 @@ class InviteTeamMember implements InvitesTeamMembers
|
||||
* @param string|null $role
|
||||
* @return void
|
||||
*/
|
||||
protected function validate($team, string $email, ?string $role)
|
||||
protected function validate($team, string $email, ?string $role): void
|
||||
{
|
||||
Validator::make([
|
||||
'email' => $email,
|
||||
@@ -65,7 +66,7 @@ class InviteTeamMember implements InvitesTeamMembers
|
||||
* @param mixed $team
|
||||
* @return array
|
||||
*/
|
||||
protected function rules($team)
|
||||
protected function rules($team): array
|
||||
{
|
||||
return array_filter([
|
||||
'email' => ['required', 'email', Rule::unique('team_invitations')->where(function ($query) use ($team) {
|
||||
@@ -84,7 +85,7 @@ class InviteTeamMember implements InvitesTeamMembers
|
||||
* @param string $email
|
||||
* @return \Closure
|
||||
*/
|
||||
protected function ensureUserIsNotAlreadyOnTeam($team, string $email)
|
||||
protected function ensureUserIsNotAlreadyOnTeam($team, string $email): Closure
|
||||
{
|
||||
return function ($validator) use ($team, $email) {
|
||||
$validator->errors()->addIf(
|
||||
|
||||
@@ -18,7 +18,7 @@ class RemoveTeamMember implements RemovesTeamMembers
|
||||
* @param mixed $teamMember
|
||||
* @return void
|
||||
*/
|
||||
public function remove($user, $team, $teamMember)
|
||||
public function remove($user, $team, $teamMember): void
|
||||
{
|
||||
$this->authorize($user, $team, $teamMember);
|
||||
|
||||
@@ -37,7 +37,7 @@ class RemoveTeamMember implements RemovesTeamMembers
|
||||
* @param mixed $teamMember
|
||||
* @return void
|
||||
*/
|
||||
protected function authorize($user, $team, $teamMember)
|
||||
protected function authorize($user, $team, $teamMember): void
|
||||
{
|
||||
if (! Gate::forUser($user)->check('removeTeamMember', $team) &&
|
||||
$user->id !== $teamMember->id) {
|
||||
@@ -52,7 +52,7 @@ class RemoveTeamMember implements RemovesTeamMembers
|
||||
* @param mixed $team
|
||||
* @return void
|
||||
*/
|
||||
protected function ensureUserDoesNotOwnTeam($teamMember, $team)
|
||||
protected function ensureUserDoesNotOwnTeam($teamMember, $team): void
|
||||
{
|
||||
if ($teamMember->id === $team->owner->id) {
|
||||
throw ValidationException::withMessages([
|
||||
|
||||
@@ -16,7 +16,7 @@ class UpdateTeamName implements UpdatesTeamNames
|
||||
* @param array $input
|
||||
* @return void
|
||||
*/
|
||||
public function update($user, $team, array $input)
|
||||
public function update($user, $team, array $input): void
|
||||
{
|
||||
Gate::forUser($user)->authorize('update', $team);
|
||||
|
||||
|
||||
Reference in New Issue
Block a user