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 Setup Failed
Pull Request — master (#2356)
by Cristian
07:00
created
src/app/Library/CrudPanel/Traits/SaveActions.php 2 patches
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -170,7 +170,6 @@
 block discarded – undo
170 170
     /**
171 171
      * Allow the developer to unset all save actions.
172 172
      *
173
-     * @param string $saveAction
174 173
      * @return void
175 174
      */
176 175
     public function forgetAllSaveActions()
Please login to merge, or discard this patch.
Spacing   +14 added lines, -15 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
      */
48 48
     public function getSaveActionByOrder($order)
49 49
     {
50
-        return array_filter($this->availableSaveActions, function ($arr) use ($order) {
50
+        return array_filter($this->availableSaveActions, function($arr) use ($order) {
51 51
             return $arr['order'] == $order;
52 52
         });
53 53
     }
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
         if (is_array($saveAction)) {
94 94
             //check for some mandatory fields
95 95
             $saveAction['name'] ?? abort(500, 'Please define save action name.');
96
-            $saveAction['redirect'] = $saveAction['redirect'] ?? function ($crud, $request, $itemId) {
96
+            $saveAction['redirect'] = $saveAction['redirect'] ?? function($crud, $request, $itemId) {
97 97
                 return $request->has('http_referrer') ? $request->get('http_referrer') : $crud->route;
98 98
             };
99 99
             $saveAction['permissions'] = $saveAction['permissions'] ?? true;
@@ -112,11 +112,11 @@  discard block
 block discarded – undo
112 112
      */
113 113
     public function reorderSaveActions($wantedOrder)
114 114
     {
115
-        if (! empty($this->availableSaveActions)) {
115
+        if (!empty($this->availableSaveActions)) {
116 116
             $lastOrder = max(array_column($this->availableSaveActions, 'order'));
117 117
             foreach ($this->availableSaveActions as &$sv) {
118 118
                 if ($wantedOrder == $sv['order']) {
119
-                    if (empty(array_filter($this->availableSaveActions, function ($arr) {
119
+                    if (empty(array_filter($this->availableSaveActions, function($arr) {
120 120
                         $arr['order'] == 1;
121 121
                     }))) {
122 122
                         $sv['order'] = 1;
@@ -241,12 +241,12 @@  discard block
 block discarded – undo
241 241
         }
242 242
 
243 243
         //get only passed permissions
244
-        $saveOptions = array_filter($this->availableSaveActions, function ($saveOption) {
244
+        $saveOptions = array_filter($this->availableSaveActions, function($saveOption) {
245 245
             return $saveOption['permissions'] == true;
246 246
         }, ARRAY_FILTER_USE_BOTH);
247 247
 
248 248
         //sort by order
249
-        uasort($saveOptions, function ($a, $b) {
249
+        uasort($saveOptions, function($a, $b) {
250 250
             return $a['order'] <=> $b['order'];
251 251
         });
252 252
 
@@ -285,8 +285,7 @@  discard block
 block discarded – undo
285 285
      */
286 286
     public function setSaveAction($forceSaveAction = null)
287 287
     {
288
-        $saveAction = $forceSaveAction ?:
289
-            \Request::input('save_action', $this->getFallBackSaveAction());
288
+        $saveAction = $forceSaveAction ?: \Request::input('save_action', $this->getFallBackSaveAction());
290 289
 
291 290
         $showBubble = $this->getOperationSetting('showSaveActionChange') ?? config('backpack.crud.operations.'.$this->getCurrentOperation().'.showSaveActionChange') ?? true;
292 291
 
@@ -353,10 +352,10 @@  discard block
 block discarded – undo
353 352
         $defaultSaveActions = [
354 353
             [
355 354
                 'name' => 'save_and_back',
356
-                'permissions' => function ($crud) {
355
+                'permissions' => function($crud) {
357 356
                     return $crud->hasAccess('list');
358 357
                 },
359
-                'redirect' => function ($crud, $request, $itemId = null) {
358
+                'redirect' => function($crud, $request, $itemId = null) {
360 359
                     return $request->has('http_referrer') ? $request->get('http_referrer') : $crud->route;
361 360
                 },
362 361
                 'button_text' => trans('backpack::crud.save_action_save_and_back'),
@@ -364,10 +363,10 @@  discard block
 block discarded – undo
364 363
             ],
365 364
             [
366 365
                 'name' => 'save_and_edit',
367
-                'permissions' => function ($crud) {
366
+                'permissions' => function($crud) {
368 367
                     return $crud->hasAccess('update');
369 368
                 },
370
-                'redirect' => function ($crud, $request, $itemId = null) {
369
+                'redirect' => function($crud, $request, $itemId = null) {
371 370
                     $itemId = $itemId ?: $request->input('id');
372 371
                     $redirectUrl = $crud->route.'/'.$itemId.'/edit';
373 372
                     if ($request->has('locale')) {
@@ -379,7 +378,7 @@  discard block
 block discarded – undo
379 378
 
380 379
                     return $redirectUrl;
381 380
                 },
382
-                'referrer_url' => function ($crud, $request, $itemId) {
381
+                'referrer_url' => function($crud, $request, $itemId) {
383 382
                     return url($crud->route.'/'.$itemId.'/edit');
384 383
                 },
385 384
                 'button_text' => trans('backpack::crud.save_action_save_and_edit'),
@@ -387,10 +386,10 @@  discard block
 block discarded – undo
387 386
             ],
388 387
             [
389 388
                 'name' => 'save_and_new',
390
-                'permissions' => function ($crud) {
389
+                'permissions' => function($crud) {
391 390
                     return $crud->hasAccess('create');
392 391
                 },
393
-                'redirect' => function ($crud, $request, $itemId = null) {
392
+                'redirect' => function($crud, $request, $itemId = null) {
394 393
                     return $this->route.'/create';
395 394
                 },
396 395
                 'button_text' => trans('backpack::crud.save_action_save_and_new'),
Please login to merge, or discard this patch.