DeleteUserAction::process() - Code Metrics - Inspection of "remove excessive exception handling;" - SlayerBirden/dataflow-server - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 3bc7c8...a199f7 )

by Oleg

created 2018-09-07 21:52 UTC

DeleteUserAction::process() A

↳ Parent: DeleteUserAction

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	8
ccs	6
cts	6
cp	1
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	2
crap	1

<?php
declare(strict_types=1);

namespace SlayerBirden\DataFlowServer\Domain\Controller;

use Doctrine\ORM\ORMException;
use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\ServerRequestInterface;
use Psr\Http\Server\MiddlewareInterface;
use Psr\Http\Server\RequestHandlerInterface;
use Psr\Log\LoggerInterface;
use SlayerBirden\DataFlowServer\Doctrine\Middleware\ResourceMiddlewareInterface;
use SlayerBirden\DataFlowServer\Doctrine\Persistence\EntityManagerRegistry;
use SlayerBirden\DataFlowServer\Domain\Entities\User;
use SlayerBirden\DataFlowServer\Stdlib\Validation\GeneralSuccessResponseFactory;
use Zend\Hydrator\HydratorInterface;

final class DeleteUserAction implements MiddlewareInterface
{
    /**
     * @var LoggerInterface
     */
    private $logger;
    /**
     * @var HydratorInterface
     */
    private $hydrator;
    /**
     * @var EntityManagerRegistry
     */
    private $managerRegistry;

    public function __construct(
        EntityManagerRegistry $managerRegistry,
        LoggerInterface $logger,
        HydratorInterface $hydrator
    ) {
        $this->managerRegistry = $managerRegistry;
        $this->logger = $logger;
        $this->hydrator = $hydrator;
    }

    /**
     * @inheritdoc
     * @throws ORMException
     */
    public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
    {
        $user = $request->getAttribute(ResourceMiddlewareInterface::DATA_RESOURCE);
        $em = $this->managerRegistry->getManagerForClass(User::class);
        $em->remove($user);
        $em->flush();
        return (new GeneralSuccessResponseFactory())('User was deleted', 'user', $this->hydrator->extract($user));
    }
}


1		<?php
2		declare(strict_types=1);
3
4		namespace SlayerBirden\DataFlowServer\Domain\Controller;
5
6		use Doctrine\ORM\ORMException;
7		use Psr\Http\Message\ResponseInterface;
8		use Psr\Http\Message\ServerRequestInterface;
9		use Psr\Http\Server\MiddlewareInterface;
10		use Psr\Http\Server\RequestHandlerInterface;
11		use Psr\Log\LoggerInterface;
12		use SlayerBirden\DataFlowServer\Doctrine\Middleware\ResourceMiddlewareInterface;
13		use SlayerBirden\DataFlowServer\Doctrine\Persistence\EntityManagerRegistry;
14		use SlayerBirden\DataFlowServer\Domain\Entities\User;
15		use SlayerBirden\DataFlowServer\Stdlib\Validation\GeneralSuccessResponseFactory;
16		use Zend\Hydrator\HydratorInterface;
17
18		final class DeleteUserAction implements MiddlewareInterface
19		{
20		/**
21		* @var LoggerInterface
22		*/
23		private $logger;
24		/**
25		* @var HydratorInterface
26		*/
27		private $hydrator;
28		/**
29		* @var EntityManagerRegistry
30		*/
31		private $managerRegistry;
32
33	2	public function __construct(
34		EntityManagerRegistry $managerRegistry,
35		LoggerInterface $logger,
36		HydratorInterface $hydrator
37		) {
38	2	$this->managerRegistry = $managerRegistry;
39	2	$this->logger = $logger;
40	2	$this->hydrator = $hydrator;
41	2	}
42
43		/**
44		* @inheritdoc
45		* @throws ORMException
46		*/
47	2	public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
48		{
49	2	$user = $request->getAttribute(ResourceMiddlewareInterface::DATA_RESOURCE);
50	2	$em = $this->managerRegistry->getManagerForClass(User::class);
51	2	$em->remove($user);
52	2	$em->flush();
53	2	return (new GeneralSuccessResponseFactory())('User was deleted', 'user', $this->hydrator->extract($user));
54		}
55		}
56

SlayerBirden / dataflow-server

Push — master ( 3bc7c8...a199f7 )

DeleteUserAction::process() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like