Obblm\Core\Service\Rule\Bb2020Rule

Complexity

Total Complexity

3

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Test Coverage

Coverage

16.66%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getActionsFormClass()	0	3	1
A	setPlayerDefaultValues()	0	15	1
A	getKey()	0	3	1

<?php

namespace Obblm\Core\Service\Rule;

use Obblm\Core\Contracts\PositionInterface;
use Obblm\Core\Contracts\RuleHelperInterface;
use Obblm\Core\Entity\PlayerVersion;
use Obblm\Core\Form\Player\ActionBb2020Type;
use Obblm\Core\Helper\CoreTranslation;
use Obblm\Core\Helper\Rule\AbstractRuleHelper;

class Bb2020Rule extends AbstractRuleHelper implements RuleHelperInterface
{
    public function getActionsFormClass(): string
    {
        return ActionBb2020Type::class;
    }

    public function getKey(): string
    {
        return 'bb2020';
    }

    public function setPlayerDefaultValues(PlayerVersion $version, PositionInterface $position): ?PlayerVersion
    {
        $version->setCharacteristics($position->getCharacteristics())
            ->setActions([
                'td' => 0,
                'cas' => 0,
                'pas' => 0,
                'int' => 0,
                'mvp' => 0,
            ])
            ->setSkills($position->getSkills())
            ->setValue($position->getCost())
            ->setSppLevel($this->getSppLevel($version));

It seems like $this->getSppLevel($version) can also be of type null; however, parameter $sppLevel of Obblm\Core\Entity\PlayerVersion::setSppLevel() does only seem to accept string, maybe add an additional type check?

        return $version;
    }
}