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

Passed

Push — master ( 4b767b...1a65fd )

by Luiz Kim

created 2024-09-05 22:29 UTC

UserService::changePassword() A

↳ Parent: UserService

Complexity

Conditions	2
Paths	2

Size

Total Lines	12
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	7
c	1
b	0
f	0
dl	0
loc	12
rs	10
cc	2
nc	2
nop	2

<?php

namespace App\Service;

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 createUser($people_id, $username, $password)
  {
    if (!$this->getPermission())
      throw new Exception("You should not pass!!!", 301);
    
    $user = new User();
    $user->setPeople($this->manager->getRepository(User::class)->find($people_id));
    $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 App\Service;
4
5			use ControleOnline\Entity\User;
6			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...
7			use Exception;
8			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...
9
10			class UserService
11			{
12			public function __construct(private EntityManagerInterface $manager, private UserPasswordEncoderInterface $encoder) {}
13			public function changePassword(User $user, $password)
14			{
15			if (!$this->getPermission())
16			throw new Exception("You should not pass!!!", 301);
17
18			$user->setHash(
19			$this->encoder->encodePassword($user, $password)
20			);
21
22			$this->manager->persist($user);
23			$this->manager->flush();
24			return $user;
25			}
26
27			public function createUser($people_id, $username, $password)
28			{
29			if (!$this->getPermission())
30			throw new Exception("You should not pass!!!", 301);
31
32			$user = new User();
33			$user->setPeople($this->manager->getRepository(User::class)->find($people_id));
34			$user->setHash($this->encoder->encodePassword($user, $password));
35			$user->setUsername($username);
36
37			$this->manager->persist($user);
38			$this->manager->flush();
39			return $user;
40			}
41
42			/**
43			* @todo arrumar
44			*/
45			private function getPermission()
46			{
47			return true;
48			}
49			}
50

ControleOnline / api-platform-users

Push — master ( 4b767b...1a65fd )

UserService::changePassword() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like