ModelSave::save() - Code Metrics - Inspection of "Updating docblocks" - flipbox/spark - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( e046f3...04364f )

by Nate

created 2017-03-28 04:40 UTC

ModelSave::save() C

↳ Parent: ModelSave

Complexity

Conditions	9
Paths	21

Size

Total Lines	83
Code Lines	33

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	83
rs	5.48
c	0
b	0
f	0
cc	9
eloc	33
nc	21
nop	4

How to fix Long Method

<?php

/**
 * @copyright  Copyright (c) Flipbox Digital Limited
 * @license    https://github.com/flipbox/spark/blob/master/LICENSE
 * @link       https://github.com/flipbox/spark
 */

namespace flipbox\spark\services\traits;

use Craft;
use craft\events\ModelEvent;
use flipbox\spark\helpers\RecordHelper;
use flipbox\spark\models\Model;
use flipbox\spark\records\Record;

/**
 * @package flipbox\spark\services\traits
 * @author Flipbox Factory <[email protected]>
 * @since 1.0.0
 */
trait ModelSave
{

    /*******************************************
     * ABSTRACTS
     *******************************************/

    /**
     * @param Model $model
     * @param bool $mirrorScenario
     * @return Record
     */
    abstract public function toRecord(Model $model, bool $mirrorScenario = true): Record;


    /*******************************************
     * SAVE
     *******************************************/

    /**
     * @param Model $model
     * @param bool $runValidation
     * @param null $attributes
     * @param bool $mirrorScenario
     * @return bool
     * @throws \Exception
     */
    public function save(Model $model, bool $runValidation = true, $attributes = null, bool $mirrorScenario = true)
    {

        // Validate
        if ($runValidation && !$model->validate($attributes)) {
            Craft::info('Model not saved due to validation error.', __METHOD__);
            return false;
        }

        // Create event
        $event = new ModelEvent([
            'isNew' => null !== $model->id
        ]);

        // Db transaction
        $transaction = RecordHelper::beginTransaction();

        try {

            // The 'before' event
            if (!$model->beforeSave($event)) {

                $transaction->rollBack();

                return false;
            }

            $record = $this->toRecord($model, $mirrorScenario);

            // Validate
            if (!$record->validate($attributes)) {

                $model->addErrors($record->getErrors());

                $transaction->rollBack();

                return false;

            }

            // Insert record
            if (!$record->save($attributes)) {
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

                // Transfer errors to model
                $model->addErrors($record->getErrors());

                $transaction->rollBack();

                return false;

            }

            // Transfer record to model
            if ($event->isNew) {
                $model->id = $record->id;
                $model->dateCreated = $record->dateCreated;
                $model->uid = $record->uid;
            }
            $model->dateUpdated = $record->dateUpdated;


            // The 'after' event
            if (!$model->afterSave($event)) {

                $transaction->rollBack();

                return false;

            }

        } catch (\Exception $e) {

            $transaction->rollBack();

            throw $e;

        }

        $transaction->commit();

        return true;

    }

}


1			<?php
2
3			/**
4			* @copyright Copyright (c) Flipbox Digital Limited
5			* @license https://github.com/flipbox/spark/blob/master/LICENSE
6			* @link https://github.com/flipbox/spark
7			*/
8
9			namespace flipbox\spark\services\traits;
10
11			use Craft;
12			use craft\events\ModelEvent;
13			use flipbox\spark\helpers\RecordHelper;
14			use flipbox\spark\models\Model;
15			use flipbox\spark\records\Record;
16
17			/**
18			* @package flipbox\spark\services\traits
19			* @author Flipbox Factory <[email protected]>
20			* @since 1.0.0
21			*/
22			trait ModelSave
23			{
24
25			/*******************************************
26			* ABSTRACTS
27			*******************************************/
28
29			/**
30			* @param Model $model
31			* @param bool $mirrorScenario
32			* @return Record
33			*/
34			abstract public function toRecord(Model $model, bool $mirrorScenario = true): Record;
35
36
37			/*******************************************
38			* SAVE
39			*******************************************/
40
41			/**
42			* @param Model $model
43			* @param bool $runValidation
44			* @param null $attributes
45			* @param bool $mirrorScenario
46			* @return bool
47			* @throws \Exception
48			*/
49			public function save(Model $model, bool $runValidation = true, $attributes = null, bool $mirrorScenario = true)
50			{
51
52			// Validate
53			if ($runValidation && !$model->validate($attributes)) {
54			Craft::info('Model not saved due to validation error.', __METHOD__);
55			return false;
56			}
57
58			// Create event
59			$event = new ModelEvent([
60			'isNew' => null !== $model->id
61			]);
62
63			// Db transaction
64			$transaction = RecordHelper::beginTransaction();
65
66			try {
67
68			// The 'before' event
69			if (!$model->beforeSave($event)) {
70
71			$transaction->rollBack();
72
73			return false;
74			}
75
76			$record = $this->toRecord($model, $mirrorScenario);
77
78			// Validate
79			if (!$record->validate($attributes)) {
80
81			$model->addErrors($record->getErrors());
82
83			$transaction->rollBack();
84
85			return false;
86
87			}
88
89			// Insert record
90			if (!$record->save($attributes)) {
			0 ignored issues – show Documentation introduced 2017-03-27 23:36 UTC by Report Bug Copy Issue Report `$attributes` is of type `null`, but the function expects a `boolean`. 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...
91
92			// Transfer errors to model
93			$model->addErrors($record->getErrors());
94
95			$transaction->rollBack();
96
97			return false;
98
99			}
100
101			// Transfer record to model
102			if ($event->isNew) {
103			$model->id = $record->id;
104			$model->dateCreated = $record->dateCreated;
105			$model->uid = $record->uid;
106			}
107			$model->dateUpdated = $record->dateUpdated;
108
109
110			// The 'after' event
111			if (!$model->afterSave($event)) {
112
113			$transaction->rollBack();
114
115			return false;
116
117			}
118
119			} catch (\Exception $e) {
120
121			$transaction->rollBack();
122
123			throw $e;
124
125			}
126
127			$transaction->commit();
128
129			return true;
130
131			}
132
133			}
134

flipbox / spark

Push — master ( e046f3...04364f )

ModelSave::save() C

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like