SaleCloseAction::__invoke() - Code Metrics - Inspection of "Added `SaleClose`" - hiqdev/billing-hiapi - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( ebb653...69cfe7 )

by Andrii

created 2020-04-02 12:45 UTC

SaleCloseAction::__invoke() A

↳ Parent: SaleCloseAction

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	5
c	1
b	0
f	0
dl	0
loc	8
rs	10
cc	1
nc	1
nop	1

<?php

namespace hiqdev\billing\hiapi\sale\Close;

use hiqdev\billing\hiapi\sale\SaleRepository;
use hiqdev\php\billing\plan\Plan;
use hiqdev\php\billing\sale\Sale;
use hiapi\exceptions\domain\RequiredInputException;

class SaleCloseAction
{
    /**
     * @var SaleRepository
     */
    private $repo;

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

    public function __invoke(SaleCloseCommand $command): Sale
    {
        $this->checkRequiredInput($command);
        $plan = new Plan($command->plan_id, null);
        $sale = new Sale(null, $command->target, $command->customer, $plan, $command->time);
        $this->repo->delete($sale);

        return $sale;
    }

    protected function checkRequiredInput(SaleCloseCommand $command)
    {
        if (empty($command->customer)) {
            throw new RequiredInputException('customer');
        }
        if (empty($command->target)) {
            throw new RequiredInputException('target');
        }
    }
}


1			<?php
2
3			namespace hiqdev\billing\hiapi\sale\Close;
4
5			use hiqdev\billing\hiapi\sale\SaleRepository;
6			use hiqdev\php\billing\plan\Plan;
7			use hiqdev\php\billing\sale\Sale;
8			use hiapi\exceptions\domain\RequiredInputException;
9
10			class SaleCloseAction
11			{
12			/**
13			* @var SaleRepository
14			*/
15			private $repo;
16
17			public function __construct(SaleRepository $repo)
18			{
19			$this->repo = $repo;
20			}
21
22			public function __invoke(SaleCloseCommand $command): Sale
23			{
24			$this->checkRequiredInput($command);
25			$plan = new Plan($command->plan_id, null);
26			$sale = new Sale(null, $command->target, $command->customer, $plan, $command->time);
27			$this->repo->delete($sale);
28
29			return $sale;
30			}
31
32			protected function checkRequiredInput(SaleCloseCommand $command)
33			{
34			if (empty($command->customer)) {
35			throw new RequiredInputException('customer');
36			}
37			if (empty($command->target)) {
38			throw new RequiredInputException('target');
39			}
40			}
41			}
42

hiqdev / billing-hiapi

Push — master ( ebb653...69cfe7 )

SaleCloseAction::__invoke() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like