VehicleMutation::deleteVehicle() - Code Metrics - Inspection of "Merge pull request #1 from Samffy/feature_mutation" - Samffy/graphql-poc - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 2d9e7d...e8a8b9 )

by Samuel

created 2018-06-30 16:56 UTC

VehicleMutation::deleteVehicle() A

↳ Parent: VehicleMutation

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	12
rs	9.8666
c	0
b	0
f	0
cc	2
nc	2
nop	1

<?php

namespace App\Vehicle\App\Mutation;

use App\Common\App\Transformer\AppGlobalId;
use App\Vehicle\Domain\Car;
use App\Vehicle\Domain\VehicleRepositoryInterface;
use GraphQL\Error\UserError;
use Overblog\GraphQLBundle\Definition\Resolver\AliasedInterface;
use Overblog\GraphQLBundle\Definition\Resolver\MutationInterface;

class VehicleMutation implements MutationInterface, AliasedInterface
{
    /**
     * @var VehicleRepositoryInterface
     */
    private $vehicleRepository;

    /**
     * @param VehicleRepositoryInterface $vehicleRepository
     */
    public function __construct(VehicleRepositoryInterface $vehicleRepository)
    {
        $this->vehicleRepository = $vehicleRepository;
    }

    /**
     * @param string $globalId
     * @return string
     * @throws \Doctrine\ORM\NonUniqueResultException
     */
    public function deleteVehicle(string $globalId): string
    {
        $id = AppGlobalId::getIdFromGlobalId($globalId);

        if (!$vehicle = $this->vehicleRepository->find($id)) {
            throw new UserError(sprintf('Vehicle [%s] not found', $globalId));
        }

        $this->vehicleRepository->delete($vehicle);

        return $globalId;
    }

    /**
     * {@inheritdoc}
     */
    public static function getAliases(): array
    {
        return [
            'resolve' => 'VehicleMutation',
        ];
    }
}


1			<?php
2
3			namespace App\Vehicle\App\Mutation;
4
5			use App\Common\App\Transformer\AppGlobalId;
6			use App\Vehicle\Domain\Car;
7			use App\Vehicle\Domain\VehicleRepositoryInterface;
8			use GraphQL\Error\UserError;
9			use Overblog\GraphQLBundle\Definition\Resolver\AliasedInterface;
10			use Overblog\GraphQLBundle\Definition\Resolver\MutationInterface;
11
12			class VehicleMutation implements MutationInterface, AliasedInterface
13			{
14			/**
15			* @var VehicleRepositoryInterface
16			*/
17			private $vehicleRepository;
18
19			/**
20			* @param VehicleRepositoryInterface $vehicleRepository
21			*/
22			public function __construct(VehicleRepositoryInterface $vehicleRepository)
23			{
24			$this->vehicleRepository = $vehicleRepository;
25			}
26
27			/**
28			* @param string $globalId
29			* @return string
30			* @throws \Doctrine\ORM\NonUniqueResultException
31			*/
32			public function deleteVehicle(string $globalId): string
33			{
34			$id = AppGlobalId::getIdFromGlobalId($globalId);
35
36			if (!$vehicle = $this->vehicleRepository->find($id)) {
37			throw new UserError(sprintf('Vehicle [%s] not found', $globalId));
38			}
39
40			$this->vehicleRepository->delete($vehicle);
41
42			return $globalId;
43			}
44
45			/**
46			* {@inheritdoc}
47			*/
48			public static function getAliases(): array
49			{
50			return [
51			'resolve' => 'VehicleMutation',
52			];
53			}
54			}
55

Samffy / graphql-poc

Push — master ( 2d9e7d...e8a8b9 )

VehicleMutation::deleteVehicle() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like