Update::__construct() - Code Metrics - Inspection of "merge v7 branch and fixed conflicts" - oscarotero/simple-crud - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 7c9586...c8ce39 )

by Oscar

created 2019-03-31 11:15 UTC

Update::__construct() A

↳ Parent: Update

Complexity

Conditions	3
Paths	4

Size

Total Lines

Duplication

Lines	18
Ratio	100 %

Importance

Changes

Metric	Value
dl	18
loc	18
rs	9.6666
c	0
b	0
f	0
cc	3
nc	4
nop	2

<?php
declare(strict_types = 1);

namespace SimpleCrud\Query;

use SimpleCrud\Events\CreateUpdateQuery;
use SimpleCrud\Table;

final class Update implements QueryInterface
{
    use Traits\Common;
    use Traits\HasRelatedWith;

    private $allowedMethods = [
        'set',
        'setFlag',
        'where',
        'orWhere',
        'catWhere',
        'orderBy',
        'limit',
        'offset',
    ];

    public function __construct(Table $table, array $data = [])
    {
        $this->table = $table;

        $this->query = $table->getDatabase()
            ->update()
            ->table((string) $table);

        foreach ($data as $fieldName => $value) {
            $this->table->{$fieldName}->update($this->query, $value);
        }

        $eventDispatcher = $table->getEventDispatcher();

        if ($eventDispatcher) {
            $eventDispatcher->dispatch(new CreateUpdateQuery($this));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        }
    }
}


1		<?php
2		declare(strict_types = 1);
3
4		namespace SimpleCrud\Query;
5
6		use SimpleCrud\Events\CreateUpdateQuery;
7		use SimpleCrud\Table;
8
9		final class Update implements QueryInterface
10		{
11		use Traits\Common;
12		use Traits\HasRelatedWith;
13
14		private $allowedMethods = [
15		'set',
16		'setFlag',
17		'where',
18		'orWhere',
19		'catWhere',
20		'orderBy',
21		'limit',
22		'offset',
23		];
24
25	View Code Duplication	public function __construct(Table $table, array $data = [])
26		{
27		$this->table = $table;
28
29		$this->query = $table->getDatabase()
30		->update()
31		->table((string) $table);
32
33		foreach ($data as $fieldName => $value) {
34		$this->table->{$fieldName}->update($this->query, $value);
35		}
36
37		$eventDispatcher = $table->getEventDispatcher();
38
39		if ($eventDispatcher) {
40		$eventDispatcher->dispatch(new CreateUpdateQuery($this));
		0 ignored issues – show Documentation introduced 2019-03-30 00:32 UTC by Report Bug Copy Issue Report `new \SimpleCrud\Events\CreateUpdateQuery($this)` is of type `object<SimpleCrud\Events\CreateUpdateQuery>`, but the function expects a `object<Psr\EventDispatcher\object>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
41		}
42		}
43		}
44

oscarotero / simple-crud

Push — master ( 7c9586...c8ce39 )

Update::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like