ControleOnline / api-platform-people

src/Controller/DeleteUserAction.php

Braces +3 added lines, -2 removed lines

@@ -113,8 +113,9 @@
 
             return true;
         } catch (\Exception $e) {
-            if ($this->manager->getConnection()->isTransactionActive())
-                $this->manager->getConnection()->rollBack();
+            if ($this->manager->getConnection()->isTransactionActive()) {
+                            $this->manager->getConnection()->rollBack();
+            }
 
             throw new \InvalidArgumentException($e->getMessage());
         }

src/Repository/PeopleRepository.php

Braces +3 added lines, -2 removed lines

@@ -51,8 +51,9 @@
             ->setParameter('company', $company->getId())
             ->setParameter('people', $people->getId());
 
-        if ($linkType)
-            $queryBuilder->setParameter('linkType', $linkType)->andWhere('pl.link_type = :linkType');
+        if ($linkType) {
+                    $queryBuilder->setParameter('linkType', $linkType)->andWhere('pl.link_type = :linkType');
+        }
 
         if ($maxResults) {
             $queryBuilder->setMaxResults($maxResults);

src/Service/PeopleRoleService.php

Braces +18 added lines, -9 removed lines

@@ -34,7 +34,9 @@ 
     $isSalesman = false;
 
     $isSalesman = $this->manager->getRepository(People::class)->getCompanyPeopleLinks($mainCompany, $people, 'salesman', 1);
-    if ($isSalesman) return true;
+    if ($isSalesman) {
+     return true;
+    }
 
     $getPeopleCompanies = $this->manager->getRepository(PeopleLink::class)->findBy([
       'people' => $people,
@@ -45,7 +47,9 @@ 
      */
     foreach ($getPeopleCompanies as $peopleCompany) {
       $isSalesman = $this->manager->getRepository(People::class)->getCompanyPeopleLinks($mainCompany, $peopleCompany->getCompany(), 'salesman', 1);
-      if ($isSalesman) return true;
+      if ($isSalesman) {
+       return true;
+      }
     }
     return $isSalesman;
   }
@@ -57,8 +61,9 @@ 
     $mainCompany = $this->getMainCompany();
 
     $isSuper = $this->manager->getRepository(People::class)->getCompanyPeopleLinks($mainCompany, $people, 'employee', 1);
-    if ($isSuper)
-      $peopleRole[] = 'super';
+    if ($isSuper) {
+          $peopleRole[] = 'super';
+    }
 
 
 
@@ -69,13 +74,15 @@ 
     }
 
     $isClient = $this->manager->getRepository(People::class)->getCompanyPeopleLinks($mainCompany, $people, 'client', 1);
-    if ($isClient)
-      $peopleRole[] = 'client';
+    if ($isClient) {
+          $peopleRole[] = 'client';
+    }
 
 
     $isSalesman = $this->isSalesman($people);
-    if ($isSalesman)
-      $peopleRole[] = 'salesman';
+    if ($isSalesman) {
+          $peopleRole[] = 'salesman';
+    }
 
 
     return array_values(array_unique(empty($peopleRole) ? ['guest'] : $peopleRole));
@@ -89,7 +96,9 @@ 
   public function getMainCompany(): People
   {
 
-    if (self::$mainCompany) return self::$mainCompany;
+    if (self::$mainCompany) {
+     return self::$mainCompany;
+    }
 
     $peopleDomain = $this->domainService->getPeopleDomain();
     self::$mainCompany =  $peopleDomain->getPeople();

src/Service/PeopleService.php

Braces +42 added lines, -27 removed lines

@@ -47,11 +47,13 @@ 
 
     $people = $this->getDocument($document)?->getPeople();
 
-    if (!$people)
-      $people = $this->getEmail($email)?->getPeople();
+    if (!$people) {
+          $people = $this->getEmail($email)?->getPeople();
+    }
 
-    if (!$people)
-      $people = $this->getPhone($phone['ddd'], $phone['phone'])?->getPeople();
+    if (!$people) {
+          $people = $this->getPhone($phone['ddd'], $phone['phone'])?->getPeople();
+    }
 
     if (!$people) {
       $people = new People();
@@ -62,12 +64,15 @@ 
       $this->manager->flush();
     }
 
-    if ($document)
-      $this->addDocument($people, $document);
-    if ($email)
-      $this->addEmail($people, $email);
-    if ($phone)
-      $this->addPhone($people, $phone);
+    if ($document) {
+          $this->addDocument($people, $document);
+    }
+    if ($email) {
+          $this->addEmail($people, $email);
+    }
+    if ($phone) {
+          $this->addPhone($people, $phone);
+    }
 
 
     return $people;
@@ -77,8 +82,9 @@ 
   {
     $phone = $this->getPhone($phone_number['ddd'], $phone_number['phone']);
     if ($phone && $phone->getPeople()) {
-      if ($phone->getPeople()->getId() != $people->getId())
-        throw new Exception("Phone is in use by people " . $people->getId(), 1);
+      if ($phone->getPeople()->getId() != $people->getId()) {
+              throw new Exception("Phone is in use by people " . $people->getId(), 1);
+      }
     } else {
       $phone = new Phone();
       $phone->setDdd($phone_number['ddd']);
@@ -94,8 +100,9 @@ 
   {
     $document = $this->getDocument($document_number, $document_type);
     if ($document) {
-      if ($document->getPeople()->getId() != $people->getId())
-        throw new Exception("Document is in use by people " . $people->getId(), 1);
+      if ($document->getPeople()->getId() != $people->getId()) {
+              throw new Exception("Document is in use by people " . $people->getId(), 1);
+      }
     } else {
       $document = new Document();
       $document->setDocument($document_number);
@@ -112,8 +119,9 @@ 
   {
     $email = $this->getEmail($email_str);
     if ($email && $email->getPeople()) {
-      if ($email->getPeople()->getId() != $people->getId())
-        throw new Exception("Email is in use by people " . $people->getId(), 1);
+      if ($email->getPeople()->getId() != $people->getId()) {
+              throw new Exception("Email is in use by people " . $people->getId(), 1);
+      }
     } else {
       $email = new Email();
       $email->setEmail($email_str);
@@ -152,8 +160,9 @@ 
 
   public function getDocument(string $document_number, ?string $document_type = null): ?Document
   {
-    if (!$document_type)
-      $document_type = $this->getDocumentTypeByDocumentLen($document_number);
+    if (!$document_type) {
+          $document_type = $this->getDocumentTypeByDocumentLen($document_number);
+    }
     return $this->manager->getRepository(Document::class)->findOneBy([
       'document' => $document_number,
       'documentType' =>
@@ -172,14 +181,15 @@ 
     $payload   = json_decode($request->getContent());
     if (isset($payload->link_type)) {
       $company = $this->manager->getRepository(People::class)->find(preg_replace('/\D/', '', $payload->company));
-      if ($company)
-        $this->addLink($company, $people, $payload->link_type);
-      else {
+      if ($company) {
+              $this->addLink($company, $people, $payload->link_type);
+      } else {
         $link = $this->manager->getRepository(People::class)->find(preg_replace('/\D/', '', $payload->link));
         if ($payload->link_type == 'employee' && $link) {
           $this->addLink($people, $link, $payload->link_type);
-          if ($payload->people_document)
-            $this->addDocument($people, $payload->people_document);
+          if ($payload->people_document) {
+                      $this->addDocument($people, $payload->people_document);
+          }
         }
       }
     }
@@ -194,8 +204,9 @@ 
       'link_type' => $link_type
     ]);
 
-    if (!$peopleLink)
-      $peopleLink = new PeopleLink();
+    if (!$peopleLink) {
+          $peopleLink = new PeopleLink();
+    }
 
     $peopleLink->setCompany($company);
     $peopleLink->setPeople($people);
@@ -247,10 +258,14 @@ 
      * @var \ControleOnline\Entity\User $currentUser
      */
     $token = $this->security->getToken();
-    if (!$token) return [];
+    if (!$token) {
+     return [];
+    }
     $currentUser  =  $token->getUser();
     $companies    = [];
-    if (!$currentUser) return [];
+    if (!$currentUser) {
+     return [];
+    }
 
     if (!$currentUser->getPeople()->getLink()->isEmpty()) {
       foreach ($currentUser->getPeople()->getLink() as $company) {