Passed
Push — master ( ddf312...be26c9 )
by Luiz Kim
02:13
created
src/Controller/CreateUserAction.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@
 block discarded – undo
26 26
      *
27 27
      * @var Request
28 28
      */
29
-    private $request  = null;
29
+    private $request = null;
30 30
 
31 31
     /**
32 32
      * Security
Please login to merge, or discard this patch.
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -119,8 +119,9 @@
 block discarded – undo
119 119
                 'id' => $user->getId()
120 120
             ];
121 121
         } catch (\Exception $e) {
122
-            if ($this->manager->getConnection()->isTransactionActive())
123
-                $this->manager->getConnection()->rollBack();
122
+            if ($this->manager->getConnection()->isTransactionActive()) {
123
+                            $this->manager->getConnection()->rollBack();
124
+            }
124 125
 
125 126
             throw new \InvalidArgumentException($e->getMessage());
126 127
         }
Please login to merge, or discard this patch.
src/Controller/DeleteUserAction.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
      *
27 27
      * @var Request
28 28
      */
29
-    private $request  = null;
29
+    private $request = null;
30 30
 
31 31
     /**
32 32
      * Security
@@ -96,12 +96,12 @@  discard block
 block discarded – undo
96 96
                 throw new \InvalidArgumentException('Document id is not defined');
97 97
             }
98 98
 
99
-            $users   = $this->manager->getRepository(User::class)->findBy(['people' => $person]);
99
+            $users = $this->manager->getRepository(User::class)->findBy(['people' => $person]);
100 100
             if (count($users) == 1) {
101 101
                 throw new \InvalidArgumentException('Deve existir pelo menos um usuário');
102 102
             }
103 103
 
104
-            $user    = $this->manager->getRepository(User::class)->findOneBy(['id' => $payload['id'], 'people' => $person]);
104
+            $user = $this->manager->getRepository(User::class)->findOneBy(['id' => $payload['id'], 'people' => $person]);
105 105
             if (!$user instanceof User) {
106 106
                 throw new \InvalidArgumentException('Person user was not found');
107 107
             }
Please login to merge, or discard this patch.
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -113,8 +113,9 @@
 block discarded – undo
113 113
 
114 114
             return true;
115 115
         } catch (\Exception $e) {
116
-            if ($this->manager->getConnection()->isTransactionActive())
117
-                $this->manager->getConnection()->rollBack();
116
+            if ($this->manager->getConnection()->isTransactionActive()) {
117
+                            $this->manager->getConnection()->rollBack();
118
+            }
118 119
 
119 120
             throw new \InvalidArgumentException($e->getMessage());
120 121
         }
Please login to merge, or discard this patch.
src/Repository/ClientRepository.php 1 patch
Spacing   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -112,31 +112,31 @@  discard block
 block discarded – undo
112 112
 
113 113
             case 'active':
114 114
 
115
-                $queryBuilder->innerJoin(SalesOrder::class,     'O'  , 'WITH', 'O.client = myclients.id');
116
-                $queryBuilder->innerJoin(People::class,         'C'  , 'WITH', 'C.id = O.provider');
115
+                $queryBuilder->innerJoin(SalesOrder::class, 'O', 'WITH', 'O.client = myclients.id');
116
+                $queryBuilder->innerJoin(People::class, 'C', 'WITH', 'C.id = O.provider');
117 117
                 $queryBuilder->innerJoin(PeopleSalesman::class, 'kkk', 'WITH', 'kkk.company = C.id');
118
-                $queryBuilder->innerJoin(People::class,         'S'  , 'WITH', 'S.id = kkk.salesman');
119
-                $queryBuilder->innerJoin(PeopleClient::class,   'PC' , 'WITH', 'PC.client = myclients.id AND PC.company_id = S.id');
118
+                $queryBuilder->innerJoin(People::class, 'S', 'WITH', 'S.id = kkk.salesman');
119
+                $queryBuilder->innerJoin(PeopleClient::class, 'PC', 'WITH', 'PC.client = myclients.id AND PC.company_id = S.id');
120 120
                 $queryBuilder->andWhere('O.provider = :provider');
121 121
                 $queryBuilder->andWhere('S.id IN (:my_companies)');
122 122
                 $queryBuilder->andWhere('O.status NOT IN (:statuses)');
123 123
                 $queryBuilder->andWhere('O.orderDate BETWEEN :from_date AND :to_date');
124
-                $queryBuilder->setParameter('provider'    , $provider);
125
-                $queryBuilder->setParameter('statuses'    , $this->getEntityManager()->getRepository(Status::class)->findBy(['realStatus' => ['open','canceled']]));
126
-                $queryBuilder->setParameter('from_date'   , $fromDate);
127
-                $queryBuilder->setParameter('to_date'     , $toDate);
124
+                $queryBuilder->setParameter('provider', $provider);
125
+                $queryBuilder->setParameter('statuses', $this->getEntityManager()->getRepository(Status::class)->findBy(['realStatus' => ['open', 'canceled']]));
126
+                $queryBuilder->setParameter('from_date', $fromDate);
127
+                $queryBuilder->setParameter('to_date', $toDate);
128 128
                 $queryBuilder->setParameter('my_companies', $this->getMyCompanies($salesman));
129 129
 
130 130
             break;
131 131
 
132 132
             case 'inactive':
133 133
 
134
-                $queryBuilder->innerJoin(SalesOrder::class,     'O'  , 'WITH', 'O.client = myclients.id');
135
-                $queryBuilder->innerJoin(People::class,         'C'  , 'WITH', 'C.id = O.provider');
134
+                $queryBuilder->innerJoin(SalesOrder::class, 'O', 'WITH', 'O.client = myclients.id');
135
+                $queryBuilder->innerJoin(People::class, 'C', 'WITH', 'C.id = O.provider');
136 136
                 $queryBuilder->innerJoin(PeopleSalesman::class, 'kkk', 'WITH', 'kkk.company = C.id');
137
-                $queryBuilder->innerJoin(People::class,         'S'  , 'WITH', 'S.id = kkk.salesman');
138
-                $queryBuilder->innerJoin(PeopleClient::class,   'PC' , 'WITH', 'PC.client = myclients.id AND PC.company_id = S.id');
139
-                $queryBuilder->leftJoin (SalesOrder::class,     'jj' , 'WITH', 'jj.client = myclients.id
137
+                $queryBuilder->innerJoin(People::class, 'S', 'WITH', 'S.id = kkk.salesman');
138
+                $queryBuilder->innerJoin(PeopleClient::class, 'PC', 'WITH', 'PC.client = myclients.id AND PC.company_id = S.id');
139
+                $queryBuilder->leftJoin(SalesOrder::class, 'jj', 'WITH', 'jj.client = myclients.id
140 140
                                                                              AND   (jj.status NOT IN (:statuses) OR jj.id IS NULL)
141 141
                                                                              AND   ((jj.orderDate  BETWEEN  :from_date AND :to_date) OR jj.id IS NULL)
142 142
                                                                              AND   (jj.provider = :provider  OR jj.id IS NULL)');
@@ -147,23 +147,23 @@  discard block
 block discarded – undo
147 147
                 $queryBuilder->groupBy('myclients.id');
148 148
                 $queryBuilder->having('COUNT(jj.id) > :count');
149 149
                 $queryBuilder->andHaving('COUNT(O.id) = :count');
150
-                $queryBuilder->setParameter('provider' , $provider);
151
-                $queryBuilder->setParameter('statuses' , $this->getEntityManager()->getRepository(Status::class)->findBy(['realStatus' => ['open','canceled']]));
150
+                $queryBuilder->setParameter('provider', $provider);
151
+                $queryBuilder->setParameter('statuses', $this->getEntityManager()->getRepository(Status::class)->findBy(['realStatus' => ['open', 'canceled']]));
152 152
                 $queryBuilder->setParameter('from_date', $fromDate);
153
-                $queryBuilder->setParameter('to_date'  , $toDate);
154
-                $queryBuilder->setParameter('count'    , 0);
153
+                $queryBuilder->setParameter('to_date', $toDate);
154
+                $queryBuilder->setParameter('count', 0);
155 155
                 $queryBuilder->setParameter('my_companies', $this->getMyCompanies($salesman));
156 156
 
157 157
             break;
158 158
 
159 159
             case 'prospect':
160 160
 
161
-                $queryBuilder->innerJoin(SalesOrder::class,     'O' , 'WITH', 'O.client = myclients.id');
162
-                $queryBuilder->innerJoin(People::class,         'C' , 'WITH', 'C.id = O.provider');
161
+                $queryBuilder->innerJoin(SalesOrder::class, 'O', 'WITH', 'O.client = myclients.id');
162
+                $queryBuilder->innerJoin(People::class, 'C', 'WITH', 'C.id = O.provider');
163 163
                 $queryBuilder->innerJoin(PeopleSalesman::class, 'PS', 'WITH', 'PS.company = C.id');
164
-                $queryBuilder->innerJoin(People::class,         'S' , 'WITH', 'S.id = PS.salesman');
165
-                $queryBuilder->innerJoin(PeopleClient::class,   'PC', 'WITH', 'PC.client = myclients.id AND PC.company_id = S.id');
166
-                $queryBuilder->leftJoin (SalesOrder::class,     'jj' , 'WITH', 'jj.client = myclients.id
164
+                $queryBuilder->innerJoin(People::class, 'S', 'WITH', 'S.id = PS.salesman');
165
+                $queryBuilder->innerJoin(PeopleClient::class, 'PC', 'WITH', 'PC.client = myclients.id AND PC.company_id = S.id');
166
+                $queryBuilder->leftJoin(SalesOrder::class, 'jj', 'WITH', 'jj.client = myclients.id
167 167
                                                                              AND   (jj.status NOT IN (:statuses) OR jj.id IS NULL)                                                                             
168 168
                                                                              AND   (jj.provider = :provider  OR jj.id IS NULL)');
169 169
                 $queryBuilder->andWhere('O.provider = :provider');
@@ -173,19 +173,19 @@  discard block
 block discarded – undo
173 173
                 $queryBuilder->having('COUNT(jj.id) = :count');
174 174
 
175 175
                 $queryBuilder->setParameter('provider', $provider);
176
-                $queryBuilder->setParameter('statuses', $this->getEntityManager()->getRepository(Status::class)->findBy(['realStatus' => ['open','canceled']]));
177
-                $queryBuilder->setParameter('count'   , 0);
176
+                $queryBuilder->setParameter('statuses', $this->getEntityManager()->getRepository(Status::class)->findBy(['realStatus' => ['open', 'canceled']]));
177
+                $queryBuilder->setParameter('count', 0);
178 178
                 $queryBuilder->setParameter('my_companies', $this->getMyCompanies($salesman));
179 179
 
180 180
             break;
181 181
 
182 182
             case 'new':
183 183
 
184
-                $queryBuilder->innerJoin(SalesOrder::class,     'O'  , 'WITH', 'O.client = myclients.id');
185
-                $queryBuilder->innerJoin(People::class,         'C'  , 'WITH', 'C.id = O.provider');
184
+                $queryBuilder->innerJoin(SalesOrder::class, 'O', 'WITH', 'O.client = myclients.id');
185
+                $queryBuilder->innerJoin(People::class, 'C', 'WITH', 'C.id = O.provider');
186 186
                 $queryBuilder->innerJoin(PeopleSalesman::class, 'kkk', 'WITH', 'kkk.company = C.id');
187
-                $queryBuilder->innerJoin(People::class,         'S'  , 'WITH', 'S.id = kkk.salesman');
188
-                $queryBuilder->innerJoin(PeopleClient::class,   'PC' , 'WITH', 'PC.client = myclients.id AND PC.company_id = S.id');
187
+                $queryBuilder->innerJoin(People::class, 'S', 'WITH', 'S.id = kkk.salesman');
188
+                $queryBuilder->innerJoin(PeopleClient::class, 'PC', 'WITH', 'PC.client = myclients.id AND PC.company_id = S.id');
189 189
 
190 190
                 $queryBuilder->andWhere('O.provider = :provider');
191 191
                 $queryBuilder->andWhere('S.id IN (:my_companies)');
@@ -193,10 +193,10 @@  discard block
 block discarded – undo
193 193
                 $queryBuilder->andWhere('O.orderDate    BETWEEN :from_date AND :to_date');
194 194
                 $queryBuilder->andWhere('myclients.registerDate BETWEEN :from_date AND :to_date');
195 195
 
196
-                $queryBuilder->setParameter('provider'    , $provider);
197
-                $queryBuilder->setParameter('statuses'    , $this->getEntityManager()->getRepository(Status::class)->findBy(['realStatus' => ['open','canceled']]));
198
-                $queryBuilder->setParameter('from_date'   , $fromDate);
199
-                $queryBuilder->setParameter('to_date'     , $toDate);
196
+                $queryBuilder->setParameter('provider', $provider);
197
+                $queryBuilder->setParameter('statuses', $this->getEntityManager()->getRepository(Status::class)->findBy(['realStatus' => ['open', 'canceled']]));
198
+                $queryBuilder->setParameter('from_date', $fromDate);
199
+                $queryBuilder->setParameter('to_date', $toDate);
200 200
                 $queryBuilder->setParameter('my_companies', $this->getMyCompanies($salesman));
201 201
 
202 202
             break;
Please login to merge, or discard this patch.
src/Entity/People.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -804,15 +804,15 @@  discard block
 block discarded – undo
804 804
         $this->enable = 0;
805 805
         $this->icms = 1;
806 806
         $this->billing = 0;
807
-        $this->registerDate =            new \DateTime('now');
808
-        $this->people =            new \Doctrine\Common\Collections\ArrayCollection();
809
-        $this->config =            new \Doctrine\Common\Collections\ArrayCollection();
810
-        $this->link =            new \Doctrine\Common\Collections\ArrayCollection();
811
-        $this->user =            new \Doctrine\Common\Collections\ArrayCollection();
812
-        $this->document =            new \Doctrine\Common\Collections\ArrayCollection();
813
-        $this->address =            new \Doctrine\Common\Collections\ArrayCollection();
814
-        $this->email =            new \Doctrine\Common\Collections\ArrayCollection();
815
-        $this->phone =            new \Doctrine\Common\Collections\ArrayCollection();
807
+        $this->registerDate = new \DateTime('now');
808
+        $this->people = new \Doctrine\Common\Collections\ArrayCollection();
809
+        $this->config = new \Doctrine\Common\Collections\ArrayCollection();
810
+        $this->link = new \Doctrine\Common\Collections\ArrayCollection();
811
+        $this->user = new \Doctrine\Common\Collections\ArrayCollection();
812
+        $this->document = new \Doctrine\Common\Collections\ArrayCollection();
813
+        $this->address = new \Doctrine\Common\Collections\ArrayCollection();
814
+        $this->email = new \Doctrine\Common\Collections\ArrayCollection();
815
+        $this->phone = new \Doctrine\Common\Collections\ArrayCollection();
816 816
         $this->billingDays = 'daily';
817 817
         $this->paymentTerm = 1;
818 818
 
@@ -1115,7 +1115,7 @@  discard block
 block discarded – undo
1115 1115
     }
1116 1116
     public function getOneDocument(): ?Document
1117 1117
     {
1118
-        $documents = $this->getDocument()->filter(function ($peopleDocument) {
1118
+        $documents = $this->getDocument()->filter(function($peopleDocument) {
1119 1119
             if ($peopleDocument->getPeople()->getPeopleType() == 'F') {
1120 1120
                 return $peopleDocument->getDocumentType()->getDocumentType() == 'CPF';
1121 1121
             }
Please login to merge, or discard this patch.