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 +24 added lines, -12 removed lines

@@ -37,7 +37,9 @@ 
      */
     foreach ($getPeopleCompanies as $peopleCompany) {
       $isFranchisee = $this->manager->getRepository(People::class)->getCompanyPeopleLinks($mainCompany, $peopleCompany->getCompany(), 'franchisee', 1);
-      if ($isFranchisee) return true;
+      if ($isFranchisee) {
+       return true;
+      }
     }
     return $isFranchisee;
   }
@@ -54,7 +56,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,
@@ -65,7 +69,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;
   }
@@ -78,8 +84,9 @@ 
     $mainCompany = $this->getMainCompany();
 
     $isSuper = $this->manager->getRepository(People::class)->getCompanyPeopleLinks($mainCompany, $people, 'employee', 1);
-    if ($isSuper)
-      $peopleRole[] = 'super';
+    if ($isSuper) {
+          $peopleRole[] = 'super';
+    }
 
 
 
@@ -90,13 +97,15 @@ 
     }
 
     $isClient = $this->manager->getRepository(People::class)->getCompanyPeopleLinks($mainCompany, $people, 'client', 1);
-    if ($isClient)
-      $peopleRole[] = 'client';
+    if ($isClient) {
+          $peopleRole[] = 'client';
+    }
 
 
     $isSalesman = $this->isSalesman($user);
-    if ($isSalesman)
-      $peopleRole[] = 'salesman';
+    if ($isSalesman) {
+          $peopleRole[] = 'salesman';
+    }
 
 
     return array_values(array_unique(empty($peopleRole) ? ['guest'] : $peopleRole));
@@ -110,15 +119,18 @@ 
   public function getMainCompany(): People
   {
 
-    if (self::$mainCompany) return self::$mainCompany;
+    if (self::$mainCompany) {
+     return self::$mainCompany;
+    }
     
     $domain  = $this->domainService->getMainDomain();
     $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)
       );
+    }
 
     self::$mainCompany =  $company->getPeople();
     return self::$mainCompany;

src/Controller/GetMyCompaniesAction.php

Braces +3 added lines, -2 removed lines

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

src/Service/PeopleService.php

Braces +6 added lines, -5 removed lines

@@ -33,9 +33,9 @@ 
     $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);
@@ -63,8 +63,9 @@ 
       'link_type' => $link_type
     ]);
 
-    if (!$peopleLink)
-      $peopleLink = new PeopleLink();
+    if (!$peopleLink) {
+          $peopleLink = new PeopleLink();
+    }
 
     $peopleLink->setCompany($company);
     $peopleLink->setPeople($people);