SwitchToNewAndRemoveOldIndex::handle() - Code Metrics - Inspection of "Add progress report" - matchish/laravel-scout-elasticsearch - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#15)

by Serhii

created 2019-05-06 11:46 UTC

SwitchToNewAndRemoveOldIndex::handle() A

↳ Parent: SwitchToNewAndRemoveOldIndex

Complexity

Conditions	3
Paths	3

Size

Total Lines	16
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	3

Importance

Changes

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

<?php

namespace Matchish\ScoutElasticSearch\Jobs\Stages;

use Elasticsearch\Client;
use Laravel\Scout\Searchable;
use Illuminate\Database\Eloquent\Model;
use Matchish\ScoutElasticSearch\ElasticSearch\Index;
use Matchish\ScoutElasticSearch\ElasticSearch\Params\Indices\Alias\Get;
use Matchish\ScoutElasticSearch\ElasticSearch\Params\Indices\Alias\Update;

/**
 * @internal
 */
final class SwitchToNewAndRemoveOldIndex
{
    /**
     * @var Model
     */
    private $searchable;
    /**
     * @var Index
     */
    private $index;

    /**
     * @param Model $searchable
     */
    public function __construct(Model $searchable, Index $index)
    {
        $this->searchable = $searchable;
        $this->index = $index;
    }

    public function handle(Client $elasticsearch): void
    {
        /** @var Searchable $searchable */
        $searchable = $this->searchable;
        $params = Get::anyIndex($searchable->searchableAs());
        $response = $elasticsearch->indices()->getAliases($params->toArray());

        $params = new Update();
        foreach ($response as $indexName => $alias) {
            if ($indexName != $this->index->name()) {
                $params->removeIndex($indexName);
            } else {
                $params->add((string) $indexName, $searchable->searchableAs());
            }
        }
        $elasticsearch->indices()->updateAliases($params->toArray());
    }

    public function estimate(): int
    {
        return 1;
    }

    public function title(): string
    {
        return 'Switching to the new index';
    }
}


1		<?php
2
3		namespace Matchish\ScoutElasticSearch\Jobs\Stages;
4
5		use Elasticsearch\Client;
6		use Laravel\Scout\Searchable;
7		use Illuminate\Database\Eloquent\Model;
8		use Matchish\ScoutElasticSearch\ElasticSearch\Index;
9		use Matchish\ScoutElasticSearch\ElasticSearch\Params\Indices\Alias\Get;
10		use Matchish\ScoutElasticSearch\ElasticSearch\Params\Indices\Alias\Update;
11
12		/**
13		* @internal
14		*/
15		final class SwitchToNewAndRemoveOldIndex
16		{
17		/**
18		* @var Model
19		*/
20		private $searchable;
21		/**
22		* @var Index
23		*/
24		private $index;
25
26		/**
27		* @param Model $searchable
28		*/
29	10	public function __construct(Model $searchable, Index $index)
30		{
31	10	$this->searchable = $searchable;
32	10	$this->index = $index;
33	10	}
34
35	9	public function handle(Client $elasticsearch): void
36		{
37		/** @var Searchable $searchable */
38	9	$searchable = $this->searchable;
39	9	$params = Get::anyIndex($searchable->searchableAs());
40	9	$response = $elasticsearch->indices()->getAliases($params->toArray());
41
42	9	$params = new Update();
43	9	foreach ($response as $indexName => $alias) {
44	9	if ($indexName != $this->index->name()) {
45	2	$params->removeIndex($indexName);
46		} else {
47	9	$params->add((string) $indexName, $searchable->searchableAs());
48		}
49		}
50	9	$elasticsearch->indices()->updateAliases($params->toArray());
51	9	}
52
53	8	public function estimate(): int
54		{
55	8	return 1;
56		}
57
58	8	public function title(): string
59		{
60	8	return 'Switching to the new index';
61		}
62		}
63

matchish / laravel-scout-elasticsearch

Pull Request — master (#15)

SwitchToNewAndRemoveOldIndex::handle() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like