Record::save() - Code Metrics - Inspection of "relations, commands, relation bucket" - spiral/components - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — feature/pre-split (7f7f80)

by Anton

created 2017-01-13 14:28 UTC

Record::save() B

↳ Parent: Record

Complexity

Conditions	4
Paths	6

Size

Total Lines	25
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
eloc	12
nc	6
nop	2
dl	0
loc	25
rs	8.5806
c	0
b	0
f	0

<?php
/**
 * components
 *
 * @author    Wolfy-J
 */
namespace Spiral\ORM;

use Spiral\Models\ActiveEntityInterface;
use Spiral\ORM\Commands\InsertCommand;
use Spiral\ORM\Commands\UpdateCommand;

/**
 * Adds ActiveRecord abilities to RecordEntity.
 */
abstract class Record extends RecordEntity implements ActiveEntityInterface
{
    public function save(TransactionInterface $transaction = null, bool $queueRelations = true): int
    {
        //Initial reacord command
        $command = $this->queueSave(false);

        if ($command instanceof InsertCommand) {
            $state = self::CREATED;
        } elseif ($command instanceof UpdateCommand) {
            $state = self::UPDATED;
        } else {
            $state = self::UNCHANGED;
        }

        if ($queueRelations) {
            //Mounting relation related updates
            $command = $this->relations->queueRelations($command);
        }

        //todo: saturate command

        //Registering command
        $transaction->addCommand($command);
function someFunction(A $objectMaybe = null)
{
    if ($objectMaybe instanceof A) {
        $objectMaybe->doSomething();
    }
}

        return $state;
    }

    public function delete(TransactionInterface $transaction = null)
    {
        //todo: saturate command

        //saturate transaction
        $transaction->addCommand($this->queueDelete());
function someFunction(A $objectMaybe = null)
{
    if ($objectMaybe instanceof A) {
        $objectMaybe->doSomething();
    }
}
    }
}

1			<?php
2			/**
3			* components
4			*
5			* @author Wolfy-J
6			*/
7			namespace Spiral\ORM;
8
9			use Spiral\Models\ActiveEntityInterface;
10			use Spiral\ORM\Commands\InsertCommand;
11			use Spiral\ORM\Commands\UpdateCommand;
12
13			/**
14			* Adds ActiveRecord abilities to RecordEntity.
15			*/
16			abstract class Record extends RecordEntity implements ActiveEntityInterface
17			{
18			public function save(TransactionInterface $transaction = null, bool $queueRelations = true): int
19			{
20			//Initial reacord command
21			$command = $this->queueSave(false);
22
23			if ($command instanceof InsertCommand) {
24			$state = self::CREATED;
25			} elseif ($command instanceof UpdateCommand) {
26			$state = self::UPDATED;
27			} else {
28			$state = self::UNCHANGED;
29			}
30
31			if ($queueRelations) {
32			//Mounting relation related updates
33			$command = $this->relations->queueRelations($command);
34			}
35
36			//todo: saturate command
37
38			//Registering command
39			$transaction->addCommand($command);
			0 ignored issues – show Bug introduced 2017-01-13 14:32 UTC by Report Bug Copy Issue Report It seems like `$transaction` is not always an object, but can also be of type `null`. Maybe add an additional type check? If a variable is not always an object, we recommend to add an additional type check to ensure your method call is safe: function someFunction(A $objectMaybe = null) { if ($objectMaybe instanceof A) { $objectMaybe->doSomething(); } } Loading history...
40
41			return $state;
42			}
43
44			public function delete(TransactionInterface $transaction = null)
45			{
46			//todo: saturate command
47
48			//saturate transaction
49			$transaction->addCommand($this->queueDelete());
			0 ignored issues – show Bug introduced 2017-01-12 22:15 UTC by Report Bug Copy Issue Report It seems like `$transaction` is not always an object, but can also be of type `null`. Maybe add an additional type check? If a variable is not always an object, we recommend to add an additional type check to ensure your method call is safe: function someFunction(A $objectMaybe = null) { if ($objectMaybe instanceof A) { $objectMaybe->doSomething(); } } Loading history...
50			}
51			}

spiral / components

Branch — feature/pre-split (7f7f80)

Record::save() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like