Update::update() - Code Metrics - Inspection of "Update .scrutinizer.yml" - FernandoCalmet/php-slim-rest-api - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Push — master ( 948172...201d68 )

by Fernando

created 2022-03-06 07:46 UTC

Update::update() A

↳ Parent: Update

Complexity

Conditions	3
Paths	4

Size

Total Lines	14
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	3

Importance

Changes	3
Bugs	1	Features	1

Metric	Value
cc	3
eloc	7
c	3
b	1
f	1
nc	4
nop	2
dl	0
loc	14
ccs	10
cts	10
cp	1
crap	3
rs	10

<?php

declare(strict_types=1);

namespace App\Service\User;

use App\Entity\User;

final class Update extends Base
{
    /**
     * @param array<string> $input
     */
    public function update(array $input, int $userId): object
    {
        $data = $this->validateUserData($input, $userId);
        /** @var User $user */
        $user = $this->userRepository->update($data);

        if (self::isRedisEnabled() === true) {
            $this->saveInCache($user->getId(), $user->toJson());
        }
        if (self::isLoggerEnabled() === true) {
            $this->loggerService->setInfo('The user with the ID ' . $user->getId() . ' has updated successfully.');
        }

        return $user->toJson();
    }

    /**
     * @param array<string> $input
     */
    private function validateUserData(array $input, int $userId): User
    {
        $user = $this->getUserFromDb($userId);
        $data = json_decode((string) json_encode($input), false);
        if (!isset($data->name) && !isset($data->email)) {
            throw new \App\Exception\UserException('Enter the data to update the user.', 400);
        }
        if (isset($data->name)) {
            $user->updateName(self::validateUserName($data->name));
        }
        if (isset($data->email) && $data->email !== $user->getEmail()) {
            $this->userRepository->checkUserByEmail($data->email);
            $user->updateEmail(self::validateEmail($data->email));
        }
        $user->updateUpdatedAt(date('Y-m-d H:i:s'));

        return $user;
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace App\Service\User;
6
7		use App\Entity\User;
8
9		final class Update extends Base
10		{
11	3	/**
12		* @param array<string> $input
13	3	*/
14	3	public function update(array $input, int $userId): object
15	3	{
16	1	$data = $this->validateUserData($input, $userId);
17		/** @var User $user */
18	2	$user = $this->userRepository->update($data);
19	2
20		if (self::isRedisEnabled() === true) {
21	1	$this->saveInCache($user->getId(), $user->toJson());
22	1	}
23		if (self::isLoggerEnabled() === true) {
24	1	$this->loggerService->setInfo('The user with the ID ' . $user->getId() . ' has updated successfully.');
25	1	}
26
27	1	return $user->toJson();
28		}
29	1
30	1	/**
31	1	* @param array<string> $input
32		*/
33	1	private function validateUserData(array $input, int $userId): User
34	1	{
35		$user = $this->getUserFromDb($userId);
36		$data = json_decode((string) json_encode($input), false);
37	1	if (!isset($data->name) && !isset($data->email)) {
38		throw new \App\Exception\UserException('Enter the data to update the user.', 400);
39		}
40		if (isset($data->name)) {
41		$user->updateName(self::validateUserName($data->name));
42		}
43		if (isset($data->email) && $data->email !== $user->getEmail()) {
44		$this->userRepository->checkUserByEmail($data->email);
45		$user->updateEmail(self::validateEmail($data->email));
46		}
47		$user->updateUpdatedAt(date('Y-m-d H:i:s'));
48
49		return $user;
50		}
51		}
52

FernandoCalmet / php-slim-rest-api

Push — master ( 948172...201d68 )

Update::update() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like