ControleOnline / api-platform-people

src/Service/PeopleService.php

Spacing +14 added lines, -14 removed lines

@@ -27,7 +27,7 @@ 
     private Security               $security,
     private RequestStack $requestStack,
   ) {
-    $this->request  = $requestStack->getCurrentRequest();
+    $this->request = $requestStack->getCurrentRequest();
   }
 
   public function prePersist(People $people)
@@ -79,7 +79,7 @@ 
     $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);
+        throw new Exception("Phone is in use by people ".$people->getId(), 1);
     } else {
       $phone = new Phone();
       $phone->setDdd((int) $phone_number['ddd']);
@@ -91,16 +91,16 @@ 
 
     return  $phone;
   }
-  public function addDocument(People $people, string|int $document_number, ?string $document_type = null): Document
+  public function addDocument(People $people, string | int $document_number, ?string $document_type = null): Document
   {
     $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);
+        throw new Exception("Document is in use by people ".$people->getId(), 1);
     } else {
       $document_type = $document_type ? $this->discoveryDocumentType($document_type) : $this->discoveryDocumentType($this->getDocumentTypeByDocumentLen($document_number));
       $document = new Document();
-      $document->setDocument((int)$document_number);
+      $document->setDocument((int) $document_number);
       $document->setDocumentType($document_type);
       $document->setPeople($people);
       $this->manager->persist($document);
@@ -115,7 +115,7 @@ 
     $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);
+        throw new Exception("Email is in use by people ".$people->getId(), 1);
     } else {
       $email = new Email();
       $email->setEmail($email_str);
@@ -140,7 +140,7 @@ 
 
   public function discoveryDocumentType(string $document_type): DocumentType
   {
-    $documentType =  $this->manager->getRepository(DocumentType::class)->findOneBy(['documentType' => $document_type]);
+    $documentType = $this->manager->getRepository(DocumentType::class)->findOneBy(['documentType' => $document_type]);
 
     if (!$documentType) {
       $documentType = new DocumentType();
@@ -222,29 +222,29 @@ 
   public function checkLink(QueryBuilder $queryBuilder, $resourceClass = null, $applyTo = null, $rootAlias = null): void
   {
 
-    $link   = $this->request->query->get('link',   null);
+    $link = $this->request->query->get('link', null);
     $company = $this->request->query->get('company', null);
     $link_type = $this->request->query->get('link_type', null);
 
     if ($link_type) {
-      $queryBuilder->join(sprintf('%s.' . ($link ? 'company' : 'link'), $rootAlias), 'PeopleLink');
+      $queryBuilder->join(sprintf('%s.'.($link ? 'company' : 'link'), $rootAlias), 'PeopleLink');
       $queryBuilder->andWhere('PeopleLink.link_type IN(:link_type)');
       $queryBuilder->setParameter('link_type', $link_type);
     }
 
     if ($company || $link) {
-      $queryBuilder->andWhere('PeopleLink.' . ($link ? 'people' : 'company') . ' IN(:people)');
+      $queryBuilder->andWhere('PeopleLink.'.($link ? 'people' : 'company').' IN(:people)');
       $queryBuilder->setParameter('people', preg_replace("/[^0-9]/", "", ($link ?: $company)));
     }
   }
   public function checkCompany($type, QueryBuilder $queryBuilder, $resourceClass = null, $applyTo = null, $rootAlias = null): void
   {
-    $companies   = $this->getMyCompanies();
-    $queryBuilder->andWhere(sprintf('%s.' . $type . ' IN(:companies)', $rootAlias, $rootAlias));
+    $companies = $this->getMyCompanies();
+    $queryBuilder->andWhere(sprintf('%s.'.$type.' IN(:companies)', $rootAlias, $rootAlias));
     $queryBuilder->setParameter('companies', $companies);
 
     if ($payer = $this->request->query->get('company', null)) {
-      $queryBuilder->andWhere(sprintf('%s.' . $type . ' IN(:people)', $rootAlias));
+      $queryBuilder->andWhere(sprintf('%s.'.$type.' IN(:people)', $rootAlias));
       $queryBuilder->setParameter('people', preg_replace("/[^0-9]/", "", $payer));
     }
   }
@@ -256,7 +256,7 @@ 
      */
     $token = $this->security->getToken();
     if (!$token) return [];
-    $currentUser  =  $token->getUser();
+    $currentUser  = $token->getUser();
     $companies    = [];
     if (!$currentUser) return [];
 

Braces +48 added lines, -30 removed lines

@@ -45,14 +45,17 @@ 
   public function discoveryPeople(?string $document = null, ?string  $email = null, ?array $phone = [], ?string $name = null, ?string $peopleType = null): People
   {
 
-    if (!empty($document))
-      $people = $this->getDocument($document)?->getPeople();
+    if (!empty($document)) {
+          $people = $this->getDocument($document)?->getPeople();
+    }
 
-    if (!$people && !empty($email))
-      $people = $this->getEmail($email)?->getPeople();
+    if (!$people && !empty($email)) {
+          $people = $this->getEmail($email)?->getPeople();
+    }
 
-    if (!$people && !empty($phone))
-      $people = $this->getPhone($phone['ddd'], $phone['phone'])?->getPeople();
+    if (!$people && !empty($phone)) {
+          $people = $this->getPhone($phone['ddd'], $phone['phone'])?->getPeople();
+    }
 
     if (!$people) {
       $people = new People();
@@ -63,12 +66,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;
@@ -78,8 +84,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((int) $phone_number['ddd']);
@@ -95,8 +102,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_type = $document_type ? $this->discoveryDocumentType($document_type) : $this->discoveryDocumentType($this->getDocumentTypeByDocumentLen($document_number));
       $document = new Document();
@@ -114,8 +122,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);
@@ -154,8 +163,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' =>
@@ -176,18 +186,21 @@ 
   public function postPersist(People $people)
   {
     $request = $this->requestStack->getCurrentRequest();
-    if (!$request) return;
+    if (!$request) {
+     return;
+    }
     $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);
+          }
         }
       }
     }
@@ -202,8 +215,9 @@ 
       'link_type' => $link_type
     ]);
 
-    if (!$peopleLink)
-      $peopleLink = new PeopleLink();
+    if (!$peopleLink) {
+          $peopleLink = new PeopleLink();
+    }
 
     $peopleLink->setCompany($company);
     $peopleLink->setPeople($people);
@@ -255,10 +269,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) {