ControleOnline / api-platform-users

src/Controller/CreateUserAction.php

Braces +5 added lines, -4 removed lines

@@ -26,10 +26,11 @@
     try {
       $payload   = json_decode($request->getContent());
 
-      if ($payload->people)
-        $people =  $this->manager->getRepository(People::class)->find($payload->people);
-      else
-        $people =  $this->service->discoveryPeople($payload->username, $payload->firstName, $payload->lastName);
+      if ($payload->people) {
+              $people =  $this->manager->getRepository(People::class)->find($payload->people);
+      } else {
+              $people =  $this->service->discoveryPeople($payload->username, $payload->firstName, $payload->lastName);
+      }
 
       $user = $this->service->createUser(
         $people,

src/Service/UserService.php

Braces +18 added lines, -12 removed lines

@@ -15,8 +15,9 @@ 
   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);
+    if (!$this->getPermission()) {
+          throw new Exception("You should not pass!!!", 301);
+    }
 
     $user->setHash(
       $this->encoder->encodePassword($user, $password)
@@ -29,8 +30,9 @@ 
 
   public function changeApiKey(User $user)
   {
-    if (!$this->getPermission())
-      throw new Exception("You should not pass!!!", 301);
+    if (!$this->getPermission()) {
+          throw new Exception("You should not pass!!!", 301);
+    }
 
     $user->generateApiKey();
 
@@ -51,8 +53,9 @@ 
 
     $people = $this->discoveryPeople($email, $firstName, $lastName);
 
-    if (!$user)
-      $user = $this->createUser($people, $email, $password);
+    if (!$user) {
+          $user = $this->createUser($people, $email, $password);
+    }
 
     return   $user;
   }
@@ -93,16 +96,18 @@ 
 
   public function createUser(People $people, $username, $password)
   {
-    if (!$this->getPermission())
-      throw new Exception("You should not pass!!!", 301);
+    if (!$this->getPermission()) {
+          throw new Exception("You should not pass!!!", 301);
+    }
 
     $user = $this->manager->getRepository(User::class)
       ->findOneBy([
         'username'       => $username,
       ]);
 
-    if ($username)
-      throw new Exception("User already exists", 301);
+    if ($username) {
+          throw new Exception("User already exists", 301);
+    }
 
     $user = new User();
     $user->setPeople($people);
@@ -119,8 +124,9 @@ 
   {
     $peopleLink = $user->getPeople()->getLink()->first();
 
-    if ($peopleLink !== false && $peopleLink->getCompany() instanceof People)
-      return $peopleLink->getCompany();
+    if ($peopleLink !== false && $peopleLink->getCompany() instanceof People) {
+          return $peopleLink->getCompany();
+    }
   }
 
   public function getCompanyId(User $user)