DocumentCleaner::__construct() - Code Metrics - Inspection of "+ Documents are deleteable" - xervice/elasticsearch - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 08b37c...1ffe05 )

by Mike

created 2019-04-12 18:24 UTC

DocumentCleaner::__construct() A

↳ Parent: DocumentCleaner

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

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

<?php
declare(strict_types=1);

namespace Xervice\Elasticsearch\Business\Model\Document;


use DataProvider\DocumentListDataProvider;
use Elastica\Client;
use Elastica\Document;

class DocumentCleaner implements DocumentCleanerInterface
{
    /**
     * @var \Elastica\Client
     */
    private $client;

    /**
     * DocumentBuilder constructor.
     *
     * @param \Elastica\Client $client
     */
    public function __construct(Client $client)
    {
        $this->client = $client;
    }

    /**
     * @param \DataProvider\DocumentListDataProvider $listDataProvider
     */
    public function deleteDocuments(DocumentListDataProvider $listDataProvider): void
    {
        $documents = $this->getDocumentsFromDataProvider($listDataProvider);

        $type = $this
            ->client
            ->getIndex($listDataProvider->getIndex())
            ->getType($listDataProvider->getType());

        $type->deleteDocuments($documents);

        $type->getIndex()->refresh();
    }

    /**
     * @param \DataProvider\DocumentListDataProvider $listDataProvider
     * @param array $document
     *
     * @return array
     */
    protected function getDocumentsFromDataProvider(DocumentListDataProvider $listDataProvider): array
    {
        $documents = [];

        foreach ($listDataProvider->getDocuments() as $documentDataProvider) {
            $documents[] = new Document(
                $documentDataProvider->getIdent(),
                $documentDataProvider->getContent()
            );
        }

        return $documents;
    }
}

1		<?php
2		declare(strict_types=1);
3
4		namespace Xervice\Elasticsearch\Business\Model\Document;
5
6
7		use DataProvider\DocumentListDataProvider;
8		use Elastica\Client;
9		use Elastica\Document;
10
11		class DocumentCleaner implements DocumentCleanerInterface
12		{
13		/**
14		* @var \Elastica\Client
15		*/
16		private $client;
17
18		/**
19		* DocumentBuilder constructor.
20		*
21		* @param \Elastica\Client $client
22		*/
23	1	public function __construct(Client $client)
24		{
25	1	$this->client = $client;
26	1	}
27
28		/**
29		* @param \DataProvider\DocumentListDataProvider $listDataProvider
30		*/
31	1	public function deleteDocuments(DocumentListDataProvider $listDataProvider): void
32		{
33	1	$documents = $this->getDocumentsFromDataProvider($listDataProvider);
34
35		$type = $this
36	1	->client
37	1	->getIndex($listDataProvider->getIndex())
38	1	->getType($listDataProvider->getType());
39
40	1	$type->deleteDocuments($documents);
41
42	1	$type->getIndex()->refresh();
43	1	}
44
45		/**
46		* @param \DataProvider\DocumentListDataProvider $listDataProvider
47		* @param array $document
48		*
49		* @return array
50		*/
51	1	protected function getDocumentsFromDataProvider(DocumentListDataProvider $listDataProvider): array
52		{
53	1	$documents = [];
54
55	1	foreach ($listDataProvider->getDocuments() as $documentDataProvider) {
56	1	$documents[] = new Document(
57	1	$documentDataProvider->getIdent(),
58	1	$documentDataProvider->getContent()
59		);
60		}
61
62	1	return $documents;
63		}
64		}

xervice / elasticsearch

Push — master ( 08b37c...1ffe05 )

DocumentCleaner::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like