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

Test Failed
Push — datatable-single-component ( 4fcf13...78891a )
by Pedro
16:24
created
src/app/Library/CrudPanel/CrudColumn.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -87,7 +87,7 @@
 block discarded – undo
87 87
      */
88 88
     public function key(string $key)
89 89
     {
90
-        if (! isset($this->attributes['name'])) {
90
+        if (!isset($this->attributes['name'])) {
91 91
             abort(500, 'Column name must be defined before changing the key.', ['developer-error-exception']);
92 92
         }
93 93
 
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/Traits/Filters.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
         $this->enableFilters();
73 73
 
74 74
         // check if another filter with the same name exists
75
-        if (! isset($options['name'])) {
75
+        if (!isset($options['name'])) {
76 76
             abort(500, 'All your filters need names.', ['developer-error-exception']);
77 77
         }
78 78
 
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
     {
121 121
         $unappliedFilters = $this->filters()->where('applied', false);
122 122
         if ($unappliedFilters->count()) {
123
-            $unappliedFilters->each(function ($filter) {
123
+            $unappliedFilters->each(function($filter) {
124 124
                 $filter->apply();
125 125
             });
126 126
         }
@@ -167,7 +167,7 @@  discard block
 block discarded – undo
167 167
     {
168 168
         $filter = $this->filters()->firstWhere('name', $name);
169 169
 
170
-        if (! $filter) {
170
+        if (!$filter) {
171 171
             abort(500, 'CRUD Filter "'.$name.'" not found. Please check the filter exists before you modify it.', ['developer-error-exception']);
172 172
         }
173 173
 
@@ -182,7 +182,7 @@  discard block
 block discarded – undo
182 182
 
183 183
     public function replaceFilter($name, $newFilter)
184 184
     {
185
-        $newFilters = $this->filters()->map(function ($filter, $key) use ($name, $newFilter) {
185
+        $newFilters = $this->filters()->map(function($filter, $key) use ($name, $newFilter) {
186 186
             if ($filter->name != $name) {
187 187
                 return $filter;
188 188
             }
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
 
196 196
     public function removeFilter($name)
197 197
     {
198
-        $strippedCollection = $this->filters()->reject(function ($filter) use ($name) {
198
+        $strippedCollection = $this->filters()->reject(function($filter) use ($name) {
199 199
             return $filter->name == $name;
200 200
         });
201 201
 
@@ -238,7 +238,7 @@  discard block
 block discarded – undo
238 238
      */
239 239
     public function makeFirstFilter()
240 240
     {
241
-        if (! $this->filters()) {
241
+        if (!$this->filters()) {
242 242
             return false;
243 243
         }
244 244
 
@@ -268,15 +268,15 @@  discard block
 block discarded – undo
268 268
         $destinationFilter = $this->firstFilterWhere('name', $destination);
269 269
         $destinationKey = $this->getFilterKey($destination);
270 270
         $newDestinationKey = ($where == 'before' ? $destinationKey : $destinationKey + 1);
271
-        $newFilters = $this->filters()->filter(function ($value, $key) use ($target) {
271
+        $newFilters = $this->filters()->filter(function($value, $key) use ($target) {
272 272
             return $value->name != $target;
273 273
         });
274 274
 
275
-        if (! $targetFilter) {
275
+        if (!$targetFilter) {
276 276
             return;
277 277
         }
278 278
 
279
-        if (! $destinationFilter) {
279
+        if (!$destinationFilter) {
280 280
             return;
281 281
         }
282 282
 
@@ -284,7 +284,7 @@  discard block
 block discarded – undo
284 284
         $lastSlice = $newFilters->slice($newDestinationKey, null);
285 285
 
286 286
         $newFilters = $firstSlice->push($targetFilter);
287
-        $lastSlice->each(function ($item, $key) use ($newFilters) {
287
+        $lastSlice->each(function($item, $key) use ($newFilters) {
288 288
             $newFilters->push($item);
289 289
         });
290 290
 
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/Traits/FieldsProtectedMethods.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -115,7 +115,7 @@  discard block
 block discarded – undo
115 115
             return ['name' => Str::replace(' ', '', $field)];
116 116
         }
117 117
 
118
-        if (is_array($field) && ! isset($field['name'])) {
118
+        if (is_array($field) && !isset($field['name'])) {
119 119
             abort(500, 'All fields must have their name defined', ['developer-error-exception']);
120 120
         }
121 121
 
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
         }
194 194
         // if there's a model defined, but no attribute
195 195
         // guess an attribute using the identifiableAttribute functionality in CrudTrait
196
-        if (isset($field['model']) && ! isset($field['attribute']) && method_exists($field['model'], 'identifiableAttribute')) {
196
+        if (isset($field['model']) && !isset($field['attribute']) && method_exists($field['model'], 'identifiableAttribute')) {
197 197
             $field['attribute'] = (new $field['model']())->identifiableAttribute();
198 198
         }
199 199
 
@@ -209,7 +209,7 @@  discard block
 block discarded – undo
209 209
      */
210 210
     protected function makeSureFieldHasLabel($field)
211 211
     {
212
-        if (! isset($field['label'])) {
212
+        if (!isset($field['label'])) {
213 213
             $name = str_replace(',', ' ', $field['name']);
214 214
             $name = str_replace('_id', '', $name);
215 215
             $field['label'] = mb_ucfirst(str_replace('_', ' ', $name));
@@ -227,7 +227,7 @@  discard block
 block discarded – undo
227 227
      */
228 228
     protected function makeSureFieldHasType($field)
229 229
     {
230
-        if (! isset($field['type'])) {
230
+        if (!isset($field['type'])) {
231 231
             $field['type'] = isset($field['relation_type']) ? $this->inferFieldTypeFromRelationType($field['relation_type']) : $this->inferFieldTypeFromDbColumnType($field['name']);
232 232
         }
233 233
 
@@ -260,16 +260,16 @@  discard block
 block discarded – undo
260 260
      */
261 261
     protected function makeSureSubfieldsHaveNecessaryAttributes($field)
262 262
     {
263
-        if (! isset($field['subfields']) || ! is_array($field['subfields'])) {
263
+        if (!isset($field['subfields']) || !is_array($field['subfields'])) {
264 264
             return $field;
265 265
         }
266 266
 
267
-        if (! is_multidimensional_array($field['subfields'], true)) {
267
+        if (!is_multidimensional_array($field['subfields'], true)) {
268 268
             abort(500, 'Subfields of «'.$field['name'].'» are malformed. Make sure you provide an array of subfields.', ['developer-error-exception']);
269 269
         }
270 270
 
271 271
         foreach ($field['subfields'] as $key => $subfield) {
272
-            if (empty($subfield) || ! isset($subfield['name'])) {
272
+            if (empty($subfield) || !isset($subfield['name'])) {
273 273
                 abort(500, 'A subfield of «'.$field['name'].'» is malformed. Subfield attribute name can\'t be empty.', ['developer-error-exception']);
274 274
             }
275 275
 
@@ -282,7 +282,7 @@  discard block
 block discarded – undo
282 282
 
283 283
             $subfield['parentFieldName'] = $field['name'];
284 284
 
285
-            if (! isset($field['model'])) {
285
+            if (!isset($field['model'])) {
286 286
                 // we're inside a simple 'repeatable' with no model/relationship, so
287 287
                 // we assume all subfields are supposed to be text fields
288 288
                 $subfield['type'] = $subfield['type'] ?? 'text';
@@ -309,11 +309,11 @@  discard block
 block discarded – undo
309 309
                 case 'BelongsToMany':
310 310
                     $pivotSelectorField = static::getPivotFieldStructure($field);
311 311
 
312
-                    $pivot = Arr::where($field['subfields'], function ($item) use ($pivotSelectorField) {
312
+                    $pivot = Arr::where($field['subfields'], function($item) use ($pivotSelectorField) {
313 313
                         return $item['name'] === $pivotSelectorField['name'];
314 314
                     });
315 315
 
316
-                    if (! empty($pivot)) {
316
+                    if (!empty($pivot)) {
317 317
                         break;
318 318
                     }
319 319
 
@@ -337,11 +337,11 @@  discard block
 block discarded – undo
337 337
                     $entity = isset($field['baseEntity']) ? $field['baseEntity'].'.'.$field['entity'] : $field['entity'];
338 338
                     $relationInstance = $this->getRelationInstance(['entity' => $entity]);
339 339
 
340
-                    $localKeyField = Arr::where($field['subfields'], function ($item) use ($relationInstance) {
340
+                    $localKeyField = Arr::where($field['subfields'], function($item) use ($relationInstance) {
341 341
                         return $item['name'] === $relationInstance->getRelated()->getKeyName();
342 342
                     });
343 343
 
344
-                    if (! empty($localKeyField)) {
344
+                    if (!empty($localKeyField)) {
345 345
                         break;
346 346
                     }
347 347
 
@@ -366,7 +366,7 @@  discard block
 block discarded – undo
366 366
     {
367 367
         // if a tab was mentioned, we should enable it
368 368
         if (isset($field['tab'])) {
369
-            if (! $this->tabsEnabled()) {
369
+            if (!$this->tabsEnabled()) {
370 370
                 $this->enableTabs();
371 371
             }
372 372
         }
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/Traits/Buttons.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
         $otherButtons = collect([]);
25 25
 
26 26
         // we get the buttons that belong to the specified stack
27
-        $stackButtons = $this->buttons()->reject(function ($item) use ($stack, $otherButtons) {
27
+        $stackButtons = $this->buttons()->reject(function($item) use ($stack, $otherButtons) {
28 28
             if ($item->stack != $stack) {
29 29
                 // if the button does not belong to this stack we just add it for merging later
30 30
                 $otherButtons->push($item);
@@ -36,8 +36,8 @@  discard block
 block discarded – undo
36 36
         });
37 37
 
38 38
         // we parse the ordered buttons
39
-        collect($order)->each(function ($btnKey) use ($newButtons, $stackButtons) {
40
-            if (! $button = $stackButtons->where('name', $btnKey)->first()) {
39
+        collect($order)->each(function($btnKey) use ($newButtons, $stackButtons) {
40
+            if (!$button = $stackButtons->where('name', $btnKey)->first()) {
41 41
                 abort(500, 'Button name [«'.$btnKey.'»] not found.', ['developer-error-exception']);
42 42
             }
43 43
             $newButtons->push($button);
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
         // we add the remaining buttons to the end of the ordered ones
48 48
         if (count($newButtons) < count($stackButtons)) {
49 49
             foreach ($stackButtons as $button) {
50
-                if (! $newButtons->where('name', $button->name)->first()) {
50
+                if (!$newButtons->where('name', $button->name)->first()) {
51 51
                     $newButtons->push($button);
52 52
                 }
53 53
             }
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
          */
117 117
         $button = $this->buttons()->firstWhere('name', $name);
118 118
 
119
-        if (! $button) {
119
+        if (!$button) {
120 120
             abort(500, 'CRUD Button "'.$name.'" not found. Please ensure the button exists before you modify it.', ['developer-error-exception']);
121 121
         }
122 122
 
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
      */
138 138
     public function removeButton($name, $stack = null)
139 139
     {
140
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($name, $stack) {
140
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($name, $stack) {
141 141
             return $stack == null ? $button->name == $name : ($button->stack == $stack) && ($button->name == $name);
142 142
         }));
143 143
     }
@@ -148,7 +148,7 @@  discard block
 block discarded – undo
148 148
      */
149 149
     public function removeButtons($names, $stack = null)
150 150
     {
151
-        if (! empty($names)) {
151
+        if (!empty($names)) {
152 152
             foreach ($names as $name) {
153 153
                 $this->removeButton($name, $stack);
154 154
             }
@@ -162,14 +162,14 @@  discard block
 block discarded – undo
162 162
 
163 163
     public function removeAllButtonsFromStack($stack)
164 164
     {
165
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($stack) {
165
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($stack) {
166 166
             return $button->stack == $stack;
167 167
         }));
168 168
     }
169 169
 
170 170
     public function removeButtonFromStack($name, $stack)
171 171
     {
172
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($name, $stack) {
172
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($name, $stack) {
173 173
             return $button->name == $name && $button->stack == $stack;
174 174
         }));
175 175
     }
@@ -190,15 +190,15 @@  discard block
 block discarded – undo
190 190
         $destinationKey = $this->getButtonKey($destination);
191 191
         $newDestinationKey = ($where == 'before' ? $destinationKey : $destinationKey + 1);
192 192
 
193
-        $newButtons = $this->buttons()->filter(function ($value, $key) use ($target) {
193
+        $newButtons = $this->buttons()->filter(function($value, $key) use ($target) {
194 194
             return $value->name != $target;
195 195
         });
196 196
 
197
-        if (! $targetButton) {
197
+        if (!$targetButton) {
198 198
             return;
199 199
         }
200 200
 
201
-        if (! $destinationButton) {
201
+        if (!$destinationButton) {
202 202
             return;
203 203
         }
204 204
 
@@ -207,7 +207,7 @@  discard block
 block discarded – undo
207 207
 
208 208
         $newButtons = $firstSlice->push($targetButton);
209 209
 
210
-        $lastSlice->each(function ($item, $key) use ($newButtons) {
210
+        $lastSlice->each(function($item, $key) use ($newButtons) {
211 211
             $newButtons->push($item);
212 212
         });
213 213
 
@@ -266,7 +266,7 @@  discard block
 block discarded – undo
266 266
     /**
267 267
      * Add a new button to the current CRUD operation.
268 268
      */
269
-    public function button(string|array $nameOrAttributes): CrudButton
269
+    public function button(string | array $nameOrAttributes): CrudButton
270 270
     {
271 271
         return new CrudButton($nameOrAttributes);
272 272
     }
Please login to merge, or discard this patch.
src/app/Http/Controllers/CrudController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -34,8 +34,8 @@  discard block
 block discarded – undo
34 34
         //
35 35
         // It's done inside a middleware closure in order to have
36 36
         // the complete request inside the CrudPanel object.
37
-        $this->middleware(function ($request, $next) {
38
-            if (! Backpack::hasCrudController(get_class($this))) {
37
+        $this->middleware(function($request, $next) {
38
+            if (!Backpack::hasCrudController(get_class($this))) {
39 39
                 $this->initializeCrud($request);
40 40
             }
41 41
 
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
     protected function setupConfigurationForCurrentOperation(?string $operation = null)
104 104
     {
105 105
         $operationName = $operation ?? $this->crud->getCurrentOperation();
106
-        if (! $operationName) {
106
+        if (!$operationName) {
107 107
             return;
108 108
         }
109 109
 
Please login to merge, or discard this patch.
src/BackpackServiceProvider.php 1 patch
Spacing   +15 added lines, -17 removed lines patch added patch discarded remove patch
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
         $this->registerBackpackErrorViews();
86 86
 
87 87
         // Bind the CrudPanel object to Laravel's service container
88
-        $this->app->bind('crud', function ($app) {
88
+        $this->app->bind('crud', function($app) {
89 89
             // loop the stack trace to find the CrudControllerContract that called this method
90 90
             $trace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS);
91 91
             $controller = null;
@@ -95,31 +95,31 @@  discard block
 block discarded – undo
95 95
                     break;
96 96
                 }
97 97
             }
98
-            if (! $controller) {
98
+            if (!$controller) {
99 99
                 throw new \Exception('Could not identify the crud controller method. You sure you are calling this from a CrudController?');
100 100
             }
101 101
 
102 102
             return Backpack::getControllerCrud($controller);
103 103
         });
104 104
 
105
-        $this->app->scoped('backpack-manager', function ($app) {
105
+        $this->app->scoped('backpack-manager', function($app) {
106 106
             return new BackpackManager();
107 107
         });
108 108
 
109
-        $this->app->scoped('DatabaseSchema', function ($app) {
109
+        $this->app->scoped('DatabaseSchema', function($app) {
110 110
             return new DatabaseSchema();
111 111
         });
112 112
 
113
-        $this->app->singleton('BackpackViewNamespaces', function ($app) {
113
+        $this->app->singleton('BackpackViewNamespaces', function($app) {
114 114
             return new ViewNamespaces();
115 115
         });
116 116
 
117 117
         // Bind the widgets collection object to Laravel's service container
118
-        $this->app->singleton('widgets', function ($app) {
118
+        $this->app->singleton('widgets', function($app) {
119 119
             return new Collection();
120 120
         });
121 121
 
122
-        $this->app->scoped('UploadersRepository', function ($app) {
122
+        $this->app->scoped('UploadersRepository', function($app) {
123 123
             return new UploadersRepository();
124 124
         });
125 125
 
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
         $middleware_key = config('backpack.base.middleware_key');
136 136
         $middleware_class = config('backpack.base.middleware_class');
137 137
 
138
-        if (! is_array($middleware_class)) {
138
+        if (!is_array($middleware_class)) {
139 139
             $router->pushMiddlewareToGroup($middleware_key, $middleware_class);
140 140
 
141 141
             return;
@@ -250,7 +250,7 @@  discard block
 block discarded – undo
250 250
         $configs = scandir(__DIR__."/config/backpack/$dir/");
251 251
         $configs = array_diff($configs, ['.', '..']);
252 252
 
253
-        if (! count($configs)) {
253
+        if (!count($configs)) {
254 254
             return;
255 255
         }
256 256
 
@@ -299,8 +299,7 @@  discard block
 block discarded – undo
299 299
         // add the backpack_users password broker to the configuration
300 300
         $laravelAuthPasswordBrokers = app()->config['auth.passwords'];
301 301
         $laravelFirstPasswordBroker = is_array($laravelAuthPasswordBrokers) && current($laravelAuthPasswordBrokers) ?
302
-                                        current($laravelAuthPasswordBrokers)['table'] :
303
-                                        '';
302
+                                        current($laravelAuthPasswordBrokers)['table'] : '';
304 303
 
305 304
         $backpackPasswordBrokerTable = config('backpack.base.password_resets_table') ??
306 305
                                         config('auth.passwords.users.table') ??
@@ -328,7 +327,7 @@  discard block
 block discarded – undo
328 327
 
329 328
     public function loadViewComponents()
330 329
     {
331
-        $this->app->afterResolving(BladeCompiler::class, function () {
330
+        $this->app->afterResolving(BladeCompiler::class, function() {
332 331
             Blade::componentNamespace('Backpack\\CRUD\\app\\View\\Components', 'backpack');
333 332
         });
334 333
     }
@@ -354,13 +353,13 @@  discard block
 block discarded – undo
354 353
     private function registerBackpackErrorViews()
355 354
     {
356 355
         // register the backpack error when the exception handler is resolved from the container
357
-        $this->callAfterResolving(ExceptionHandler::class, function ($handler) {
358
-            if (! Str::startsWith(request()->path(), config('backpack.base.route_prefix'))) {
356
+        $this->callAfterResolving(ExceptionHandler::class, function($handler) {
357
+            if (!Str::startsWith(request()->path(), config('backpack.base.route_prefix'))) {
359 358
                 return;
360 359
             }
361 360
 
362 361
             // parse the namespaces set in config
363
-            [$themeNamespace, $themeFallbackNamespace] = (function () {
362
+            [$themeNamespace, $themeFallbackNamespace] = (function() {
364 363
                 $themeNamespace = config('backpack.ui.view_namespace');
365 364
                 $themeFallbackNamespace = config('backpack.ui.view_namespace_fallback');
366 365
 
@@ -377,8 +376,7 @@  discard block
 block discarded – undo
377 376
             // - fallback theme paths
378 377
             // - ui path
379 378
             $themeErrorPaths = $viewFinderHints[$themeNamespace] ?? [];
380
-            $themeErrorPaths = $themeNamespace === $themeFallbackNamespace ? $themeErrorPaths :
381
-                array_merge($viewFinderHints[$themeFallbackNamespace] ?? [], $themeErrorPaths);
379
+            $themeErrorPaths = $themeNamespace === $themeFallbackNamespace ? $themeErrorPaths : array_merge($viewFinderHints[$themeFallbackNamespace] ?? [], $themeErrorPaths);
382 380
             $uiErrorPaths = [base_path('vendor/backpack/crud/src/resources/views/ui')];
383 381
             $themeErrorPaths = array_merge($themeErrorPaths, $uiErrorPaths);
384 382
 
Please login to merge, or discard this patch.