Zikula\UsersModule\Command\DeleteCommand::execute()

Complexity

Conditions	18
Paths	197

Size

Total Lines	80
Code Lines	51

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    protected function execute(InputInterface $input, OutputInterface $output): int
    {
        $io = new SymfonyStyle($input, $output);
        $uid = $input->getOption('uid');
        $gid = $input->getOption('gid');
        $status = $input->getOption('status');
        $date = $input->getOption('date');

        if (($uid && $gid) || ($uid && $status) || ($gid && $status)) {
            $io->error('Do not use more than one option or argument.');

            return 1;
        }

        if (isset($gid)) {
            if (Constant::GROUP_ID_USERS === (int) $gid) {
                if (!$io->confirm('You have selected to delete from the main user group. This is not recommended. Do you wish to proceed?', false)) {
                    $io->caution('Deletion cancelled');

                    return 0;
                }
            }
            $users = $this->groupRespository->find($gid)->getUsers();
        }

        if (isset($status)) {
            $statuses = [
                'I' => UsersConstant::ACTIVATED_INACTIVE,
                'P' => UsersConstant::ACTIVATED_PENDING_REG,
                'M' => UsersConstant::ACTIVATED_PENDING_DELETE
            ];
            if (!array_key_exists($status, $statuses)) {
                $io->error('Invalid status value');

                return 2;
            }
            $users = $this->userRepository->findBy(['activated' => $statuses[$status]]);
            $users = new ArrayCollection($users);
        }

        if (isset($uid)) {
            $user = $this->userRepository->find($uid);
            $users = new ArrayCollection([$user]);
        }

        if (isset($date)) {
            $date = \DateTime::createFromFormat('YmdHis', $date, new \DateTimeZone('UTC'));

It seems like $date can also be of type string[]; however, parameter $time of DateTime::createFromFormat() does only seem to accept string, maybe add an additional type check?

            $users = $users->filter(function (UserEntity $user) use ($date) {

The variable $users does not seem to be defined for all execution paths leading up to this point.

                return $user->getRegistrationDate() < $date;
            });
        }

        $adminUser = $this->userRepository->find(UsersConstant::USER_ID_ADMIN);
        if ($users->contains($adminUser)) {
            $users->removeElement($adminUser);
            $io->note(sprintf('The main admin user cannot be deleted (uname: %s)', $adminUser->getUname()));
        }

        if ($users->isEmpty()) {
            $io->error('No users found!');

            return 3;
        }

        $io->title('Zikula user deletion');
        $count = count($users);
        $io->progressStart($count);
        foreach ($users as $user) {
            $eventName = UsersConstant::ACTIVATED_ACTIVE === $user->getActivated() ? UserEvents::DELETE_ACCOUNT : RegistrationEvents::DELETE_REGISTRATION;
            $this->eventDispatcher->dispatch(new GenericEvent($user->getUid()), $eventName);
            $this->eventDispatcher->dispatch(new GenericEvent(null, ['id' => $user->getUid()]), UserEvents::DELETE_PROCESS);
            $this->hookDispatcher->dispatch(UserManagementUiHooksSubscriber::DELETE_PROCESS, new ProcessHook($user->getUid()));
            $this->userRepository->removeAndFlush($user);
            $io->progressAdvance();
        }
        $io->progressFinish();

        $io->success(sprintf('Success! %d users deleted.', $count));

        return 0;
    }