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 (#4267)
by Pedro
11:33
created
src/app/Library/CrudPanel/Traits/Update.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
         }
59 59
 
60 60
         // always have a hidden input for the entry id
61
-        if (! array_key_exists('id', $fields)) {
61
+        if (!array_key_exists('id', $fields)) {
62 62
             $fields['id'] = [
63 63
                 'name' => $entry->getKeyName(),
64 64
                 'value' => $entry->getKey(),
@@ -83,12 +83,12 @@  discard block
 block discarded – undo
83 83
         $fieldEntity = $field['entity'] ?? false;
84 84
         $fakeField = $field['fake'] ?? false;
85 85
 
86
-        if ($fieldEntity && ! $fakeField) {
86
+        if ($fieldEntity && !$fakeField) {
87 87
             return $this->getModelAttributeValueFromRelationship($model, $field);
88 88
         }
89 89
 
90 90
         if ($this->holdsMultipleInputs($field['name'])) {
91
-            $result = array_map(function ($item) use ($model) {
91
+            $result = array_map(function($item) use ($model) {
92 92
                 return $model->{$item};
93 93
             }, explode(',', $field['name']));
94 94
 
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
     {
111 111
         [$relatedModel, $relationMethod] = $this->getModelAndMethodFromEntity($model, $field);
112 112
 
113
-        if (! method_exists($relatedModel, $relationMethod)) {
113
+        if (!method_exists($relatedModel, $relationMethod)) {
114 114
             return $relatedModel->{$relationMethod};
115 115
         }
116 116
 
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
                     $model = $this->setLocaleOnModel($model);
130 130
                     // when subfields are NOT set we don't need to get any more values
131 131
                     // we just return the plain models as we only need the ids
132
-                    if (! isset($field['subfields'])) {
132
+                    if (!isset($field['subfields'])) {
133 133
                         $result->push($model);
134 134
 
135 135
                         continue;
@@ -157,13 +157,13 @@  discard block
 block discarded – undo
157 157
                 break;
158 158
             case 'HasOne':
159 159
             case 'MorphOne':
160
-                if (! method_exists($relatedModel, $relationMethod)) {
160
+                if (!method_exists($relatedModel, $relationMethod)) {
161 161
                     return;
162 162
                 }
163 163
 
164 164
                 $model = $relatedModel->{$relationMethod};
165 165
 
166
-                if (! $model) {
166
+                if (!$model) {
167 167
                     return;
168 168
                 }
169 169
 
@@ -226,7 +226,7 @@  discard block
 block discarded – undo
226 226
 
227 227
         $relationArray = explode('.', $relationEntity);
228 228
 
229
-        $relatedModel = array_reduce(array_splice($relationArray, 0, -1), function ($obj, $method) {
229
+        $relatedModel = array_reduce(array_splice($relationArray, 0, -1), function($obj, $method) {
230 230
             // if the string ends with `_id` we strip it out
231 231
             $method = Str::endsWith($method, '_id') ? Str::replaceLast('_id', '', $method) : $method;
232 232
 
@@ -253,7 +253,7 @@  discard block
 block discarded – undo
253 253
             // if the subfield name does not contain a dot we just need to check
254 254
             // if it has subfields and return the result accordingly.
255 255
             foreach ((array) $subfield['name'] as $name) {
256
-                if (! Str::contains($name, '.')) {
256
+                if (!Str::contains($name, '.')) {
257 257
                     // when subfields are present, $relatedModel->{$name} returns a model instance
258 258
                     // otherwise returns the model attribute.
259 259
                     if ($relatedModel->{$name} && isset($subfield['subfields'])) {
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/Traits/Read.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
     {
53 53
         $entry = $this->getCurrentEntry();
54 54
 
55
-        if (! $entry) {
55
+        if (!$entry) {
56 56
             return false;
57 57
         }
58 58
 
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
      */
68 68
     public function getEntry($id)
69 69
     {
70
-        if (! $this->entry) {
70
+        if (!$this->entry) {
71 71
             $this->entry = $this->getModelWithCrudPanelQuery()->findOrFail($id);
72 72
             $this->entry = $this->entry->withFakes();
73 73
         }
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
      */
90 90
     public function getEntryWithLocale($id)
91 91
     {
92
-        if (! $this->entry) {
92
+        if (!$this->entry) {
93 93
             $this->entry = $this->getEntry($id);
94 94
         }
95 95
 
@@ -171,7 +171,7 @@  discard block
 block discarded – undo
171 171
      */
172 172
     public function enableDetailsRow()
173 173
     {
174
-        if (! backpack_pro()) {
174
+        if (!backpack_pro()) {
175 175
             throw new BackpackProRequiredException('Details row');
176 176
         }
177 177
 
@@ -270,7 +270,7 @@  discard block
 block discarded – undo
270 270
                 // we will apply the same labels as the values to the menu if developer didn't
271 271
                 $this->abortIfInvalidPageLength($menu[0]);
272 272
 
273
-                if (! isset($menu[1]) || ! is_array($menu[1])) {
273
+                if (!isset($menu[1]) || !is_array($menu[1])) {
274 274
                     $menu[1] = $menu[0];
275 275
                 }
276 276
             } else {
@@ -358,7 +358,7 @@  discard block
 block discarded – undo
358 358
      */
359 359
     public function enableExportButtons()
360 360
     {
361
-        if (! backpack_pro()) {
361
+        if (!backpack_pro()) {
362 362
             throw new BackpackProRequiredException('Export buttons');
363 363
         }
364 364
 
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/CrudPanel.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -109,11 +109,11 @@  discard block
 block discarded – undo
109 109
      */
110 110
     public function setModel($model_namespace)
111 111
     {
112
-        if (! class_exists($model_namespace)) {
112
+        if (!class_exists($model_namespace)) {
113 113
             throw new Exception('The model does not exist.', 500);
114 114
         }
115 115
 
116
-        if (! method_exists($model_namespace, 'hasCrudTrait')) {
116
+        if (!method_exists($model_namespace, 'hasCrudTrait')) {
117 117
             throw new Exception('Please use CrudTrait on the model.', 500);
118 118
         }
119 119
 
@@ -206,7 +206,7 @@  discard block
 block discarded – undo
206 206
 
207 207
         $complete_route = $route.'.index';
208 208
 
209
-        if (! \Route::has($complete_route)) {
209
+        if (!\Route::has($complete_route)) {
210 210
             throw new Exception('There are no routes for this route name.', 404);
211 211
         }
212 212
 
@@ -304,7 +304,7 @@  discard block
 block discarded – undo
304 304
      */
305 305
     public function getFirstOfItsTypeInArray($type, $array)
306 306
     {
307
-        return Arr::first($array, function ($item) use ($type) {
307
+        return Arr::first($array, function($item) use ($type) {
308 308
             return $item['type'] == $type;
309 309
         });
310 310
     }
@@ -324,8 +324,8 @@  discard block
 block discarded – undo
324 324
      */
325 325
     public function sync($type, $fields, $attributes)
326 326
     {
327
-        if (! empty($this->{$type})) {
328
-            $this->{$type} = array_map(function ($field) use ($fields, $attributes) {
327
+        if (!empty($this->{$type})) {
328
+            $this->{$type} = array_map(function($field) use ($fields, $attributes) {
329 329
                 if (in_array($field['name'], (array) $fields)) {
330 330
                     $field = array_merge($field, $attributes);
331 331
                 }
@@ -355,15 +355,15 @@  discard block
 block discarded – undo
355 355
     {
356 356
         $relationArray = explode('.', $relationString);
357 357
 
358
-        if (! isset($length)) {
358
+        if (!isset($length)) {
359 359
             $length = count($relationArray);
360 360
         }
361 361
 
362
-        if (! isset($model)) {
362
+        if (!isset($model)) {
363 363
             $model = $this->model;
364 364
         }
365 365
 
366
-        $result = array_reduce(array_splice($relationArray, 0, $length), function ($obj, $method) {
366
+        $result = array_reduce(array_splice($relationArray, 0, $length), function($obj, $method) {
367 367
             try {
368 368
                 $result = $obj->$method();
369 369
 
@@ -399,7 +399,7 @@  discard block
 block discarded – undo
399 399
             if (is_array($entries)) {
400 400
                 //if attribute does not exist in main array we have more than one entry OR the attribute
401 401
                 //is an acessor that is not in $appends property of model.
402
-                if (! isset($entries[$attribute])) {
402
+                if (!isset($entries[$attribute])) {
403 403
                     //we first check if we don't have the attribute because it's an acessor that is not in appends.
404 404
                     if ($model_instance->hasGetMutator($attribute) && isset($entries[$modelKey])) {
405 405
                         $entry_in_database = $model_instance->find($entries[$modelKey]);
@@ -438,21 +438,21 @@  discard block
 block discarded – undo
438 438
      */
439 439
     public function parseTranslatableAttributes($model, $attribute, $value)
440 440
     {
441
-        if (! method_exists($model, 'isTranslatableAttribute')) {
441
+        if (!method_exists($model, 'isTranslatableAttribute')) {
442 442
             return $value;
443 443
         }
444 444
 
445
-        if (! $model->isTranslatableAttribute($attribute)) {
445
+        if (!$model->isTranslatableAttribute($attribute)) {
446 446
             return $value;
447 447
         }
448 448
 
449
-        if (! is_array($value)) {
449
+        if (!is_array($value)) {
450 450
             $decodedAttribute = json_decode($value, true) ?? ($value !== null ? [$value] : []);
451 451
         } else {
452 452
             $decodedAttribute = $value;
453 453
         }
454 454
 
455
-        if (is_array($decodedAttribute) && ! empty($decodedAttribute)) {
455
+        if (is_array($decodedAttribute) && !empty($decodedAttribute)) {
456 456
             if (isset($decodedAttribute[app()->getLocale()])) {
457 457
                 return $decodedAttribute[app()->getLocale()];
458 458
             } else {
@@ -463,7 +463,7 @@  discard block
 block discarded – undo
463 463
         return $value;
464 464
     }
465 465
 
466
-    public function setLocaleOnModel(Model $model, null|bool|string $useFallbackLocale = null)
466
+    public function setLocaleOnModel(Model $model, null | bool | string $useFallbackLocale = null)
467 467
     {
468 468
         $useFallbackLocale = $useFallbackLocale ?? $this->shouldUseFallbackLocale();
469 469
 
@@ -493,7 +493,7 @@  discard block
 block discarded – undo
493 493
         $relation = $model->{$firstRelationName};
494 494
 
495 495
         $results = [];
496
-        if (! is_null($relation)) {
496
+        if (!is_null($relation)) {
497 497
             if ($relation instanceof Collection) {
498 498
                 $currentResults = $relation->all();
499 499
             } elseif (is_array($relation)) {
@@ -506,7 +506,7 @@  discard block
 block discarded – undo
506 506
 
507 507
             array_shift($relationArray);
508 508
 
509
-            if (! empty($relationArray)) {
509
+            if (!empty($relationArray)) {
510 510
                 foreach ($currentResults as $currentResult) {
511 511
                     $results = array_merge_recursive($results, $this->getRelatedEntries($currentResult, implode('.', $relationArray)));
512 512
                 }
Please login to merge, or discard this patch.
src/app/Http/Controllers/Operations/ShowOperation.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -32,26 +32,26 @@
 block discarded – undo
32 32
         $this->crud->allowAccess('show');
33 33
         $this->crud->setOperationSetting('setFromDb', true);
34 34
 
35
-        $this->crud->operation('show', function () {
35
+        $this->crud->operation('show', function() {
36 36
             $this->crud->loadDefaultOperationSettingsFromConfig();
37 37
             $this->setupTranslatorInstance();
38 38
 
39
-            if (! method_exists($this, 'setupShowOperation')) {
39
+            if (!method_exists($this, 'setupShowOperation')) {
40 40
                 $this->autoSetupShowOperation();
41 41
             }
42 42
         });
43 43
 
44
-        $this->crud->operation('list', function () {
44
+        $this->crud->operation('list', function() {
45 45
             $this->crud->addButton('line', 'show', 'view', 'crud::buttons.show', 'beginning');
46 46
         });
47 47
 
48
-        $this->crud->operation(['create', 'update'], function () {
48
+        $this->crud->operation(['create', 'update'], function() {
49 49
             $this->crud->addSaveAction([
50 50
                 'name' => 'save_and_preview',
51
-                'visible' => function ($crud) {
51
+                'visible' => function($crud) {
52 52
                     return $crud->hasAccess('show');
53 53
                 },
54
-                'redirect' => function ($crud, $request, $itemId = null) {
54
+                'redirect' => function($crud, $request, $itemId = null) {
55 55
                     $itemId = $itemId ?: $request->input('id');
56 56
                     $redirectUrl = $crud->route.'/'.$itemId.'/show';
57 57
                     if ($request->has('_locale')) {
Please login to merge, or discard this patch.
src/app/Http/Controllers/Operations/Concerns/HasTranslations.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -2,12 +2,12 @@
 block discarded – undo
2 2
 
3 3
 namespace Backpack\CRUD\app\Http\Controllers\Operations\Concerns;
4 4
 
5
-if (! method_exists(__CLASS__, 'setupTranslatorInstance')) {
5
+if (!method_exists(__CLASS__, 'setupTranslatorInstance')) {
6 6
     trait HasTranslations
7 7
     {
8 8
         public function setupTranslatorInstance()
9 9
         {
10
-            if (! method_exists($this->crud->model, 'translationEnabledForModel') || ! class_exists('Spatie\Translatable\Translatable')) {
10
+            if (!method_exists($this->crud->model, 'translationEnabledForModel') || !class_exists('Spatie\Translatable\Translatable')) {
11 11
                 return;
12 12
             }
13 13
 
Please login to merge, or discard this patch.
src/app/Http/Controllers/Operations/ListOperation.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
             'operation' => 'list',
30 30
         ]);
31 31
 
32
-        if (! isset($this->setupDetailsRowRoute) || $this->setupDetailsRowRoute === true) {
32
+        if (!isset($this->setupDetailsRowRoute) || $this->setupDetailsRowRoute === true) {
33 33
             Route::get($segment.'/{id}/details', [
34 34
                 'as' => $routeName.'.showDetailsRow',
35 35
                 'uses' => $controller.'@showDetailsRow',
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
     {
46 46
         $this->crud->allowAccess('list');
47 47
 
48
-        $this->crud->operation('list', function () {
48
+        $this->crud->operation('list', function() {
49 49
             $this->crud->loadDefaultOperationSettingsFromConfig();
50 50
             $this->setupTranslatorInstance();
51 51
         });
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
         $search = request()->input('search');
84 84
 
85 85
         // check if length is allowed by developer
86
-        if ($length && ! in_array($length, $this->crud->getPageLengthMenu()[0])) {
86
+        if ($length && !in_array($length, $this->crud->getPageLengthMenu()[0])) {
87 87
             return response()->json([
88 88
                 'error' => 'Unknown page length.',
89 89
             ], 400);
Please login to merge, or discard this patch.