Scrutinizer GitHub App not installed

We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.

Install GitHub App

Passed
Pull Request — main (#5317)
by Cristian
30:20 queued 15:27
created
src/app/Library/CrudPanel/Traits/Relationships.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
         $model = $this->getRelationModel($entity, -1);
66 66
         $lastSegmentAfterDot = Str::of($field['entity'])->afterLast('.');
67 67
 
68
-        if (! method_exists($model, $lastSegmentAfterDot)) {
68
+        if (!method_exists($model, $lastSegmentAfterDot)) {
69 69
             return (string) Str::of($field['entity'])->beforeLast('.');
70 70
         }
71 71
 
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
 
87 87
         return collect($this->getCleanStateFields())
88 88
             ->whereIn('relation_type', $relation_types)
89
-            ->filter(function ($item) use ($nested) {
89
+            ->filter(function($item) use ($nested) {
90 90
                 if ($nested) {
91 91
                     return true;
92 92
                 }
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
      */
131 131
     private function getRelationFieldsWithoutRelationType($relations, $fields = [])
132 132
     {
133
-        if (! is_array($relations)) {
133
+        if (!is_array($relations)) {
134 134
             $relations = [$relations];
135 135
         }
136 136
 
@@ -139,8 +139,8 @@  discard block
 block discarded – undo
139 139
         }
140 140
 
141 141
         foreach ($relations as $relation) {
142
-            $fields = array_filter($fields, function ($field) use ($relation) {
143
-                if (! isset($field['relation_type'])) {
142
+            $fields = array_filter($fields, function($field) use ($relation) {
143
+                if (!isset($field['relation_type'])) {
144 144
                     return false;
145 145
                 }
146 146
 
@@ -174,7 +174,7 @@  discard block
 block discarded – undo
174 174
                     $fields = array_merge($field['subfields'], $fields);
175 175
                 }
176 176
             }
177
-            $fields = array_filter($fields, function ($field) {
177
+            $fields = array_filter($fields, function($field) {
178 178
                 return isset($field['relation_type']) && $field['relation_type'] === 'BelongsTo';
179 179
             });
180 180
         }
@@ -241,8 +241,8 @@  discard block
 block discarded – undo
241 241
     {
242 242
         $all_relation_fields = $this->getRelationFields();
243 243
 
244
-        return Arr::where($all_relation_fields, function ($value, $key) {
245
-            return isset($value['pivot']) && ! $value['pivot'];
244
+        return Arr::where($all_relation_fields, function($value, $key) {
245
+            return isset($value['pivot']) && !$value['pivot'];
246 246
         });
247 247
     }
248 248
 
@@ -255,7 +255,7 @@  discard block
 block discarded – undo
255 255
     {
256 256
         $all_relation_fields = $this->getRelationFields();
257 257
 
258
-        return Arr::where($all_relation_fields, function ($value, $key) {
258
+        return Arr::where($all_relation_fields, function($value, $key) {
259 259
             return isset($value['pivot']) && $value['pivot'];
260 260
         });
261 261
     }
@@ -336,7 +336,7 @@  discard block
 block discarded – undo
336 336
         }
337 337
 
338 338
         // relationships are always public methods.
339
-        if (! $methodReflection->isPublic()) {
339
+        if (!$methodReflection->isPublic()) {
340 340
             return false;
341 341
         }
342 342
 
@@ -363,7 +363,7 @@  discard block
 block discarded – undo
363 363
      */
364 364
     public function isAttributeInRelationString(array $field): bool
365 365
     {
366
-        if (! str_contains($field['entity'], '.')) {
366
+        if (!str_contains($field['entity'], '.')) {
367 367
             return false;
368 368
         }
369 369
 
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/Traits/Support/MacroableWithAttributes.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@
 block discarded – undo
44 44
                                         ->filter(fn ($item) => isset($item[$macro]));
45 45
 
46 46
                 $subfieldsWithMacros->each(
47
-                    function ($item) use ($subfieldsWithMacros, $macro) {
47
+                    function($item) use ($subfieldsWithMacros, $macro) {
48 48
                         if ($subfieldsWithMacros->last() === $item) {
49 49
                             $this->{$macro}($item[$macro], $item);
50 50
                         } else {
Please login to merge, or discard this patch.
src/macros.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -19,8 +19,8 @@  discard block
 block discarded – undo
19 19
  *      - when true: `address[street]`
20 20
  *      - when false: `[address][street]`
21 21
  */
22
-if (! Str::hasMacro('dotsToSquareBrackets')) {
23
-    Str::macro('dotsToSquareBrackets', function ($string, $ignore = [], $keyFirst = true) {
22
+if (!Str::hasMacro('dotsToSquareBrackets')) {
23
+    Str::macro('dotsToSquareBrackets', function($string, $ignore = [], $keyFirst = true) {
24 24
         $stringParts = explode('.', $string);
25 25
         $result = '';
26 26
 
@@ -34,8 +34,8 @@  discard block
 block discarded – undo
34 34
         return $result;
35 35
     });
36 36
 }
37
-if (! CrudColumn::hasMacro('withFiles')) {
38
-    CrudColumn::macro('withFiles', function ($uploadDefinition = [], $subfield = null, $registerUploaderEvents = true) {
37
+if (!CrudColumn::hasMacro('withFiles')) {
38
+    CrudColumn::macro('withFiles', function($uploadDefinition = [], $subfield = null, $registerUploaderEvents = true) {
39 39
         $uploadDefinition = is_array($uploadDefinition) ? $uploadDefinition : [];
40 40
         /** @var CrudField|CrudColumn $this */
41 41
         RegisterUploadEvents::handle($this, $uploadDefinition, 'withFiles', $subfield, $registerUploaderEvents);
@@ -44,8 +44,8 @@  discard block
 block discarded – undo
44 44
     });
45 45
 }
46 46
 
47
-if (! CrudField::hasMacro('withFiles')) {
48
-    CrudField::macro('withFiles', function ($uploadDefinition = [], $subfield = null, $registerUploaderEvents = true) {
47
+if (!CrudField::hasMacro('withFiles')) {
48
+    CrudField::macro('withFiles', function($uploadDefinition = [], $subfield = null, $registerUploaderEvents = true) {
49 49
         $uploadDefinition = is_array($uploadDefinition) ? $uploadDefinition : [];
50 50
         /** @var CrudField|CrudColumn $this */
51 51
         RegisterUploadEvents::handle($this, $uploadDefinition, 'withFiles', $subfield, $registerUploaderEvents);
@@ -54,8 +54,8 @@  discard block
 block discarded – undo
54 54
     });
55 55
 }
56 56
 
57
-if (! CrudColumn::hasMacro('linkTo')) {
58
-    CrudColumn::macro('linkTo', function (string|array|Closure $routeOrConfiguration, ?array $parameters = []): static {
57
+if (!CrudColumn::hasMacro('linkTo')) {
58
+    CrudColumn::macro('linkTo', function(string | array | Closure $routeOrConfiguration, ?array $parameters = []): static {
59 59
         $wrapper = $this->attributes['wrapper'] ?? [];
60 60
         if (is_array($routeOrConfiguration)) {
61 61
             $route = $routeOrConfiguration['route'] ?? null;
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
         }
66 66
 
67 67
         if ($route instanceof Closure) {
68
-            $wrapper['href'] = function ($crud, $column, $entry, $related_key) use ($route) {
68
+            $wrapper['href'] = function($crud, $column, $entry, $related_key) use ($route) {
69 69
                 return $route($entry, $related_key, $column, $crud);
70 70
             };
71 71
             $this->wrapper($wrapper);
@@ -75,13 +75,13 @@  discard block
 block discarded – undo
75 75
 
76 76
         $routeInstance = Route::getRoutes()->getByName($route);
77 77
 
78
-        if (! $routeInstance) {
78
+        if (!$routeInstance) {
79 79
             throw new \Exception("Route [{$route}] not found while building the link for column [{$this->attributes['name']}].");
80 80
         }
81 81
 
82 82
         $expectedParameters = $routeInstance->parameterNames();
83 83
 
84
-        $parameters = (function () use ($parameters, $expectedParameters, $route) {
84
+        $parameters = (function() use ($parameters, $expectedParameters, $route) {
85 85
             if (count($expectedParameters) === 0) {
86 86
                 return $parameters;
87 87
             }
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
             if (count($autoInferedParameter) > 1) {
90 90
                 throw new \Exception("Route [{$route}] expects parameters [".implode(', ', $expectedParameters)."]. Insuficient parameters provided in column: [{$this->attributes['name']}].");
91 91
             }
92
-            $autoInferedParameter = current($autoInferedParameter) ? [current($autoInferedParameter) => function ($entry, $related_key, $column, $crud) {
92
+            $autoInferedParameter = current($autoInferedParameter) ? [current($autoInferedParameter) => function($entry, $related_key, $column, $crud) {
93 93
                 $entity = $crud->isAttributeInRelationString($column) ? Str::before($column['entity'], '.') : $column['entity'];
94 94
 
95 95
                 return $related_key ?? $entry->{$entity}?->getKey();
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
             return array_merge($autoInferedParameter, $parameters);
99 99
         })();
100 100
 
101
-        $wrapper['href'] = function ($crud, $column, $entry, $related_key) use ($route, $parameters) {
101
+        $wrapper['href'] = function($crud, $column, $entry, $related_key) use ($route, $parameters) {
102 102
             $parameters = collect($parameters)->map(fn ($item) => is_callable($item) ? $item($entry, $related_key, $column, $crud) : $item)->toArray();
103 103
 
104 104
             return route($route, $parameters);
@@ -115,8 +115,8 @@  discard block
 block discarded – undo
115 115
  *
116 116
  * It will go to the given CrudController and get the setupRoutes() method on it.
117 117
  */
118
-if (! Route::hasMacro('crud')) {
119
-    Route::macro('crud', function ($name, $controller) {
118
+if (!Route::hasMacro('crud')) {
119
+    Route::macro('crud', function($name, $controller) {
120 120
         // put together the route name prefix,
121 121
         // as passed to the Route::group() statements
122 122
         $routeName = '';
Please login to merge, or discard this patch.
tests/BaseTestClass.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@
 block discarded – undo
23 23
             (array) config('backpack.base.middleware_key', 'admin'),
24 24
             'prefix'     => config('backpack.base.route_prefix', 'admin'),
25 25
         ],
26
-            function () {
26
+            function() {
27 27
                 Route::get('articles/{id}/show/{detail}', ['as' => 'article.show.detail', 'action' => 'Backpack\CRUD\Tests\config\Http\Controllers\ArticleCrudController@detail']);
28 28
                 Route::crud('users', 'Backpack\CRUD\Tests\config\Http\Controllers\UserCrudController');
29 29
                 Route::crud('articles', 'Backpack\CRUD\Tests\config\Http\Controllers\ArticleCrudController');
Please login to merge, or discard this patch.