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 (#5733)
by
unknown
24:39 queued 09:47
created
src/app/Library/CrudPanel/Traits/Search.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -22,9 +22,9 @@  discard block
 block discarded – undo
22 22
      */
23 23
     public function applySearchTerm($searchTerm)
24 24
     {
25
-        return $this->query->where(function ($query) use ($searchTerm) {
25
+        return $this->query->where(function($query) use ($searchTerm) {
26 26
             foreach ($this->columns() as $column) {
27
-                if (! isset($column['type'])) {
27
+                if (!isset($column['type'])) {
28 28
                     abort(500, 'Missing column type when trying to apply search term.', ['developer-error-exception']);
29 29
                 }
30 30
 
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
 
85 85
                 case 'select':
86 86
                 case 'select_multiple':
87
-                    $query->orWhereHas($column['entity'], function ($q) use ($column, $searchTerm, $searchOperator) {
87
+                    $query->orWhereHas($column['entity'], function($q) use ($column, $searchTerm, $searchOperator) {
88 88
                         $q->where($this->getColumnWithTableNamePrefixed($q, $column['attribute']), $searchOperator, '%'.$searchTerm.'%');
89 89
                     });
90 90
                     break;
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
                 $column_direction = (strtolower((string) $order['dir']) == 'asc' ? 'ASC' : 'DESC');
109 109
                 $column = $this->findColumnById($column_number);
110 110
 
111
-                if ($column['tableColumn'] && ! isset($column['orderLogic'])) {
111
+                if ($column['tableColumn'] && !isset($column['orderLogic'])) {
112 112
                     if (method_exists($this->model, 'translationEnabled') &&
113 113
                         $this->model->translationEnabled() &&
114 114
                         $this->model->isTranslatableAttribute($column['name']) &&
@@ -136,12 +136,12 @@  discard block
 block discarded – undo
136 136
         $key = $this->model->getKeyName();
137 137
         $groupBy = $this->query->toBase()->groups;
138 138
 
139
-        $hasOrderByPrimaryKey = collect($orderBy)->some(function ($item) use ($key, $table) {
139
+        $hasOrderByPrimaryKey = collect($orderBy)->some(function($item) use ($key, $table) {
140 140
             return (isset($item['column']) && $item['column'] === $key)
141 141
                 || (isset($item['sql']) && str_contains($item['sql'], "$table.$key"));
142 142
         });
143 143
 
144
-        if (! $hasOrderByPrimaryKey && empty($groupBy)) {
144
+        if (!$hasOrderByPrimaryKey && empty($groupBy)) {
145 145
             $this->orderByWithPrefix($key, 'DESC');
146 146
         }
147 147
     }
@@ -320,13 +320,13 @@  discard block
 block discarded – undo
320 320
         if (isset($column['type'])) {
321 321
             // create a list of paths to column blade views
322 322
             // including the configured view_namespaces
323
-            $columnPaths = array_map(function ($item) use ($column) {
323
+            $columnPaths = array_map(function($item) use ($column) {
324 324
                 return $item.'.'.$column['type'];
325 325
             }, ViewNamespaces::getFor('columns'));
326 326
 
327 327
             // but always fall back to the stock 'text' column
328 328
             // if a view doesn't exist
329
-            if (! in_array('crud::columns.text', $columnPaths)) {
329
+            if (!in_array('crud::columns.text', $columnPaths)) {
330 330
                 $columnPaths[] = 'crud::columns.text';
331 331
             }
332 332
 
@@ -365,7 +365,7 @@  discard block
 block discarded – undo
365 365
      */
366 366
     private function renderCellView($view, $column, $entry, $rowNumber = false)
367 367
     {
368
-        if (! view()->exists($view)) {
368
+        if (!view()->exists($view)) {
369 369
             $view = 'crud::columns.text'; // fallback to text column
370 370
         }
371 371
 
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/CrudColumn.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -87,7 +87,7 @@  discard block
 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
 
@@ -179,7 +179,7 @@  discard block
 block discarded – undo
179 179
      */
180 180
     public function subfields($subfields)
181 181
     {
182
-        $callAttributeMacro = ! isset($this->attributes['subfields']);
182
+        $callAttributeMacro = !isset($this->attributes['subfields']);
183 183
         $this->attributes['subfields'] = $subfields;
184 184
         $this->attributes = $this->crud()->makeSureColumnHasNeededAttributes($this->attributes);
185 185
         if ($callAttributeMacro) {
Please login to merge, or discard this patch.
src/ThemeServiceProvider.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
         }
146 146
 
147 147
         // Registering package commands.
148
-        if (! empty($this->commands)) {
148
+        if (!empty($this->commands)) {
149 149
             $this->commands($this->commands);
150 150
         }
151 151
     }
@@ -240,7 +240,7 @@  discard block
 block discarded – undo
240 240
     protected function packageDirectoryExistsAndIsNotEmpty($name)
241 241
     {
242 242
         // check if directory exists
243
-        if (! is_dir($this->getPath().'/'.$name)) {
243
+        if (!is_dir($this->getPath().'/'.$name)) {
244 244
             return false;
245 245
         }
246 246
 
@@ -265,7 +265,7 @@  discard block
 block discarded – undo
265 265
     public function registerPackageBladeComponents()
266 266
     {
267 267
         if ($this->componentsNamespace) {
268
-            $this->app->afterResolving(BladeCompiler::class, function () {
268
+            $this->app->afterResolving(BladeCompiler::class, function() {
269 269
                 Blade::componentNamespace($this->componentsNamespace, $this->packageName);
270 270
             });
271 271
         }
Please login to merge, or discard this patch.
src/app/Library/Uploaders/Support/RegisterUploadEvents.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -14,18 +14,18 @@  discard block
 block discarded – undo
14 14
     private string $crudObjectType;
15 15
 
16 16
     public function __construct(
17
-        private readonly CrudField|CrudColumn $crudObject,
17
+        private readonly CrudField | CrudColumn $crudObject,
18 18
         private readonly array $uploaderConfiguration,
19 19
         private readonly string $macro
20 20
         ) {
21 21
         $this->crudObjectType = is_a($crudObject, CrudField::class) ? 'field' : (is_a($crudObject, CrudColumn::class) ? 'column' : null);
22 22
 
23
-        if (! $this->crudObjectType) {
23
+        if (!$this->crudObjectType) {
24 24
             abort(500, 'Upload handlers only work for CrudField and CrudColumn classes.', ['developer-error-exception']);
25 25
         }
26 26
     }
27 27
 
28
-    public static function handle(CrudField|CrudColumn $crudObject, array $uploaderConfiguration, string $macro, ?array $subfield = null, bool $registerModelEvents = true): void
28
+    public static function handle(CrudField | CrudColumn $crudObject, array $uploaderConfiguration, string $macro, ?array $subfield = null, bool $registerModelEvents = true): void
29 29
     {
30 30
         $instance = new self($crudObject, $uploaderConfiguration, $macro);
31 31
 
@@ -35,9 +35,9 @@  discard block
 block discarded – undo
35 35
     /*******************************
36 36
      * Private methods - implementation
37 37
      *******************************/
38
-    private function registerEvents(array|null $subfield = [], ?bool $registerModelEvents = true): void
38
+    private function registerEvents(array | null $subfield = [], ?bool $registerModelEvents = true): void
39 39
     {
40
-        if (! empty($subfield)) {
40
+        if (!empty($subfield)) {
41 41
             $this->registerSubfieldEvent($subfield, $registerModelEvents);
42 42
 
43 43
             return;
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
         }
82 82
 
83 83
         $subfields = collect($this->crudObject->getAttributes()['subfields']);
84
-        $subfields = $subfields->map(function ($item) use ($subfield, $uploader) {
84
+        $subfields = $subfields->map(function($item) use ($subfield, $uploader) {
85 85
             if ($item['name'] === $subfield['name']) {
86 86
                 $item['upload'] = true;
87 87
                 $item['disk'] = $uploader->getDisk();
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
         }
112 112
 
113 113
         if ($this->crudObjectType === 'field') {
114
-            $model::saving(function ($entry) use ($uploader) {
114
+            $model::saving(function($entry) use ($uploader) {
115 115
                 $entry = $uploader->storeUploadedFiles($entry);
116 116
             });
117 117
         }
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
             // is not called in pivot models when loading the relations.
125 125
             $retrieveModel = $this->getModelForRetrieveEvent($model, $uploader);
126 126
 
127
-            $retrieveModel::retrieved(function ($entry) use ($uploader) {
127
+            $retrieveModel::retrieved(function($entry) use ($uploader) {
128 128
                 if ($entry->translationEnabled()) {
129 129
                     $locale = request('_locale', app()->getLocale());
130 130
                     if (in_array($locale, array_keys($entry->getAvailableLocales()))) {
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
             });
136 136
         }
137 137
 
138
-        $model::deleting(function ($entry) use ($uploader) {
138
+        $model::deleting(function($entry) use ($uploader) {
139 139
             $uploader->deleteUploadedFiles($entry);
140 140
         });
141 141
 
@@ -157,13 +157,13 @@  discard block
 block discarded – undo
157 157
         $customUploader = isset($uploaderConfiguration['uploader']) && class_exists($uploaderConfiguration['uploader']);
158 158
 
159 159
         if ($customUploader) {
160
-            return $uploaderConfiguration['uploader']::for($crudObject, $uploaderConfiguration);
160
+            return $uploaderConfiguration['uploader']::for ($crudObject, $uploaderConfiguration);
161 161
         }
162 162
 
163 163
         $uploader = app('UploadersRepository')->hasUploadFor($crudObject['type'], $this->macro);
164 164
 
165 165
         if ($uploader) {
166
-            return app('UploadersRepository')->getUploadFor($crudObject['type'], $this->macro)::for($crudObject, $uploaderConfiguration);
166
+            return app('UploadersRepository')->getUploadFor($crudObject['type'], $this->macro)::for ($crudObject, $uploaderConfiguration);
167 167
         }
168 168
 
169 169
         throw new Exception('Undefined upload type for '.$this->crudObjectType.' type: '.$crudObject['type']);
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
184 184
 
185 185
     private function getSubfieldModel(array $subfield, UploaderInterface $uploader)
186 186
     {
187
-        if (! $uploader->isRelationship()) {
187
+        if (!$uploader->isRelationship()) {
188 188
             return $subfield['baseModel'] ?? get_class(app('crud')->getModel());
189 189
         }
190 190
 
@@ -197,7 +197,7 @@  discard block
 block discarded – undo
197 197
 
198 198
     private function getModelForRetrieveEvent(string $model, UploaderInterface $uploader)
199 199
     {
200
-        if (! $uploader->isRelationship()) {
200
+        if (!$uploader->isRelationship()) {
201 201
             return $model;
202 202
         }
203 203
 
Please login to merge, or discard this patch.
src/app/Library/Validation/Rules/ValidFileArray.php 1 patch
Spacing   +5 added lines, -6 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
      */
24 24
     public function validate(string $attribute, mixed $value, Closure $fail): void
25 25
     {
26
-        if (! $value = self::ensureValidValue($value)) {
26
+        if (!$value = self::ensureValidValue($value)) {
27 27
             $fail('Unable to determine the value type.');
28 28
 
29 29
             return;
@@ -33,12 +33,12 @@  discard block
 block discarded – undo
33 33
         $this->validateItems($attribute, $value, $fail);
34 34
     }
35 35
 
36
-    public static function field(string|array|ValidationRule|Rule $rules = []): self
36
+    public static function field(string | array | ValidationRule | Rule $rules = []): self
37 37
     {
38 38
         $instance = new static();
39 39
         $instance->fieldRules = self::getRulesAsArray($rules);
40 40
 
41
-        if (! in_array('array', $instance->getFieldRules())) {
41
+        if (!in_array('array', $instance->getFieldRules())) {
42 42
             $instance->fieldRules[] = 'array';
43 43
         }
44 44
 
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
 
82 82
     protected static function ensureValidValue($value)
83 83
     {
84
-        if (! is_array($value)) {
84
+        if (!is_array($value)) {
85 85
             try {
86 86
                 $value = json_decode($value, true);
87 87
             } catch (\Exception $e) {
@@ -95,7 +95,6 @@  discard block
 block discarded – undo
95 95
     private function getValidationAttributeString($attribute)
96 96
     {
97 97
         return Str::substrCount($attribute, '.') > 1 ?
98
-                Str::before($attribute, '.').'.*.'.Str::afterLast($attribute, '.') :
99
-                $attribute;
98
+                Str::before($attribute, '.').'.*.'.Str::afterLast($attribute, '.') : $attribute;
100 99
     }
101 100
 }
Please login to merge, or discard this patch.
src/app/Http/Controllers/Auth/VerifyEmailController.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -20,21 +20,21 @@  discard block
 block discarded – undo
20 20
      */
21 21
     public function __construct()
22 22
     {
23
-        if (! app('router')->getMiddleware()['signed'] ?? null) {
23
+        if (!app('router')->getMiddleware()['signed'] ?? null) {
24 24
             throw new Exception('Missing "signed" alias middleware in App/Http/Kernel.php. More info: https://backpackforlaravel.com/docs/6.x/base-how-to#enable-email-verification-in-backpack-routes');
25 25
         }
26 26
 
27 27
         $this->middleware('signed')->only('verifyEmail');
28 28
         $this->middleware('throttle:'.config('backpack.base.email_verification_throttle_access'))->only('resendVerificationEmail');
29 29
 
30
-        if (! backpack_users_have_email()) {
30
+        if (!backpack_users_have_email()) {
31 31
             abort(500, trans('backpack::base.no_email_column'));
32 32
         }
33 33
         // where to redirect after the email is verified
34 34
         $this->redirectTo = $this->redirectTo ?? backpack_url('dashboard');
35 35
     }
36 36
 
37
-    public function emailVerificationRequired(Request $request): \Illuminate\Contracts\View\View|\Illuminate\Http\RedirectResponse
37
+    public function emailVerificationRequired(Request $request): \Illuminate\Contracts\View\View | \Illuminate\Http\RedirectResponse
38 38
     {
39 39
         $this->getUserOrRedirect($request);
40 40
 
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
         return $request->user(backpack_guard_name()) ?? (new UserFromCookie())();
78 78
     }
79 79
 
80
-    private function getUserOrRedirect(Request $request): \Illuminate\Contracts\Auth\MustVerifyEmail|\Illuminate\Http\RedirectResponse
80
+    private function getUserOrRedirect(Request $request): \Illuminate\Contracts\Auth\MustVerifyEmail | \Illuminate\Http\RedirectResponse
81 81
     {
82 82
         if ($user = $this->getUser($request)) {
83 83
             return $user;
Please login to merge, or discard this patch.