UserService::createUser() - Code Metrics - Inspection of "Update" - ControleOnline/api-platform-users - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 34e91e...8ddd25 )

by Luiz Kim

created 2024-09-06 13:32 UTC

UserService::createUser() A

↳ Parent: UserService

Complexity

Conditions	2
Paths	2

Size

Total Lines	13
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	9
c	1
b	0
f	0
dl	0
loc	13
rs	9.9666
cc	2
nc	2
nop	3

<?php

namespace ControleOnline\Service;

use ControleOnline\Entity\People;
filter:
    dependency_paths: ["lib/*"]
use ControleOnline\Entity\User;
use Doctrine\ORM\EntityManagerInterface;
filter:
    dependency_paths: ["lib/*"]
use Exception;
use Symfony\Component\Security\Core\Encoder\UserPasswordEncoderInterface;
filter:
    dependency_paths: ["lib/*"]

class UserService
{
  public function __construct(private EntityManagerInterface $manager, private  UserPasswordEncoderInterface $encoder) {}
  public function changePassword(User $user, $password)
  {
    if (!$this->getPermission())
      throw new Exception("You should not pass!!!", 301);

    $user->setHash(
      $this->encoder->encodePassword($user, $password)
    );

    $this->manager->persist($user);
    $this->manager->flush();
    return $user;
  }

  public function changeApiKey(User $user)
  {
    if (!$this->getPermission())
      throw new Exception("You should not pass!!!", 301);

    $user->generateApiKey();

    $this->manager->persist($user);
    $this->manager->flush();
    return $user;
  }

  public function createUser(People $people, $username, $password)
  {
    if (!$this->getPermission())
      throw new Exception("You should not pass!!!", 301);

    $user = new User();
    $user->setPeople($people);
    $user->setHash($this->encoder->encodePassword($user, $password));
    $user->setUsername($username);

    $this->manager->persist($user);
    $this->manager->flush();
    return $user;
  }

  /**
   * @todo arrumar 
   */
  private function getPermission()
  {
    return true;
  }
}


1			<?php
2
3			namespace ControleOnline\Service;
4
5			use ControleOnline\Entity\People;
			0 ignored issues – show Bug introduced 2024-09-06 00:56 UTC by Report Bug Copy Issue Report The type `ControleOnline\Entity\People` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
6			use ControleOnline\Entity\User;
7			use Doctrine\ORM\EntityManagerInterface;
			0 ignored issues – show Bug introduced 2024-09-05 22:38 UTC by Report Bug Copy Issue Report The type `Doctrine\ORM\EntityManagerInterface` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
8			use Exception;
9			use Symfony\Component\Security\Core\Encoder\UserPasswordEncoderInterface;
			0 ignored issues – show Bug introduced 2024-03-19 23:43 UTC by Report Bug Copy Issue Report The type `Symfony\Component\Securi...asswordEncoderInterface` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
10
11			class UserService
12			{
13			public function __construct(private EntityManagerInterface $manager, private UserPasswordEncoderInterface $encoder) {}
14			public function changePassword(User $user, $password)
15			{
16			if (!$this->getPermission())
17			throw new Exception("You should not pass!!!", 301);
18
19			$user->setHash(
20			$this->encoder->encodePassword($user, $password)
21			);
22
23			$this->manager->persist($user);
24			$this->manager->flush();
25			return $user;
26			}
27
28			public function changeApiKey(User $user)
29			{
30			if (!$this->getPermission())
31			throw new Exception("You should not pass!!!", 301);
32
33			$user->generateApiKey();
34
35			$this->manager->persist($user);
36			$this->manager->flush();
37			return $user;
38			}
39
40			public function createUser(People $people, $username, $password)
41			{
42			if (!$this->getPermission())
43			throw new Exception("You should not pass!!!", 301);
44
45			$user = new User();
46			$user->setPeople($people);
47			$user->setHash($this->encoder->encodePassword($user, $password));
48			$user->setUsername($username);
49
50			$this->manager->persist($user);
51			$this->manager->flush();
52			return $user;
53			}
54
55			/**
56			* @todo arrumar
57			*/
58			private function getPermission()
59			{
60			return true;
61			}
62			}
63

ControleOnline / api-platform-users

Push — master ( 34e91e...8ddd25 )

UserService::createUser() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like