We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.
@@ -36,7 +36,7 @@ discard block |
||
36 | 36 | { |
37 | 37 | $this->crudPanel->setModel(User::class); |
38 | 38 | $request = request()->create('/admin/users/create', 'POST', ['name' => 'foo']); |
39 | - $request->setRouteResolver(function () use ($request) { |
|
39 | + $request->setRouteResolver(function() use ($request) { |
|
40 | 40 | return (new Route('POST', 'admin/users/create', ['UserCrudController', 'create']))->bind($request); |
41 | 41 | }); |
42 | 42 | $this->crudPanel->setRequest($request); |
@@ -51,7 +51,7 @@ discard block |
||
51 | 51 | { |
52 | 52 | $this->crudPanel->setModel(User::class); |
53 | 53 | $request = request()->create('/admin/users/create', 'POST', ['name' => 'foo']); |
54 | - $request->setRouteResolver(function () use ($request) { |
|
54 | + $request->setRouteResolver(function() use ($request) { |
|
55 | 55 | return (new Route('POST', 'admin/users/create', ['UserCrudController', 'create']))->bind($request); |
56 | 56 | }); |
57 | 57 | $this->crudPanel->setRequest($request); |
@@ -66,7 +66,7 @@ discard block |
||
66 | 66 | { |
67 | 67 | $this->crudPanel->setModel(User::class); |
68 | 68 | $request = request()->create('/admin/users/create', 'POST', ['name' => 'foo']); |
69 | - $request->setRouteResolver(function () use ($request) { |
|
69 | + $request->setRouteResolver(function() use ($request) { |
|
70 | 70 | return (new Route('POST', 'admin/users/create', ['UserCrudController', 'create']))->bind($request); |
71 | 71 | }); |
72 | 72 | $this->crudPanel->setRequest($request); |
@@ -9,7 +9,7 @@ discard block |
||
9 | 9 | { |
10 | 10 | public function testItCanRegisterMacro() |
11 | 11 | { |
12 | - $this->crudPanel::macro('validMacro', function () { |
|
12 | + $this->crudPanel::macro('validMacro', function() { |
|
13 | 13 | return true; |
14 | 14 | }); |
15 | 15 | |
@@ -19,7 +19,7 @@ discard block |
||
19 | 19 | public function testThrowsErrorIfMacroExists() |
20 | 20 | { |
21 | 21 | try { |
22 | - $this->crudPanel::macro('setModel', function () { |
|
22 | + $this->crudPanel::macro('setModel', function() { |
|
23 | 23 | return true; |
24 | 24 | }); |
25 | 25 | } catch (\Throwable $e) { |
@@ -605,7 +605,7 @@ discard block |
||
605 | 605 | |
606 | 606 | public function testCreateBelongsToFake() |
607 | 607 | { |
608 | - $belongsToField = [ // select_grouped |
|
608 | + $belongsToField = [// select_grouped |
|
609 | 609 | 'label' => 'Select_grouped', |
610 | 610 | 'type' => 'select_grouped', //https://github.com/Laravel-Backpack/CRUD/issues/502 |
611 | 611 | 'name' => 'bang_relation_field', |
@@ -741,7 +741,7 @@ discard block |
||
741 | 741 | 'nickname' => 'i_have_has_one', |
742 | 742 | 'profile_picture' => 'ohh my picture 1.jpg', |
743 | 743 | 'article' => 1, |
744 | - 'addresses' => [ // HasOne is tested in other test function |
|
744 | + 'addresses' => [// HasOne is tested in other test function |
|
745 | 745 | [ |
746 | 746 | 'city' => 2, |
747 | 747 | 'street' => 'test', |
@@ -16,7 +16,7 @@ |
||
16 | 16 | |
17 | 17 | public function testItCanConfigureOperations() |
18 | 18 | { |
19 | - $this->crudPanel->operation(['create', 'update'], function () { |
|
19 | + $this->crudPanel->operation(['create', 'update'], function() { |
|
20 | 20 | $this->crudPanel->addField(['name' => 'test', 'type' => 'text']); |
21 | 21 | }); |
22 | 22 | $this->crudPanel->applyConfigurationFromSettings('create'); |
@@ -28,7 +28,7 @@ discard block |
||
28 | 28 | $higherAction = $this->getSaveActionByOrder(1); |
29 | 29 | |
30 | 30 | //if there is an higher action and that action is not the backpack default higher one `save_and_back` we return it. |
31 | - if (! empty($higherAction) && key($higherAction) !== 'save_and_back') { |
|
31 | + if (!empty($higherAction) && key($higherAction) !== 'save_and_back') { |
|
32 | 32 | return key($higherAction); |
33 | 33 | } |
34 | 34 | |
@@ -47,7 +47,7 @@ discard block |
||
47 | 47 | */ |
48 | 48 | public function getSaveActionByOrder($order) |
49 | 49 | { |
50 | - return array_filter($this->getOperationSetting('save_actions') ?? [], function ($arr) use ($order) { |
|
50 | + return array_filter($this->getOperationSetting('save_actions') ?? [], function($arr) use ($order) { |
|
51 | 51 | return $arr['order'] == $order; |
52 | 52 | }); |
53 | 53 | } |
@@ -81,7 +81,7 @@ discard block |
||
81 | 81 | $orderCounter = $this->getOperationSetting('save_actions') !== null ? (count($this->getOperationSetting('save_actions')) + 1) : 1; |
82 | 82 | //check for some mandatory fields |
83 | 83 | $saveAction['name'] ?? abort(500, 'Please define save action name.'); |
84 | - $saveAction['redirect'] = $saveAction['redirect'] ?? function ($crud, $request, $itemId) { |
|
84 | + $saveAction['redirect'] = $saveAction['redirect'] ?? function($crud, $request, $itemId) { |
|
85 | 85 | return $request->has('_http_referrer') ? $request->get('_http_referrer') : $crud->route; |
86 | 86 | }; |
87 | 87 | $saveAction['visible'] = $saveAction['visible'] ?? true; |
@@ -90,7 +90,7 @@ discard block |
||
90 | 90 | |
91 | 91 | $actions = $this->getOperationSetting('save_actions') ?? []; |
92 | 92 | |
93 | - if (! in_array($saveAction['name'], $actions)) { |
|
93 | + if (!in_array($saveAction['name'], $actions)) { |
|
94 | 94 | $actions[$saveAction['name']] = $saveAction; |
95 | 95 | } |
96 | 96 | |
@@ -107,7 +107,7 @@ discard block |
||
107 | 107 | public function orderSaveAction(string $saveAction, int $wantedOrder) |
108 | 108 | { |
109 | 109 | $actions = $this->getOperationSetting('save_actions') ?? []; |
110 | - if (! empty($actions)) { |
|
110 | + if (!empty($actions)) { |
|
111 | 111 | $replaceOrder = isset($actions[$saveAction]) ? $actions[$saveAction]['order'] : count($actions) + 1; |
112 | 112 | |
113 | 113 | foreach ($actions as $key => $sv) { |
@@ -201,7 +201,7 @@ discard block |
||
201 | 201 | public function orderSaveActions(array $saveActions) |
202 | 202 | { |
203 | 203 | foreach ($saveActions as $sv => $order) { |
204 | - if (! is_int($order)) { |
|
204 | + if (!is_int($order)) { |
|
205 | 205 | $this->orderSaveAction($order, $sv + 1); |
206 | 206 | } else { |
207 | 207 | $this->orderSaveAction($sv, $order); |
@@ -218,7 +218,7 @@ discard block |
||
218 | 218 | { |
219 | 219 | $actions = $this->getOperationSetting('save_actions') ?? []; |
220 | 220 | |
221 | - uasort($actions, function ($a, $b) { |
|
221 | + uasort($actions, function($a, $b) { |
|
222 | 222 | return $a['order'] <=> $b['order']; |
223 | 223 | }); |
224 | 224 | |
@@ -240,7 +240,7 @@ discard block |
||
240 | 240 | } |
241 | 241 | } |
242 | 242 | |
243 | - return array_filter($actions, function ($action) { |
|
243 | + return array_filter($actions, function($action) { |
|
244 | 244 | return $action['visible'] == true; |
245 | 245 | }, ARRAY_FILTER_USE_BOTH); |
246 | 246 | } |
@@ -303,8 +303,7 @@ discard block |
||
303 | 303 | */ |
304 | 304 | public function setSaveAction($forceSaveAction = null) |
305 | 305 | { |
306 | - $saveAction = $forceSaveAction ?: |
|
307 | - \Request::input('_save_action', $this->getFallBackSaveAction()); |
|
306 | + $saveAction = $forceSaveAction ?: \Request::input('_save_action', $this->getFallBackSaveAction()); |
|
308 | 307 | |
309 | 308 | $showBubble = $this->getOperationSetting('showSaveActionChange') ?? config('backpack.crud.operations.'.$this->getCurrentOperation().'.showSaveActionChange') ?? true; |
310 | 309 | |
@@ -371,20 +370,20 @@ discard block |
||
371 | 370 | $defaultSaveActions = [ |
372 | 371 | [ |
373 | 372 | 'name' => 'save_and_back', |
374 | - 'visible' => function ($crud) { |
|
373 | + 'visible' => function($crud) { |
|
375 | 374 | return $crud->hasAccess('list'); |
376 | 375 | }, |
377 | - 'redirect' => function ($crud, $request, $itemId = null) { |
|
376 | + 'redirect' => function($crud, $request, $itemId = null) { |
|
378 | 377 | return $request->request->has('_http_referrer') ? $request->request->get('_http_referrer') : $crud->route; |
379 | 378 | }, |
380 | 379 | 'button_text' => trans('backpack::crud.save_action_save_and_back'), |
381 | 380 | ], |
382 | 381 | [ |
383 | 382 | 'name' => 'save_and_edit', |
384 | - 'visible' => function ($crud) { |
|
383 | + 'visible' => function($crud) { |
|
385 | 384 | return $crud->hasAccess('update'); |
386 | 385 | }, |
387 | - 'redirect' => function ($crud, $request, $itemId = null) { |
|
386 | + 'redirect' => function($crud, $request, $itemId = null) { |
|
388 | 387 | $itemId = $itemId ?: $request->request->get('id'); |
389 | 388 | $redirectUrl = $crud->route.'/'.$itemId.'/edit'; |
390 | 389 | if ($request->request->has('_locale')) { |
@@ -396,17 +395,17 @@ discard block |
||
396 | 395 | |
397 | 396 | return $redirectUrl; |
398 | 397 | }, |
399 | - 'referrer_url' => function ($crud, $request, $itemId) { |
|
398 | + 'referrer_url' => function($crud, $request, $itemId) { |
|
400 | 399 | return url($crud->route.'/'.$itemId.'/edit'); |
401 | 400 | }, |
402 | 401 | 'button_text' => trans('backpack::crud.save_action_save_and_edit'), |
403 | 402 | ], |
404 | 403 | [ |
405 | 404 | 'name' => 'save_and_new', |
406 | - 'visible' => function ($crud) { |
|
405 | + 'visible' => function($crud) { |
|
407 | 406 | return $crud->hasAccess('create'); |
408 | 407 | }, |
409 | - 'redirect' => function ($crud, $request, $itemId = null) { |
|
408 | + 'redirect' => function($crud, $request, $itemId = null) { |
|
410 | 409 | return $this->route.'/create'; |
411 | 410 | }, |
412 | 411 | 'button_text' => trans('backpack::crud.save_action_save_and_new'), |
@@ -609,7 +609,7 @@ discard block |
||
609 | 609 | { |
610 | 610 | $this->crudPanel->setOperationSetting( |
611 | 611 | 'strippedRequest', |
612 | - static function (Request $request) { |
|
612 | + static function(Request $request) { |
|
613 | 613 | return $request->toArray(); |
614 | 614 | }, |
615 | 615 | 'update' |
@@ -743,7 +743,7 @@ discard block |
||
743 | 743 | ->validationMessages(['required' => 'is_required', 'min' => 'min_2']) |
744 | 744 | ->store_in('some') |
745 | 745 | ->size(6) |
746 | - ->on('created', function () { |
|
746 | + ->on('created', function() { |
|
747 | 747 | }) |
748 | 748 | ->subfields([['name' => 'sub_1']]) |
749 | 749 | ->entity('bang'); |
@@ -775,7 +775,7 @@ discard block |
||
775 | 775 | 'class' => 'form-group col-md-6', |
776 | 776 | ], |
777 | 777 | 'events' => [ |
778 | - 'created' => function () { |
|
778 | + 'created' => function() { |
|
779 | 779 | }, |
780 | 780 | ], |
781 | 781 | 'subfields' => [ |
@@ -48,7 +48,7 @@ discard block |
||
48 | 48 | { |
49 | 49 | $this->crudPanel->setModel(User::class); |
50 | 50 | $request = request()->create('/admin/users', 'GET', ['my_custom_filter' => 'foo']); |
51 | - $request->setRouteResolver(function () use ($request) { |
|
51 | + $request->setRouteResolver(function() use ($request) { |
|
52 | 52 | return (new Route('GET', 'admin/users', ['UserCrudController', 'index']))->bind($request); |
53 | 53 | }); |
54 | 54 | $this->crudPanel->setRequest($request); |
@@ -64,10 +64,10 @@ discard block |
||
64 | 64 | ->options(['test' => 'test']) |
65 | 65 | ->view('simple') |
66 | 66 | ->viewNamespace('crud::filters') |
67 | - ->ifActive(function () { |
|
67 | + ->ifActive(function() { |
|
68 | 68 | return true; |
69 | 69 | }) |
70 | - ->ifInactive(function () { |
|
70 | + ->ifInactive(function() { |
|
71 | 71 | return true; |
72 | 72 | }); |
73 | 73 | |
@@ -84,10 +84,10 @@ discard block |
||
84 | 84 | public function testWhenActiveAndWhenInactiveAliases() |
85 | 85 | { |
86 | 86 | $filter = CrudFilter::name('my_filter') |
87 | - ->whenActive(function () { |
|
87 | + ->whenActive(function() { |
|
88 | 88 | return true; |
89 | 89 | }) |
90 | - ->whenInactive(function () { |
|
90 | + ->whenInactive(function() { |
|
91 | 91 | return true; |
92 | 92 | }); |
93 | 93 | |
@@ -98,7 +98,7 @@ discard block |
||
98 | 98 | |
99 | 99 | public function testWhenNotActiveAlias() |
100 | 100 | { |
101 | - $filter = CrudFilter::name('my_filter')->whenNotActive(function () { |
|
101 | + $filter = CrudFilter::name('my_filter')->whenNotActive(function() { |
|
102 | 102 | return true; |
103 | 103 | }); |
104 | 104 | $this->assertCount(1, $this->crudPanel->filters()); |
@@ -107,7 +107,7 @@ discard block |
||
107 | 107 | |
108 | 108 | public function testIfNotActiveAlias() |
109 | 109 | { |
110 | - $filter = CrudFilter::name('my_filter')->ifNotActive(function () { |
|
110 | + $filter = CrudFilter::name('my_filter')->ifNotActive(function() { |
|
111 | 111 | return true; |
112 | 112 | }); |
113 | 113 | $this->assertCount(1, $this->crudPanel->filters()); |
@@ -116,7 +116,7 @@ discard block |
||
116 | 116 | |
117 | 117 | public function testElseAlias() |
118 | 118 | { |
119 | - $filter = CrudFilter::name('my_filter')->else(function () { |
|
119 | + $filter = CrudFilter::name('my_filter')->else(function() { |
|
120 | 120 | return true; |
121 | 121 | }); |
122 | 122 | $this->assertCount(1, $this->crudPanel->filters()); |
@@ -109,11 +109,11 @@ discard block |
||
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 | |
@@ -205,7 +205,7 @@ discard block |
||
205 | 205 | |
206 | 206 | $complete_route = $route.'.index'; |
207 | 207 | |
208 | - if (! \Route::has($complete_route)) { |
|
208 | + if (!\Route::has($complete_route)) { |
|
209 | 209 | throw new \Exception('There are no routes for this route name.', 404); |
210 | 210 | } |
211 | 211 | |
@@ -303,7 +303,7 @@ discard block |
||
303 | 303 | */ |
304 | 304 | public function getFirstOfItsTypeInArray($type, $array) |
305 | 305 | { |
306 | - return Arr::first($array, function ($item) use ($type) { |
|
306 | + return Arr::first($array, function($item) use ($type) { |
|
307 | 307 | return $item['type'] == $type; |
308 | 308 | }); |
309 | 309 | } |
@@ -322,8 +322,8 @@ discard block |
||
322 | 322 | */ |
323 | 323 | public function sync($type, $fields, $attributes) |
324 | 324 | { |
325 | - if (! empty($this->{$type})) { |
|
326 | - $this->{$type} = array_map(function ($field) use ($fields, $attributes) { |
|
325 | + if (!empty($this->{$type})) { |
|
326 | + $this->{$type} = array_map(function($field) use ($fields, $attributes) { |
|
327 | 327 | if (in_array($field['name'], (array) $fields)) { |
328 | 328 | $field = array_merge($field, $attributes); |
329 | 329 | } |
@@ -353,15 +353,15 @@ discard block |
||
353 | 353 | { |
354 | 354 | $relationArray = explode('.', $relationString); |
355 | 355 | |
356 | - if (! isset($length)) { |
|
356 | + if (!isset($length)) { |
|
357 | 357 | $length = count($relationArray); |
358 | 358 | } |
359 | 359 | |
360 | - if (! isset($model)) { |
|
360 | + if (!isset($model)) { |
|
361 | 361 | $model = $this->model; |
362 | 362 | } |
363 | 363 | |
364 | - $result = array_reduce(array_splice($relationArray, 0, $length), function ($obj, $method) { |
|
364 | + $result = array_reduce(array_splice($relationArray, 0, $length), function($obj, $method) { |
|
365 | 365 | try { |
366 | 366 | $result = $obj->$method(); |
367 | 367 | |
@@ -396,7 +396,7 @@ discard block |
||
396 | 396 | if (is_array($entries)) { |
397 | 397 | //if attribute does not exist in main array we have more than one entry OR the attribute |
398 | 398 | //is an acessor that is not in $appends property of model. |
399 | - if (! isset($entries[$attribute])) { |
|
399 | + if (!isset($entries[$attribute])) { |
|
400 | 400 | //we first check if we don't have the attribute because it's an acessor that is not in appends. |
401 | 401 | if ($model_instance->hasGetMutator($attribute) && isset($entries[$modelKey])) { |
402 | 402 | $entry_in_database = $model_instance->find($entries[$modelKey]); |
@@ -435,21 +435,21 @@ discard block |
||
435 | 435 | */ |
436 | 436 | public function parseTranslatableAttributes($model, $attribute, $value) |
437 | 437 | { |
438 | - if (! method_exists($model, 'isTranslatableAttribute')) { |
|
438 | + if (!method_exists($model, 'isTranslatableAttribute')) { |
|
439 | 439 | return $value; |
440 | 440 | } |
441 | 441 | |
442 | - if (! $model->isTranslatableAttribute($attribute)) { |
|
442 | + if (!$model->isTranslatableAttribute($attribute)) { |
|
443 | 443 | return $value; |
444 | 444 | } |
445 | 445 | |
446 | - if (! is_array($value)) { |
|
446 | + if (!is_array($value)) { |
|
447 | 447 | $decodedAttribute = json_decode($value, true); |
448 | 448 | } else { |
449 | 449 | $decodedAttribute = $value; |
450 | 450 | } |
451 | 451 | |
452 | - if (is_array($decodedAttribute) && ! empty($decodedAttribute)) { |
|
452 | + if (is_array($decodedAttribute) && !empty($decodedAttribute)) { |
|
453 | 453 | if (isset($decodedAttribute[app()->getLocale()])) { |
454 | 454 | return $decodedAttribute[app()->getLocale()]; |
455 | 455 | } else { |
@@ -475,7 +475,7 @@ discard block |
||
475 | 475 | $relation = $model->{$firstRelationName}; |
476 | 476 | |
477 | 477 | $results = []; |
478 | - if (! is_null($relation)) { |
|
478 | + if (!is_null($relation)) { |
|
479 | 479 | if ($relation instanceof Collection) { |
480 | 480 | $currentResults = $relation->all(); |
481 | 481 | } elseif (is_array($relation)) { |
@@ -488,7 +488,7 @@ discard block |
||
488 | 488 | |
489 | 489 | array_shift($relationArray); |
490 | 490 | |
491 | - if (! empty($relationArray)) { |
|
491 | + if (!empty($relationArray)) { |
|
492 | 492 | foreach ($currentResults as $currentResult) { |
493 | 493 | $results = array_merge_recursive($results, $this->getRelatedEntries($currentResult, implode('.', $relationArray))); |
494 | 494 | } |
@@ -25,10 +25,10 @@ discard block |
||
25 | 25 | $this->singleSaveAction = [ |
26 | 26 | 'name' => 'save_action_one', |
27 | 27 | 'button_text' => 'custom', |
28 | - 'redirect' => function ($crud, $request, $itemId) { |
|
28 | + 'redirect' => function($crud, $request, $itemId) { |
|
29 | 29 | return $crud->route; |
30 | 30 | }, |
31 | - 'visible' => function ($crud) { |
|
31 | + 'visible' => function($crud) { |
|
32 | 32 | return true; |
33 | 33 | }, |
34 | 34 | ]; |
@@ -36,19 +36,19 @@ discard block |
||
36 | 36 | $this->multipleSaveActions = [ |
37 | 37 | [ |
38 | 38 | 'name' => 'save_action_one', |
39 | - 'redirect' => function ($crud, $request, $itemId) { |
|
39 | + 'redirect' => function($crud, $request, $itemId) { |
|
40 | 40 | return $crud->route; |
41 | 41 | }, |
42 | - 'visible' => function ($crud) { |
|
42 | + 'visible' => function($crud) { |
|
43 | 43 | return true; |
44 | 44 | }, |
45 | 45 | ], |
46 | 46 | [ |
47 | 47 | 'name' => 'save_action_two', |
48 | - 'redirect' => function ($crud, $request, $itemId) { |
|
48 | + 'redirect' => function($crud, $request, $itemId) { |
|
49 | 49 | return $crud->route; |
50 | 50 | }, |
51 | - 'visible' => function ($crud) { |
|
51 | + 'visible' => function($crud) { |
|
52 | 52 | return true; |
53 | 53 | }, |
54 | 54 | ], |