Insert::__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

Insert::__construct() A

↳ Parent: Insert

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\CreateInsertQuery;
use SimpleCrud\Table;

final class Insert implements QueryInterface
{
    use Traits\Common;

    private $allowedMethods = [
        'setFlag',
        'set',
    ];

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

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

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

        $eventDispatcher = $table->getEventDispatcher();

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

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

// Instead of
acceptsInteger($x);

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

    public function run()
    {
        $this->__invoke();

        $id = $this->table->getDatabase()->lastInsertId();

        return $this->table->id->format($id);
    }
}


1		<?php
2		declare(strict_types = 1);
3
4		namespace SimpleCrud\Query;
5
6		use SimpleCrud\Events\CreateInsertQuery;
7		use SimpleCrud\Table;
8
9		final class Insert implements QueryInterface
10		{
11		use Traits\Common;
12
13		private $allowedMethods = [
14		'setFlag',
15		'set',
16		];
17
18	View Code Duplication	public function __construct(Table $table, array $data = [])
19		{
20		$this->table = $table;
21
22		$this->query = $table->getDatabase()
23		->insert()
24		->into((string) $table);
25
26		foreach ($data as $fieldName => $value) {
27		$this->table->{$fieldName}->insert($this->query, $value);
28		}
29
30		$eventDispatcher = $table->getEventDispatcher();
31
32		if ($eventDispatcher) {
33		$eventDispatcher->dispatch(new CreateInsertQuery($this));
		0 ignored issues – show Documentation introduced 2019-03-30 00:32 UTC by Report Bug Copy Issue Report `new \SimpleCrud\Events\CreateInsertQuery($this)` is of type `object<SimpleCrud\Events\CreateInsertQuery>`, 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...
34		}
35		}
36
37		public function run()
38		{
39		$this->__invoke();
40
41		$id = $this->table->getDatabase()->lastInsertId();
42
43		return $this->table->id->format($id);
44		}
45		}
46

oscarotero / simple-crud

Push — master ( 7c9586...c8ce39 )

Insert::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like