App\Src\UseCases\Domain\Invitation\AttachUserToAnOrganization

Complexity

Total Complexity

4

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	sendMailUserJoinsToAdmin()	0	7	2
A	__construct()	0	4	1
A	attach()	0	6	1

<?php


namespace App\Src\UseCases\Domain\Invitation;


use App\Mail\UserJoinsOrganizationToAdmin;
use App\Src\UseCases\Domain\Ports\OrganizationRepository;
use App\Src\UseCases\Domain\Ports\UserRepository;
use App\Src\UseCases\Domain\User;
use Illuminate\Support\Facades\Mail;

class AttachUserToAnOrganization
{
    private $userRepository;
    private $organizationRepository;

    public function __construct(UserRepository $userRepository, OrganizationRepository $organizationRepository)
    {
        $this->userRepository = $userRepository;
        $this->organizationRepository = $organizationRepository;
    }

    public function attach(string $userId, string $organizationId)
    {
        $user = $this->userRepository->getById($userId);
        $user->joinsOrganization($organizationId);

        $this->sendMailUserJoinsToAdmin($organizationId, $user);

It seems like $user can also be of type null; however, parameter $user of App\Src\UseCases\Domain\Invitation\AttachUserToAnOrganization::sendMailUserJoinsToAdmin() does only seem to accept App\Src\UseCases\Domain\User, maybe add an additional type check?

    }

    private function sendMailUserJoinsToAdmin(string $organizationId, User $user): void
    {
        $admins = $this->userRepository->getAdminOfOrganization($organizationId);
        $fullname = $user->fullname();
        $organization = $this->organizationRepository->get($organizationId);
        foreach ($admins as $admin) {
            Mail::to($admin->email())->send(new UserJoinsOrganizationToAdmin($fullname, $organization->name()));
        }
    }
}