Application\Repository\AccountRepository::getOrCreate()

Complexity

Conditions	3
Paths	3

Size

Total Lines	22
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	15
CRAP Score	3.0021

Importance

Changes

0

<?php
    public function getOrCreate(User $user): Account
    {
        $account = $this->findOneByOwner($user);

The method findOneByOwner() does not exist on Application\Repository\AccountRepository. Since you implemented __call, consider adding a @method annotation.

        if (!$account) {
            $account = new Account();
            $this->getEntityManager()->persist($account);
            $account->setOwner($user);
            $account->setType(AccountTypeType::LIABILITY);
            $account->setName($user->getName());

            $maxCode = $this->getEntityManager()->getConnection()->fetchColumn('SELECT MAX(code) FROM account WHERE parent_id = ' . self::PARENT_ACCOUNT_ID_FOR_USER);
            $newCode = ++$maxCode;
            $account->setCode((string) $newCode);

            $parent = $this->findOneById(self::PARENT_ACCOUNT_ID_FOR_USER);
            if (!$parent) {
                throw new \Exception('Cannot find parent account for creation of user account');
            }
            $account->setParent($parent);
        }

        return $account;
    }