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());
         }

Spacing +4 added lines, -4 removed lines

@@ -27,13 +27,13 @@ 
      *
      * @var Request
      */
-    private $request  = null;
+    private $request = null;
 
 
 
     public function __construct(EntityManagerInterface $manager)
     {
-        $this->manager     = $manager;
+        $this->manager = $manager;
     }
 
     public function __invoke(People $data, Request $request): JsonResponse
@@ -75,12 +75,12 @@ 
                 throw new \InvalidArgumentException('Document id is not defined');
             }
 
-            $users   = $this->manager->getRepository(User::class)->findBy(['people' => $person]);
+            $users = $this->manager->getRepository(User::class)->findBy(['people' => $person]);
             if (count($users) == 1) {
                 throw new \InvalidArgumentException('Deve existir pelo menos um usuário');
             }
 
-            $user    = $this->manager->getRepository(User::class)->findOneBy(['id' => $payload['id'], 'people' => $person]);
+            $user = $this->manager->getRepository(User::class)->findOneBy(['id' => $payload['id'], 'people' => $person]);
             if (!$user instanceof User) {
                 throw new \InvalidArgumentException('Person user was not found');
             }

src/Repository/PeopleRepository.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
     }
 
 
-    public function getCompanyPeopleLinks(People $company,  People $people, $linkType = null, $maxResults = null)
+    public function getCompanyPeopleLinks(People $company, People $people, $linkType = null, $maxResults = null)
     {
         $queryBuilder = $this->getEntityManager()->createQueryBuilder();
         $queryBuilder->select('pl')

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/Repository/ClientRepository.php

Spacing +32 added lines, -32 removed lines

@@ -112,31 +112,31 @@ 
 
             case 'active':
 
-                $queryBuilder->innerJoin(Order::class,     'O'  , 'WITH', 'O.client = myclients.id');
-                $queryBuilder->innerJoin(People::class,         'C'  , 'WITH', 'C.id = O.provider');
+                $queryBuilder->innerJoin(Order::class, 'O', 'WITH', 'O.client = myclients.id');
+                $queryBuilder->innerJoin(People::class, 'C', 'WITH', 'C.id = O.provider');
                 $queryBuilder->innerJoin(PeopleSalesman::class, 'kkk', 'WITH', 'kkk.company = C.id');
-                $queryBuilder->innerJoin(People::class,         'S'  , 'WITH', 'S.id = kkk.salesman');
-                $queryBuilder->innerJoin(PeopleClient::class,   'PC' , 'WITH', 'PC.client = myclients.id AND PC.company_id = S.id');
+                $queryBuilder->innerJoin(People::class, 'S', 'WITH', 'S.id = kkk.salesman');
+                $queryBuilder->innerJoin(PeopleClient::class, 'PC', 'WITH', 'PC.client = myclients.id AND PC.company_id = S.id');
                 $queryBuilder->andWhere('O.provider = :provider');
                 $queryBuilder->andWhere('S.id IN (:my_companies)');
                 $queryBuilder->andWhere('O.status NOT IN (:statuses)');
                 $queryBuilder->andWhere('O.orderDate BETWEEN :from_date AND :to_date');
-                $queryBuilder->setParameter('provider'    , $provider);
-                $queryBuilder->setParameter('statuses'    , $this->getEntityManager()->getRepository(Status::class)->findBy(['realStatus' => ['open','canceled']]));
-                $queryBuilder->setParameter('from_date'   , $fromDate);
-                $queryBuilder->setParameter('to_date'     , $toDate);
+                $queryBuilder->setParameter('provider', $provider);
+                $queryBuilder->setParameter('statuses', $this->getEntityManager()->getRepository(Status::class)->findBy(['realStatus' => ['open', 'canceled']]));
+                $queryBuilder->setParameter('from_date', $fromDate);
+                $queryBuilder->setParameter('to_date', $toDate);
                 $queryBuilder->setParameter('my_companies', $this->getMyCompanies($salesman));
 
             break;
 
             case 'inactive':
 
-                $queryBuilder->innerJoin(Order::class,     'O'  , 'WITH', 'O.client = myclients.id');
-                $queryBuilder->innerJoin(People::class,         'C'  , 'WITH', 'C.id = O.provider');
+                $queryBuilder->innerJoin(Order::class, 'O', 'WITH', 'O.client = myclients.id');
+                $queryBuilder->innerJoin(People::class, 'C', 'WITH', 'C.id = O.provider');
                 $queryBuilder->innerJoin(PeopleSalesman::class, 'kkk', 'WITH', 'kkk.company = C.id');
-                $queryBuilder->innerJoin(People::class,         'S'  , 'WITH', 'S.id = kkk.salesman');
-                $queryBuilder->innerJoin(PeopleClient::class,   'PC' , 'WITH', 'PC.client = myclients.id AND PC.company_id = S.id');
-                $queryBuilder->leftJoin (Order::class,     'jj' , 'WITH', 'jj.client = myclients.id
+                $queryBuilder->innerJoin(People::class, 'S', 'WITH', 'S.id = kkk.salesman');
+                $queryBuilder->innerJoin(PeopleClient::class, 'PC', 'WITH', 'PC.client = myclients.id AND PC.company_id = S.id');
+                $queryBuilder->leftJoin(Order::class, 'jj', 'WITH', 'jj.client = myclients.id
                                                                              AND   (jj.status NOT IN (:statuses) OR jj.id IS NULL)
                                                                              AND   ((jj.orderDate  BETWEEN  :from_date AND :to_date) OR jj.id IS NULL)
                                                                              AND   (jj.provider = :provider  OR jj.id IS NULL)');
@@ -147,23 +147,23 @@ 
                 $queryBuilder->groupBy('myclients.id');
                 $queryBuilder->having('COUNT(jj.id) > :count');
                 $queryBuilder->andHaving('COUNT(O.id) = :count');
-                $queryBuilder->setParameter('provider' , $provider);
-                $queryBuilder->setParameter('statuses' , $this->getEntityManager()->getRepository(Status::class)->findBy(['realStatus' => ['open','canceled']]));
+                $queryBuilder->setParameter('provider', $provider);
+                $queryBuilder->setParameter('statuses', $this->getEntityManager()->getRepository(Status::class)->findBy(['realStatus' => ['open', 'canceled']]));
                 $queryBuilder->setParameter('from_date', $fromDate);
-                $queryBuilder->setParameter('to_date'  , $toDate);
-                $queryBuilder->setParameter('count'    , 0);
+                $queryBuilder->setParameter('to_date', $toDate);
+                $queryBuilder->setParameter('count', 0);
                 $queryBuilder->setParameter('my_companies', $this->getMyCompanies($salesman));
 
             break;
 
             case 'prospect':
 
-                $queryBuilder->innerJoin(Order::class,     'O' , 'WITH', 'O.client = myclients.id');
-                $queryBuilder->innerJoin(People::class,         'C' , 'WITH', 'C.id = O.provider');
+                $queryBuilder->innerJoin(Order::class, 'O', 'WITH', 'O.client = myclients.id');
+                $queryBuilder->innerJoin(People::class, 'C', 'WITH', 'C.id = O.provider');
                 $queryBuilder->innerJoin(PeopleSalesman::class, 'PS', 'WITH', 'PS.company = C.id');
-                $queryBuilder->innerJoin(People::class,         'S' , 'WITH', 'S.id = PS.salesman');
-                $queryBuilder->innerJoin(PeopleClient::class,   'PC', 'WITH', 'PC.client = myclients.id AND PC.company_id = S.id');
-                $queryBuilder->leftJoin (Order::class,     'jj' , 'WITH', 'jj.client = myclients.id
+                $queryBuilder->innerJoin(People::class, 'S', 'WITH', 'S.id = PS.salesman');
+                $queryBuilder->innerJoin(PeopleClient::class, 'PC', 'WITH', 'PC.client = myclients.id AND PC.company_id = S.id');
+                $queryBuilder->leftJoin(Order::class, 'jj', 'WITH', 'jj.client = myclients.id
                                                                              AND   (jj.status NOT IN (:statuses) OR jj.id IS NULL)                                                                             
                                                                              AND   (jj.provider = :provider  OR jj.id IS NULL)');
                 $queryBuilder->andWhere('O.provider = :provider');
@@ -173,19 +173,19 @@ 
                 $queryBuilder->having('COUNT(jj.id) = :count');
 
                 $queryBuilder->setParameter('provider', $provider);
-                $queryBuilder->setParameter('statuses', $this->getEntityManager()->getRepository(Status::class)->findBy(['realStatus' => ['open','canceled']]));
-                $queryBuilder->setParameter('count'   , 0);
+                $queryBuilder->setParameter('statuses', $this->getEntityManager()->getRepository(Status::class)->findBy(['realStatus' => ['open', 'canceled']]));
+                $queryBuilder->setParameter('count', 0);
                 $queryBuilder->setParameter('my_companies', $this->getMyCompanies($salesman));
 
             break;
 
             case 'new':
 
-                $queryBuilder->innerJoin(Order::class,     'O'  , 'WITH', 'O.client = myclients.id');
-                $queryBuilder->innerJoin(People::class,         'C'  , 'WITH', 'C.id = O.provider');
+                $queryBuilder->innerJoin(Order::class, 'O', 'WITH', 'O.client = myclients.id');
+                $queryBuilder->innerJoin(People::class, 'C', 'WITH', 'C.id = O.provider');
                 $queryBuilder->innerJoin(PeopleSalesman::class, 'kkk', 'WITH', 'kkk.company = C.id');
-                $queryBuilder->innerJoin(People::class,         'S'  , 'WITH', 'S.id = kkk.salesman');
-                $queryBuilder->innerJoin(PeopleClient::class,   'PC' , 'WITH', 'PC.client = myclients.id AND PC.company_id = S.id');
+                $queryBuilder->innerJoin(People::class, 'S', 'WITH', 'S.id = kkk.salesman');
+                $queryBuilder->innerJoin(PeopleClient::class, 'PC', 'WITH', 'PC.client = myclients.id AND PC.company_id = S.id');
 
                 $queryBuilder->andWhere('O.provider = :provider');
                 $queryBuilder->andWhere('S.id IN (:my_companies)');
@@ -193,10 +193,10 @@ 
                 $queryBuilder->andWhere('O.orderDate    BETWEEN :from_date AND :to_date');
                 $queryBuilder->andWhere('myclients.registerDate BETWEEN :from_date AND :to_date');
 
-                $queryBuilder->setParameter('provider'    , $provider);
-                $queryBuilder->setParameter('statuses'    , $this->getEntityManager()->getRepository(Status::class)->findBy(['realStatus' => ['open','canceled']]));
-                $queryBuilder->setParameter('from_date'   , $fromDate);
-                $queryBuilder->setParameter('to_date'     , $toDate);
+                $queryBuilder->setParameter('provider', $provider);
+                $queryBuilder->setParameter('statuses', $this->getEntityManager()->getRepository(Status::class)->findBy(['realStatus' => ['open', 'canceled']]));
+                $queryBuilder->setParameter('from_date', $fromDate);
+                $queryBuilder->setParameter('to_date', $toDate);
                 $queryBuilder->setParameter('my_companies', $this->getMyCompanies($salesman));
 
             break;

src/Controller/GetMyCompaniesAction.php

Spacing +8 added lines, -8 removed lines

@@ -130,7 +130,7 @@ 
             }
           }
 
-          $peopleemployee =   $this->em->getRepository(PeopleLink::class)->findOneBy(['company' => $company, 'employee' => $userPeople]);
+          $peopleemployee = $this->em->getRepository(PeopleLink::class)->findOneBy(['company' => $company, 'employee' => $userPeople]);
 
           $permissions[$company->getId()][] = 'salesman';
           $myCompanies[$company->getId()] = [
@@ -158,7 +158,7 @@ 
         $myCompanies[$key]['permission'] = array_values($permission);
       }
 
-      usort($myCompanies, function ($a, $b) {
+      usort($myCompanies, function($a, $b) {
 
         if ($a['alias'] == $b['alias']) {
           return 0;
@@ -204,11 +204,11 @@ 
         $p_m[$package_module->getId()]['module'] = $package_module->getModule()->getName();
       }
 
-      $packages[$people_package->getId()]['id']                   =  $people_package->getId();
-      $packages[$people_package->getId()]['package']['id']        =  $package->getId();
-      $packages[$people_package->getId()]['package']['name']      =  $package->getName();
-      $packages[$people_package->getId()]['package']['active']    =  $package->isActive() ? true : false;
-      $packages[$people_package->getId()]['package']['modules']   =  $p_m;
+      $packages[$people_package->getId()]['id']                   = $people_package->getId();
+      $packages[$people_package->getId()]['package']['id']        = $package->getId();
+      $packages[$people_package->getId()]['package']['name']      = $package->getName();
+      $packages[$people_package->getId()]['package']['active']    = $package->isActive() ? true : false;
+      $packages[$people_package->getId()]['package']['modules']   = $p_m;
     }
 
     return $packages;
@@ -243,7 +243,7 @@ 
     /**
      * @var \ControleOnline\Entity\Document $document
      */
-    $documents = $documents->filter(function ($document) {
+    $documents = $documents->filter(function($document) {
       return $document->getDocumentType()->getDocumentType() == 'CNPJ';
     });
 

src/Controller/GetDefaultCompanyAction.php

Spacing +1 added lines, -1 removed lines

@@ -88,7 +88,7 @@
       'background'  =>  $this->domainService->getPeopleDomain()->getTheme()->getBackground() ? [
         'id'     =>  $this->domainService->getPeopleDomain()->getTheme()->getBackground(),
         'domain' => $this->domainService->getMainDomain(),
-        'url'    => '/files/' .  $this->domainService->getPeopleDomain()->getTheme()->getBackground() . '/download'
+        'url'    => '/files/'.$this->domainService->getPeopleDomain()->getTheme()->getBackground().'/download'
       ] : null,
     ];
   }

src/Service/PeopleRoleService.php

Spacing +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
     if (self::$mainCompany) return self::$mainCompany;
 
     $peopleDomain = $this->domainService->getPeopleDomain();
-    self::$mainCompany =  $peopleDomain->getPeople();
+    self::$mainCompany = $peopleDomain->getPeople();
     
     return self::$mainCompany;
   }

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/Entity/People.php

Spacing +1 added lines, -1 removed lines

@@ -698,7 +698,7 @@
 
     public function getOneDocument(): ?Document
     {
-        $documents = $this->getDocument()->filter(function ($peopleDocument) {
+        $documents = $this->getDocument()->filter(function($peopleDocument) {
             if ($peopleDocument->getPeople()->getPeopleType() == 'F') {
                 return $peopleDocument->getDocumentType()->getDocumentType() == 'CPF';
             }

src/Service/PeopleService.php

Spacing +15 added lines, -15 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)
@@ -47,7 +47,7 @@ 
 
   public function discoveryLink(People $company, People $people, $linkType): PeopleLink
   {
-    $peopleLink =   $this->manager->getRepository(PeopleLink::class)->findOneBy([
+    $peopleLink = $this->manager->getRepository(PeopleLink::class)->findOneBy([
       'company' => $company,
       'people' => $people,
       'link_type' => $linkType
@@ -96,7 +96,7 @@ 
     $phone = $this->getPhone($phone_number['ddi'], $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->setDdi((int) $phone_number['ddi']);
@@ -109,16 +109,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);
@@ -133,7 +133,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);
@@ -162,7 +162,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();
@@ -244,29 +244,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));
     }
   }
@@ -278,7 +278,7 @@ 
      */
     $token = $this->security->getToken();
     if (!$token) return [];
-    $currentUser  =  $token->getUser();
+    $currentUser  = $token->getUser();
     $companies    = [];
     if (!$currentUser) return [];
 

Braces +51 added lines, -32 removed lines

@@ -53,8 +53,9 @@ 
       'link_type' => $linkType
     ]);
 
-    if (!$peopleLink)
-      $peopleLink = $this->addLink($company, $people, $linkType);
+    if (!$peopleLink) {
+          $peopleLink = $this->addLink($company, $people, $linkType);
+    }
 
     return $peopleLink;
   }
@@ -62,14 +63,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['ddi'], $phone['ddd'], $phone['phone'])?->getPeople();
+    if (!$people && !empty($phone)) {
+          $people = $this->getPhone($phone['ddi'], $phone['ddd'], $phone['phone'])?->getPeople();
+    }
 
     if (!$people) {
       $people = new People();
@@ -80,12 +84,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;
@@ -95,8 +102,9 @@ 
   {
     $phone = $this->getPhone($phone_number['ddi'], $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->setDdi((int) $phone_number['ddi']);
@@ -113,8 +121,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();
@@ -132,8 +141,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);
@@ -176,8 +186,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' =>
@@ -198,18 +209,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->discoveryLink($company, $people, $payload->link_type);
-      else {
+      if ($company) {
+              $this->discoveryLink($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->discoveryLink($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);
+          }
         }
       }
     }
@@ -224,8 +238,9 @@ 
       'link_type' => $link_type
     ]);
 
-    if (!$peopleLink)
-      $peopleLink = new PeopleLink();
+    if (!$peopleLink) {
+          $peopleLink = new PeopleLink();
+    }
 
     $peopleLink->setCompany($company);
     $peopleLink->setPeople($people);
@@ -277,10 +292,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) {