ControleOnline / api-platform-people

src/Controller/CreateUserAction.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
      *
      * @var Request
      */
-    private $request  = null;
+    private $request = null;
 
     /**
      * Security

Braces +3 added lines, -2 removed lines

@@ -119,8 +119,9 @@
                 'id' => $user->getId()
             ];
         } 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/Controller/DeleteUserAction.php

Spacing +3 added lines, -3 removed lines

@@ -26,7 +26,7 @@ 
      *
      * @var Request
      */
-    private $request  = null;
+    private $request = null;
 
     /**
      * Security
@@ -96,12 +96,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');
             }

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

Spacing +32 added lines, -32 removed lines

@@ -112,31 +112,31 @@ 
 
             case 'active':
 
-                $queryBuilder->innerJoin(SalesOrder::class,     'O'  , 'WITH', 'O.client = myclients.id');
-                $queryBuilder->innerJoin(People::class,         'C'  , 'WITH', 'C.id = O.provider');
+                $queryBuilder->innerJoin(SalesOrder::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(SalesOrder::class,     'O'  , 'WITH', 'O.client = myclients.id');
-                $queryBuilder->innerJoin(People::class,         'C'  , 'WITH', 'C.id = O.provider');
+                $queryBuilder->innerJoin(SalesOrder::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 (SalesOrder::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(SalesOrder::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(SalesOrder::class,     'O' , 'WITH', 'O.client = myclients.id');
-                $queryBuilder->innerJoin(People::class,         'C' , 'WITH', 'C.id = O.provider');
+                $queryBuilder->innerJoin(SalesOrder::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 (SalesOrder::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(SalesOrder::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(SalesOrder::class,     'O'  , 'WITH', 'O.client = myclients.id');
-                $queryBuilder->innerJoin(People::class,         'C'  , 'WITH', 'C.id = O.provider');
+                $queryBuilder->innerJoin(SalesOrder::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/Service/PeopleRoleService.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
    *
    * @var PeopleService
    */
-  private $people  = null;
+  private $people = null;
 
   public function __construct(
     EntityManagerInterface $entityManager,

Braces +12 added lines, -8 removed lines

@@ -64,8 +64,9 @@ 
     $mainCompany = $this->getMainCompany();
 
     $isSuper = $this->manager->getRepository(People::class)->getCompanyPeopleLinks($mainCompany, $people, 'employee', 1);
-    if ($isSuper) 
-      $peopleRole[] = 'super';
+    if ($isSuper) {
+          $peopleRole[] = 'super';
+    }
 
     $isFranchisee = $this->manager->getRepository(People::class)->getCompanyPeopleLinks($mainCompany, $people, 'franchisee', 1);
     if ($isFranchisee) {
@@ -74,13 +75,15 @@ 
     }
 
     $isClient = $this->manager->getRepository(People::class)->getCompanyPeopleLinks($mainCompany, $people, 'client', 1);
-    if ($isClient) 
-      $peopleRole[] = 'client';
+    if ($isClient) {
+          $peopleRole[] = 'client';
+    }
     
 
     $isSalesman = $this->manager->getRepository(People::class)->getCompanyPeopleLinks($mainCompany, $people, 'salesman', 1);
-    if ($isSalesman) 
-      $peopleRole[] = 'salesman';
+    if ($isSalesman) {
+          $peopleRole[] = 'salesman';
+    }
     
 
     return array_values(array_unique(empty($peopleRole) ? ['guest'] : $peopleRole));
@@ -96,10 +99,11 @@ 
     $domain  = $_SERVER['HTTP_HOST'];
     $company = $this->manager->getRepository(PeopleDomain::class)->findOneBy(['domain' => $domain]);
 
-    if ($company === null)
-      throw new \Exception(
+    if ($company === null) {
+          throw new \Exception(
         sprintf('Main company "%s" not found', $domain)
       );
+    }
 
     return $company->getPeople();
   }

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/Controller/GetDefaultCompanyAction.php

Spacing +2 added lines, -2 removed lines

@@ -69,7 +69,7 @@ 
           'logo'       => $this->company->getPeople()->getFile() ? [
             'id'     => $this->company->getPeople()->getFile()->getId(),
             'domain' => $_SERVER['HTTP_HOST'],
-            'url'    => '/files/download/' . $this->company->getPeople()->getFile()->getId()
+            'url'    => '/files/download/'.$this->company->getPeople()->getFile()->getId()
           ] : null,
         ];
       }
@@ -107,7 +107,7 @@ 
       'background'  =>  $this->company->getTheme()->getBackground() ? [
         'id'     =>  $this->company->getTheme()->getBackground(),
         'domain' => $_SERVER['HTTP_HOST'],
-        'url'    => '/files/download/' .  $this->company->getTheme()->getBackground()
+        'url'    => '/files/download/'.$this->company->getTheme()->getBackground()
       ] : null,
     ];
   }

src/Controller/GetMyCompaniesAction.php

Spacing +10 added lines, -10 removed lines

@@ -27,7 +27,7 @@ 
   public function __construct(Security $security, EntityManagerInterface $entityManager, PeopleRoleService $roles)
   {
     $this->security = $security;
-    $this->em      = $entityManager;
+    $this->em = $entityManager;
     $this->roles = $roles;
   }
 
@@ -134,7 +134,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()] = [
@@ -162,7 +162,7 @@ 
         $myCompanies[$key]['permission'] = array_values($permission);
       }
 
-      usort($myCompanies, function ($a, $b) {
+      usort($myCompanies, function($a, $b) {
 
         if ($a['alias'] == $b['alias']) {
           return 0;
@@ -208,11 +208,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;
@@ -247,7 +247,7 @@ 
     /**
      * @var \ControleOnline\Entity\Document $document
      */
-    $documents = $documents->filter(function ($document) {
+    $documents = $documents->filter(function($document) {
       return $document->getDocumentType()->getDocumentType() == 'CNPJ';
     });
 
@@ -260,7 +260,7 @@ 
       return [
         'id'     => $company->getFile()->getId(),
         'domain' => $_SERVER['HTTP_HOST'],
-        'url'    => '/files/download/' . $company->getFile()->getId()
+        'url'    => '/files/download/'.$company->getFile()->getId()
       ];
 
     return null;

Braces +3 added lines, -2 removed lines

@@ -256,12 +256,13 @@
 
   private function getLogo(People $company): ?array
   {
-    if ($company->getFile() instanceof File)
-      return [
+    if ($company->getFile() instanceof File) {
+          return [
         'id'     => $company->getFile()->getId(),
         'domain' => $_SERVER['HTTP_HOST'],
         'url'    => '/files/download/' . $company->getFile()->getId()
       ];
+    }
 
     return null;
   }

src/Entity/People.php

Spacing +10 added lines, -10 removed lines

@@ -316,15 +316,15 @@ 
     public function __construct()
     {
         $this->enable = 0;
-        $this->registerDate =            new \DateTime('now');
-        $this->company =            new \Doctrine\Common\Collections\ArrayCollection();
-        $this->config =            new \Doctrine\Common\Collections\ArrayCollection();
-        $this->link =            new \Doctrine\Common\Collections\ArrayCollection();
-        $this->user =            new \Doctrine\Common\Collections\ArrayCollection();
-        $this->document =            new \Doctrine\Common\Collections\ArrayCollection();
-        $this->address =            new \Doctrine\Common\Collections\ArrayCollection();
-        $this->email =            new \Doctrine\Common\Collections\ArrayCollection();
-        $this->phone =            new \Doctrine\Common\Collections\ArrayCollection();
+        $this->registerDate = new \DateTime('now');
+        $this->company = new \Doctrine\Common\Collections\ArrayCollection();
+        $this->config = new \Doctrine\Common\Collections\ArrayCollection();
+        $this->link = new \Doctrine\Common\Collections\ArrayCollection();
+        $this->user = new \Doctrine\Common\Collections\ArrayCollection();
+        $this->document = new \Doctrine\Common\Collections\ArrayCollection();
+        $this->address = new \Doctrine\Common\Collections\ArrayCollection();
+        $this->email = new \Doctrine\Common\Collections\ArrayCollection();
+        $this->phone = new \Doctrine\Common\Collections\ArrayCollection();
         $this->otherInformations = json_encode(
             new stdClass()
         );
@@ -582,7 +582,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';
             }