EnsureConsistencyPublisher::isRecordExists() - Code Metrics - umbrellio/php-table-sync - Measure and Improve Code Quality continuously with Scrutinizer

EnsureConsistencyPublisher::isRecordExists() A
last analyzed 2023-03-17 08:45 UTC

↳ Parent: EnsureConsistencyPublisher

Complexity

Conditions	2
Paths	2

Size

Total Lines	13
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	2.0078

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	7
c	1
b	0
f	0
dl	0
loc	13
ccs	7
cts	8
cp	0.875
rs	10
cc	2
nc	2
nop	1
crap	2.0078

<?php

declare(strict_types=1);

namespace Umbrellio\TableSync\Integration\Laravel\Publishers;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Support\Facades\App;
use Umbrellio\TableSync\Messages\PublishMessage;
use Umbrellio\TableSync\Publisher;

final class EnsureConsistencyPublisher implements Publisher
{
    private $publisher;

    public function __construct(Publisher $publisher)
    {
        $this->publisher = $publisher;
    }

    public function publish(PublishMessage $message): void
    {
        if (!$this->isRecordExists($message) && !$message->isDestroyed()) {
            return;
        }

        $this->publisher->publish($message);
    }

    private function isRecordExists(PublishMessage $message): bool
    {
        /** @var Model $model */
        $model = App::make($message->className());

        if (!isset($message->attributes()[$model->getKeyName()])) {
            return false;
        }

        return $model

            ->newQuery()
            ->where($model->getKeyName(), $message->attributes()[$model->getKeyName()])
            ->exists();
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Umbrellio\TableSync\Integration\Laravel\Publishers;
6
7		use Illuminate\Database\Eloquent\Model;
8		use Illuminate\Support\Facades\App;
9		use Umbrellio\TableSync\Messages\PublishMessage;
10		use Umbrellio\TableSync\Publisher;
11
12		final class EnsureConsistencyPublisher implements Publisher
13		{
14		private $publisher;
15
16	2	public function __construct(Publisher $publisher)
17		{
18	2	$this->publisher = $publisher;
19		}
20
21	2	public function publish(PublishMessage $message): void
22		{
23	2	if (!$this->isRecordExists($message) && !$message->isDestroyed()) {
24	1	return;
25		}
26
27	1	$this->publisher->publish($message);
28		}
29
30	2	private function isRecordExists(PublishMessage $message): bool
31		{
32		/** @var Model $model */
33	2	$model = App::make($message->className());
34
35	2	if (!isset($message->attributes()[$model->getKeyName()])) {
36		return false;
37		}
38
39	2	return $model
		0 ignored issues – show Bug Best Practice introduced 2020-12-26 21:21 UTC by Report Bug Copy Issue Report The expression `return $model->newQuery(...etKeyName()])->exists()` could return the type `Illuminate\Database\Eloquent\Builder` which is incompatible with the type-hinted return `boolean`. Consider adding an additional type-check to rule them out. Loading history...
40	2	->newQuery()
41	2	->where($model->getKeyName(), $message->attributes()[$model->getKeyName()])
42	2	->exists();
43		}
44		}
45

umbrellio / php-table-sync

EnsureConsistencyPublisher::isRecordExists() A last analyzed 2023-03-17 08:45 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

EnsureConsistencyPublisher::isRecordExists() A
last analyzed 2023-03-17 08:45 UTC