Passed
Push — master ( 525ba8...c3ad1f )
by Jonathan
19:02 queued 09:34
created
app/Helpers/Uccello.php 2 patches
Spacing   +26 added lines, -26 removed lines patch added patch discarded remove patch
@@ -234,26 +234,26 @@  discard block
 block discarded – undo
234 234
 
235 235
         if (is_numeric($nameOrId)) {
236 236
             // Use cache
237
-            $modules = Cache::rememberForever('modules_by_id', function () {
237
+            $modules = Cache::rememberForever('modules_by_id', function() {
238 238
                 $modulesGroupedById = collect();
239 239
                 Module::all()->map(function($item) use($modulesGroupedById) {
240
-                    $modulesGroupedById[$item->id] = $item;
240
+                    $modulesGroupedById[ $item->id ] = $item;
241 241
                     return $modulesGroupedById;
242 242
                 });
243 243
                 return $modulesGroupedById;
244 244
             });
245
-            return $modules[(string) $nameOrId] ?? null;
245
+            return $modules[ (string)$nameOrId ] ?? null;
246 246
         } else {
247 247
             // Use cache
248
-            $modules = Cache::rememberForever('modules_by_name', function () {
248
+            $modules = Cache::rememberForever('modules_by_name', function() {
249 249
                 $modulesGroupedByName = collect();
250 250
                 Module::all()->map(function($item) use($modulesGroupedByName) {
251
-                    $modulesGroupedByName[$item->name] = $item;
251
+                    $modulesGroupedByName[ $item->name ] = $item;
252 252
                     return $modulesGroupedByName;
253 253
                 });
254 254
                 return $modulesGroupedByName;
255 255
             });
256
-            return $modules[(string) $nameOrId] ?? null;
256
+            return $modules[ (string)$nameOrId ] ?? null;
257 257
         }
258 258
     }
259 259
 
@@ -271,26 +271,26 @@  discard block
 block discarded – undo
271 271
 
272 272
         if (is_numeric($nameOrId)) {
273 273
             // Use cache
274
-            $uitypes = Cache::rememberForever('uitypes_by_id', function () {
274
+            $uitypes = Cache::rememberForever('uitypes_by_id', function() {
275 275
                 $uitypesGroupedById = collect();
276 276
                 Uitype::all()->map(function($item) use($uitypesGroupedById) {
277
-                    $uitypesGroupedById[$item->id] = $item;
277
+                    $uitypesGroupedById[ $item->id ] = $item;
278 278
                     return $uitypesGroupedById;
279 279
                 });
280 280
                 return $uitypesGroupedById;
281 281
             });
282
-            return $uitypes[(string) $nameOrId] ?? null;
282
+            return $uitypes[ (string)$nameOrId ] ?? null;
283 283
         } else {
284 284
             // Use cache
285
-            $uitypes = Cache::rememberForever('uitypes_by_name', function () {
285
+            $uitypes = Cache::rememberForever('uitypes_by_name', function() {
286 286
                 $uitypesGroupedByName = collect();
287 287
                 Uitype::all()->map(function($item) use($uitypesGroupedByName) {
288
-                    $uitypesGroupedByName[$item->name] = $item;
288
+                    $uitypesGroupedByName[ $item->name ] = $item;
289 289
                     return $uitypesGroupedByName;
290 290
                 });
291 291
                 return $uitypesGroupedByName;
292 292
             });
293
-            return $uitypes[(string) $nameOrId] ?? null;
293
+            return $uitypes[ (string)$nameOrId ] ?? null;
294 294
         }
295 295
     }
296 296
 
@@ -308,26 +308,26 @@  discard block
 block discarded – undo
308 308
 
309 309
         if (is_numeric($nameOrId)) {
310 310
             // Use cache
311
-            $displaytypes = Cache::rememberForever('displaytypes_by_id', function () {
311
+            $displaytypes = Cache::rememberForever('displaytypes_by_id', function() {
312 312
                 $displaytypesGroupedById = collect();
313 313
                 Displaytype::all()->map(function($item) use($displaytypesGroupedById) {
314
-                    $displaytypesGroupedById[$item->id] = $item;
314
+                    $displaytypesGroupedById[ $item->id ] = $item;
315 315
                     return $displaytypesGroupedById;
316 316
                 });
317 317
                 return $displaytypesGroupedById;
318 318
             });
319
-            return $displaytypes[(string) $nameOrId] ?? null;
319
+            return $displaytypes[ (string)$nameOrId ] ?? null;
320 320
         } else {
321 321
             // Use cache
322
-            $displaytypes = Cache::rememberForever('displaytypes_by_name', function () {
322
+            $displaytypes = Cache::rememberForever('displaytypes_by_name', function() {
323 323
                 $displaytypesGroupedByName = collect();
324 324
                 Displaytype::all()->map(function($item) use($displaytypesGroupedByName) {
325
-                    $displaytypesGroupedByName[$item->name] = $item;
325
+                    $displaytypesGroupedByName[ $item->name ] = $item;
326 326
                     return $displaytypesGroupedByName;
327 327
                 });
328 328
                 return $displaytypesGroupedByName;
329 329
             });
330
-            return $displaytypes[(string) $nameOrId] ?? null;
330
+            return $displaytypes[ (string)$nameOrId ] ?? null;
331 331
         }
332 332
     }
333 333
 
@@ -345,26 +345,26 @@  discard block
 block discarded – undo
345 345
 
346 346
         if (is_numeric($nameOrId)) {
347 347
             // Use cache
348
-            $capabilities = Cache::rememberForever('capabilities_by_id', function () {
348
+            $capabilities = Cache::rememberForever('capabilities_by_id', function() {
349 349
                 $capabilitiesGroupedById = collect();
350 350
                 Capability::all()->map(function($item) use($capabilitiesGroupedById) {
351
-                    $capabilitiesGroupedById[$item->id] = $item;
351
+                    $capabilitiesGroupedById[ $item->id ] = $item;
352 352
                     return $capabilitiesGroupedById;
353 353
                 });
354 354
                 return $capabilitiesGroupedById;
355 355
             });
356
-            return $capabilities[(string) $nameOrId] ?? null;
356
+            return $capabilities[ (string)$nameOrId ] ?? null;
357 357
         } else {
358 358
             // Use cache
359
-            $capabilities = Cache::rememberForever('capabilities_by_name', function () {
359
+            $capabilities = Cache::rememberForever('capabilities_by_name', function() {
360 360
                 $capabilitiesGroupedByName = collect();
361 361
                 Capability::all()->map(function($item) use($capabilitiesGroupedByName) {
362
-                    $capabilitiesGroupedByName[$item->name] = $item;
362
+                    $capabilitiesGroupedByName[ $item->name ] = $item;
363 363
                     return $capabilitiesGroupedByName;
364 364
                 });
365 365
                 return $capabilitiesGroupedByName;
366 366
             });
367
-            return $capabilities[(string) $nameOrId] ?? null;
367
+            return $capabilities[ (string)$nameOrId ] ?? null;
368 368
         }
369 369
     }
370 370
 
@@ -406,7 +406,7 @@  discard block
 block discarded – undo
406 406
      * @param string $type
407 407
      * @return array
408 408
      */
409
-    public function getDatatableColumns(Module $module, $filterId=null, $type='list'): array
409
+    public function getDatatableColumns(Module $module, $filterId = null, $type = 'list'): array
410 410
     {
411 411
         $columns = [ ];
412 412
 
@@ -492,7 +492,7 @@  discard block
 block discarded – undo
492 492
     {
493 493
         $entity = Entity::find($uid);
494 494
         
495
-        if($entity) {
495
+        if ($entity) {
496 496
             return $entity->record;
497 497
         }
498 498
         else {
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -494,8 +494,7 @@
 block discarded – undo
494 494
         
495 495
         if($entity) {
496 496
             return $entity->record;
497
-        }
498
-        else {
497
+        } else {
499 498
             return null;
500 499
         }
501 500
     }
Please login to merge, or discard this patch.
app/Fields/Uitype/AssignedUser.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -109,8 +109,8 @@
 block discarded – undo
109 109
      */
110 110
     public function addConditionToSearchQuery(Builder $query, Field $field, $value) : Builder
111 111
     {
112
-        $query->where(function ($query) use($field, $value) {
113
-            foreach ((array) $value as $_value) {
112
+        $query->where(function($query) use($field, $value) {
113
+            foreach ((array)$value as $_value) {
114 114
                 // Replace me by connected user's id
115 115
                 if ($_value === 'me') {
116 116
                     $_value = Auth::user()->uid;
Please login to merge, or discard this patch.
resources/views/modules/default/uitypes/search/assigned_user.blade.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -3,9 +3,9 @@
 block discarded – undo
3 3
         <?php
4 4
         $autocompleteSearch = false;
5 5
 
6
-        $entities = [];
6
+        $entities = [ ];
7 7
 
8
-        if (isset($column['data']->autocomplete_search) && $column['data']->autocomplete_search === true) {
8
+        if (isset($column[ 'data' ]->autocomplete_search) && $column[ 'data' ]->autocomplete_search === true) {
9 9
             $autocompleteSearch = true;
10 10
         } else {
11 11
             $entities = auth()->user()->getAllowedGroupsAndUsers(false);
Please login to merge, or discard this patch.
app/Models/User.php 1 patch
Spacing   +33 added lines, -33 removed lines patch added patch discarded remove patch
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
 
148 148
         $i = 0;
149 149
         foreach ($words as $w) {
150
-            $initials .= $w[0];
150
+            $initials .= $w[ 0 ];
151 151
             $i++;
152 152
 
153 153
             if ($i === 3) { // Maximum: 3 letters
@@ -168,7 +168,7 @@  discard block
 block discarded – undo
168 168
         $image = 'vendor/uccello/uccello/images/user-no-image.png';
169 169
 
170 170
         if ($this->avatarType === 'gravatar') {
171
-            $image = 'https://www.gravatar.com/avatar/' . md5($this->email) . '?d=mm';
171
+            $image = 'https://www.gravatar.com/avatar/'.md5($this->email).'?d=mm';
172 172
 
173 173
         } elseif ($this->avatarType === 'image' && !empty($this->avatar->path)) {
174 174
             $image = $this->avatar->path;
@@ -185,7 +185,7 @@  discard block
 block discarded – undo
185 185
      */
186 186
     public function rolesOnDomain($domain) : Collection
187 187
     {
188
-        return Cache::remember('domain_'.$domain->slug.'_roles', 600, function () use($domain) {
188
+        return Cache::remember('domain_'.$domain->slug.'_roles', 600, function() use($domain) {
189 189
             $roles = collect();
190 190
 
191 191
             if (config('uccello.roles.display_ancestors_roles')) {
@@ -233,7 +233,7 @@  discard block
 block discarded – undo
233 233
 
234 234
         $hasRole = false;
235 235
 
236
-        $descendants = Cache::remember('domain_'.$domain->slug.'_descendants', 600, function () use($domain) {
236
+        $descendants = Cache::remember('domain_'.$domain->slug.'_descendants', 600, function() use($domain) {
237 237
             return $domain->findDescendants()->get();
238 238
         });
239 239
 
@@ -259,7 +259,7 @@  discard block
 block discarded – undo
259 259
     {
260 260
         $keyName = 'user_'.$this->id.'_'.$domain->slug.'_'.$module->name.'_capabilities';
261 261
 
262
-        return Cache::remember($keyName, 600, function () use($domain, $module) {
262
+        return Cache::remember($keyName, 600, function() use($domain, $module) {
263 263
             $capabilities = collect();
264 264
 
265 265
             // Get the domain and all its parents
@@ -313,7 +313,7 @@  discard block
 block discarded – undo
313 313
 
314 314
         $keyName = 'user_'.$this->id.'_'.$domain->slug.'_can_access_to_settings_panel';
315 315
 
316
-        return Cache::remember($keyName, 600, function () use($domain) {
316
+        return Cache::remember($keyName, 600, function() use($domain) {
317 317
 
318 318
             $hasCapability = false;
319 319
 
@@ -465,8 +465,8 @@  discard block
 block discarded – undo
465 465
     {
466 466
         // Use cache
467 467
         $allowedGroups = Cache::rememberForever(
468
-            'allowed_groups_for_' . ($this->is_admin ? 'admin' : $this->getKey()), 
469
-            function () {
468
+            'allowed_groups_for_'.($this->is_admin ? 'admin' : $this->getKey()), 
469
+            function() {
470 470
                 return $this->getAllowedGroupUidsProcess();
471 471
             }
472 472
         );
@@ -478,8 +478,8 @@  discard block
 block discarded – undo
478 478
     {
479 479
         // Use cache
480 480
         $allowedGroupsAndUsers = Cache::rememberForever(
481
-            'allowed_group_users_for_' . ($addUsers ? 'u_' : '') . ($this->is_admin ? 'admin' : $this->getKey()), 
482
-            function () use ($addUsers) {
481
+            'allowed_group_users_for_'.($addUsers ? 'u_' : '').($this->is_admin ? 'admin' : $this->getKey()), 
482
+            function() use ($addUsers) {
483 483
                 return $this->getAllowedGroupsAndUsersProcess($addUsers);
484 484
             }
485 485
         );
@@ -489,16 +489,16 @@  discard block
 block discarded – undo
489 489
 
490 490
     protected function getAllowedGroupUidsProcess()
491 491
     {
492
-        $allowedUserUids = collect([$this->uid]);
492
+        $allowedUserUids = collect([ $this->uid ]);
493 493
 
494 494
         if ($this->is_admin) {
495 495
             $groups = Group::all();
496 496
         } else {
497
-            $groups = [];
498
-            $users = [];
497
+            $groups = [ ];
498
+            $users = [ ];
499 499
 
500 500
             foreach ($this->groups as $group) {
501
-                $groups[$group->uid] = $group;
501
+                $groups[ $group->uid ] = $group;
502 502
             };
503 503
 
504 504
             $this->addRecursiveChildrenGroups($groups, $users, $groups, false);
@@ -507,7 +507,7 @@  discard block
 block discarded – undo
507 507
         }
508 508
 
509 509
         foreach ($groups as $uid => $group) {
510
-            $allowedUserUids[] = $uid;
510
+            $allowedUserUids[ ] = $uid;
511 511
         }
512 512
 
513 513
         return $allowedUserUids;
@@ -516,19 +516,19 @@  discard block
 block discarded – undo
516 516
     protected function addRecursiveChildrenGroups(&$groups, &$users, $searchGroups, $addUsers = false)
517 517
     {
518 518
         foreach ($searchGroups as $uid => $searchGroup) {
519
-            $searchChildrenGroups = [];
519
+            $searchChildrenGroups = [ ];
520 520
 
521 521
             foreach ($searchGroup->childrenGroups as $childrenGroup) {
522
-                if (empty($groups[$childrenGroup->uid])) {
523
-                    $groups[$childrenGroup->uid] = $childrenGroup;
524
-                    $searchChildrenGroups[$childrenGroup->uid] = $childrenGroup;
522
+                if (empty($groups[ $childrenGroup->uid ])) {
523
+                    $groups[ $childrenGroup->uid ] = $childrenGroup;
524
+                    $searchChildrenGroups[ $childrenGroup->uid ] = $childrenGroup;
525 525
                 }
526 526
 
527
-                if($addUsers)
527
+                if ($addUsers)
528 528
                 {
529 529
                     foreach ($childrenGroup->users as $user) {
530
-                        if (empty($users[$user->uid])) {
531
-                            $users[$user->uid] = $user;
530
+                        if (empty($users[ $user->uid ])) {
531
+                            $users[ $user->uid ] = $user;
532 532
                         }
533 533
                     }
534 534
                 }
@@ -540,26 +540,26 @@  discard block
 block discarded – undo
540 540
 
541 541
     protected function getAllowedGroupsAndUsersProcess($addUsers = true)
542 542
     {
543
-        $allowedUserUids = collect([[
543
+        $allowedUserUids = collect([ [
544 544
             'uid' => $this->uid,
545 545
             'recordLabel' => uctrans('me', $this->module)
546
-        ]]);
546
+        ] ]);
547 547
 
548 548
         if ($this->is_admin) {
549 549
             $groups = Group::orderBy('name')->get();
550 550
             $users  = \App\User::orderBy('name')->get();
551 551
         } else {
552
-            $groups = [];
553
-            $users = [];
552
+            $groups = [ ];
553
+            $users = [ ];
554 554
 
555 555
             foreach ($this->groups as $group) {
556
-                $groups[$group->uid] = $group;
556
+                $groups[ $group->uid ] = $group;
557 557
 
558
-                if($addUsers)
558
+                if ($addUsers)
559 559
                 {
560 560
                     foreach ($group->users as $user) {
561
-                        if (empty($users[$user->uid])) {
562
-                            $users[$user->uid] = $user;
561
+                        if (empty($users[ $user->uid ])) {
562
+                            $users[ $user->uid ] = $user;
563 563
                         }
564 564
                     }
565 565
                 }
@@ -572,15 +572,15 @@  discard block
 block discarded – undo
572 572
         }
573 573
 
574 574
         foreach ($groups as $uid => $group) {
575
-            $allowedUserUids[] = [
575
+            $allowedUserUids[ ] = [
576 576
                 'uid' => $group->uid,
577 577
                 'recordLabel' => $group->recordLabel
578 578
             ];
579 579
         }
580 580
 
581 581
         foreach ($users as $uid => $user) {
582
-            if($user->getKey() != $this->getKey()) {
583
-                $allowedUserUids[] = [
582
+            if ($user->getKey() != $this->getKey()) {
583
+                $allowedUserUids[ ] = [
584 584
                     'uid' => $user->uid,
585 585
                     'recordLabel' => $user->recordLabel
586 586
                 ];
Please login to merge, or discard this patch.