ProfileUpdate::run() - Code Metrics - Inspection of "PHP 8.1 minimum required (#6)" - ROAResearch/yii2-roa - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( c2b947...93db86 )

by Carlos

created 2022-11-02 01:28 UTC

ProfileUpdate::run() A

↳ Parent: ProfileUpdate

Complexity

Conditions	3
Paths	2

Size

Total Lines	16
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	3.0175

Importance

Changes

Metric	Value
cc	3
eloc	9
nc	2
nop	0
dl	0
loc	16
ccs	7
cts	8
cp	0.875
crap	3.0175
rs	9.9666
c	0
b	0
f	0

<?php

namespace roaresearch\yii2\roa\actions;

use Yii;
use yii\{base\Model, db\ActiveRecordInterface, web\ServerErrorHttpException};

class ProfileUpdate extends \yii\rest\Action
{
    use LoadFileTrait;

    /**
     * @var string the scenario to be assigned to the model before it is validated and updated.
     */
    public string $scenario = Model::SCENARIO_DEFAULT;

    /**
     * @var string[] that defines which columns will be recibe files
     */
    public array $fileAttributes = [];

    /**
     * @inheritdoc
     */
    public function init()
    {
    }

    /**
     * Updates the information of the logged user.
     *
     * @return ActiveRecordInterface
     * @throws ServerErrorHttpException if there is any error when updating the model
     */
    public function run(): ActiveRecordInterface
    {
        /* @var $model ActiveRecordInterface */
        $model = Yii::$app->user->identity;
        $model->scenario = $this->scenario;

        $model->load(
            Yii::$app->getRequest()->getBodyParams() + $this->parseFileAttributes(),
            ''
        );
        if ($model->save() === false && !$model->hasErrors()) {
            throw new ServerErrorHttpException(
                'Failed to update the object for unknown reason.'
            );
        }

        return $model;
    }
}


1		<?php
2
3		namespace roaresearch\yii2\roa\actions;
4
5		use Yii;
6		use yii\{base\Model, db\ActiveRecordInterface, web\ServerErrorHttpException};
7
8		class ProfileUpdate extends \yii\rest\Action
9		{
10		use LoadFileTrait;
11
12		/**
13		* @var string the scenario to be assigned to the model before it is validated and updated.
14		*/
15		public string $scenario = Model::SCENARIO_DEFAULT;
16
17		/**
18		* @var string[] that defines which columns will be recibe files
19		*/
20		public array $fileAttributes = [];
21
22		/**
23		* @inheritdoc
24		*/
25		public function init()
26		{
27		}
28
29		/**
30		* Updates the information of the logged user.
31		*
32		* @return ActiveRecordInterface
33		* @throws ServerErrorHttpException if there is any error when updating the model
34		*/
35	1	public function run(): ActiveRecordInterface
36		{
37		/* @var $model ActiveRecordInterface */
38	1	$model = Yii::$app->user->identity;
39	1	$model->scenario = $this->scenario;
		0 ignored issues – show Bug introduced 2019-11-29 16:37 UTC by Report Bug Copy Issue Report Accessing `scenario` on the interface `yii\db\ActiveRecordInterface` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history...
40	1	$model->load(
41	1	Yii::$app->getRequest()->getBodyParams() + $this->parseFileAttributes(),
42		''
43		);
44	1	if ($model->save() === false && !$model->hasErrors()) {
45		throw new ServerErrorHttpException(
46		'Failed to update the object for unknown reason.'
47		);
48		}
49
50	1	return $model;
51		}
52		}
53

ROAResearch / yii2-roa

Push — master ( c2b947...93db86 )

ProfileUpdate::run() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like