DeletePersistentSubscriptionRequestFactory::__construct() - Code Metrics - Inspection of "[WIP] Add competing consumers support." - rayrutjes/php-geteventstore-client - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#1)

by Raymond

created 2016-03-07 15:48 UTC

__construct() A

↳ Parent: DeletePersistentSubscriptionRequestFactory

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	3

Duplication

Lines	5
Ratio	100 %

Metric	Value
dl	5
loc	5
rs	9.4285
cc	1
eloc	3
nc	1
nop	2

<?php

namespace RayRutjes\GetEventStore\Client\Http;

use GuzzleHttp\Psr7\Request;
use Psr\Http\Message\RequestInterface;
use RayRutjes\GetEventStore\StreamId;

final class DeletePersistentSubscriptionRequestFactory implements RequestFactoryInterface

{
    /**
     * @var StreamId
     */
    private $streamId;

    /**
     * @var string
     */
    private $groupName;

    /**
     * @param StreamId $streamId
     * @param string   $groupName
     */
    public function __construct(StreamId $streamId, string $groupName)
    {
        $this->streamId = $streamId;
        $this->groupName = $groupName;
    }

    /**
     * @return RequestInterface
     */
    public function buildRequest(): RequestInterface
    {
        return new Request(
            'DELETE',
            sprintf('subscriptions/%s/%s', $this->streamId->toString(), $this->groupName),
            [
                RequestHeader::CONTENT_TYPE => ContentType::JSON,
            ]
        );
    }
}


1		<?php
2
3		namespace RayRutjes\GetEventStore\Client\Http;
4
5		use GuzzleHttp\Psr7\Request;
6		use Psr\Http\Message\RequestInterface;
7		use RayRutjes\GetEventStore\StreamId;
8
9	View Code Duplication	final class DeletePersistentSubscriptionRequestFactory implements RequestFactoryInterface
		0 ignored issues – show Duplication introduced 2016-03-07 15:51 UTC by Report Bug Copy Issue Report This class seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
10		{
11		/**
12		* @var StreamId
13		*/
14		private $streamId;
15
16		/**
17		* @var string
18		*/
19		private $groupName;
20
21		/**
22		* @param StreamId $streamId
23		* @param string $groupName
24		*/
25		public function __construct(StreamId $streamId, string $groupName)
26		{
27		$this->streamId = $streamId;
28		$this->groupName = $groupName;
29		}
30
31		/**
32		* @return RequestInterface
33		*/
34		public function buildRequest(): RequestInterface
35		{
36		return new Request(
37		'DELETE',
38		sprintf('subscriptions/%s/%s', $this->streamId->toString(), $this->groupName),
39		[
40		RequestHeader::CONTENT_TYPE => ContentType::JSON,
41		]
42		);
43		}
44		}
45

rayrutjes / php-geteventstore-client

Pull Request — master (#1)

__construct() A

Complexity

Size

Duplication

Duplication Side-by-Side

Filter issues like