hiqdev\billing\hiapi\target\TargetLoader

Complexity

Total Complexity

5

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	findTarget()	0	7	2
A	execute()	0	7	2

<?php
/**
 * API for Billing
 *
 * @link      https://github.com/hiqdev/billing-hiapi
 * @package   billing-hiapi
 * @license   BSD-3-Clause
 * @copyright Copyright (c) 2017-2018, HiQDev (http://hiqdev.com/)
 */

namespace hiqdev\billing\hiapi\target;

use League\Tactician\Middleware;
use hiqdev\php\billing\target\Target;
use hiqdev\billing\hiapi\target\TargetRepository;

class TargetLoader implements Middleware
{
    private $repo;

    public function __construct(TargetRepository $repo)
    {
        $this->repo = $repo;
    }

    public function execute($command, callable $next)
    {
        if (empty($command->target)) {
            $command->target = $this->findTarget($command);
        }

        return $next($command);
    }

    private function findTarget($command): ?Target
    {
        if (empty($command->target_id)) {
            return null;
        }

        return $this->repo->findById($command->target_id);

The method findById() does not exist on hiqdev\billing\hiapi\target\TargetRepository. Since you implemented __call, consider adding a @method annotation.

    }
}