App\User\Console\AssignRoleCommand::execute()

Complexity

Conditions	6
Paths	17

Size

Total Lines	38
Code Lines	23

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	42

Importance

Changes

0

<?php
    protected function execute(InputInterface $input, OutputInterface $output): int
    {
        $io = new SymfonyStyle($input, $output);

        $roleName = $input->getArgument('role');
        $userId = $input->getArgument('userId');

        try {
            $orm = $this->promise->getORM();
            $userRepo = $orm->getRepository(User::class);
            $user = $userRepo->findByPK($userId);
            if (null === $user) {
                throw new \Exception('Can\'t find user');
            }

            $role = $this->storage->getRoleByName($roleName);

It seems like $roleName can also be of type null and string[]; however, parameter $name of Yiisoft\Rbac\StorageInterface::getRoleByName() does only seem to accept string, maybe add an additional type check?

            if (null === $role) {
                $helper = $this->getHelper('question');
                $question = new ConfirmationQuestion('Role doesn\'t exist. Create new one? ', false);

                if (!$helper->ask($input, $output, $question)) {
                    return ExitCode::OK;
                }

                $role = new Role($roleName);

It seems like $roleName can also be of type null and string[]; however, parameter $name of Yiisoft\Rbac\Role::__construct() does only seem to accept string, maybe add an additional type check?

                $this->manager->addRole($role);
            }

            $this->manager->assign($role, $user->getId());

            $io->success('Role was assigned to given user');
        } catch (\Throwable $t) {
            $io->error($t->getMessage());
            return $t->getCode() ?: ExitCode::UNSPECIFIED_ERROR;
        }

        return ExitCode::OK;
    }