We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.
@@ -14,18 +14,18 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
@@ -72,7 +72,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
@@ -115,7 +115,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 | } |
@@ -24,7 +24,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 | } |
@@ -87,7 +87,7 @@ discard block |
||
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 |
||
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) { |
@@ -29,17 +29,17 @@ |
||
29 | 29 | { |
30 | 30 | $this->crud->allowAccess('delete'); |
31 | 31 | |
32 | - $this->crud->operation('delete', function () { |
|
32 | + $this->crud->operation('delete', function() { |
|
33 | 33 | $this->crud->loadDefaultOperationSettingsFromConfig(); |
34 | 34 | }); |
35 | 35 | |
36 | - $this->crud->operation(['list', 'show'], function () { |
|
36 | + $this->crud->operation(['list', 'show'], function() { |
|
37 | 37 | $this->crud->addButton('line', 'delete', 'view', 'crud::buttons.delete', 'end'); |
38 | 38 | }); |
39 | 39 | |
40 | 40 | // setup the default redirect to where user will be redirected after delete |
41 | 41 | // if user has access to list, redirect to list, otherwise redirect to previous page |
42 | - $this->crud->operation('show', function () { |
|
42 | + $this->crud->operation('show', function() { |
|
43 | 43 | $this->crud->setOperationSetting('deleteButtonRedirect', function() { |
44 | 44 | if ($this->crud->hasAccess('list')) { |
45 | 45 | return url($this->crud->route); |