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

Completed
Pull Request — master (#1951)
by ziming
03:15
created
src/app/Http/Controllers/Operations/SaveActions.php 1 patch
Spacing   +3 added lines, -4 removed lines patch added patch discarded remove patch
@@ -22,11 +22,11 @@  discard block
 block discarded – undo
22 22
 
23 23
         $saveOptions = collect($permissions)
24 24
             // Restrict list to allowed actions.
25
-            ->filter(function ($action, $permission) {
25
+            ->filter(function($action, $permission) {
26 26
                 return $this->crud->hasAccess($permission);
27 27
             })
28 28
             // Generate list of possible actions.
29
-            ->mapWithKeys(function ($action, $permission) {
29
+            ->mapWithKeys(function($action, $permission) {
30 30
                 return [$action => $this->getSaveActionButtonName($action)];
31 31
             })
32 32
             ->all();
@@ -61,8 +61,7 @@  discard block
 block discarded – undo
61 61
      */
62 62
     public function setSaveAction($forceSaveAction = null)
63 63
     {
64
-        $saveAction = $forceSaveAction ?:
65
-            \Request::input('save_action', config('backpack.crud.default_save_action', 'save_and_back'));
64
+        $saveAction = $forceSaveAction ?: \Request::input('save_action', config('backpack.crud.default_save_action', 'save_and_back'));
66 65
 
67 66
         if (config('backpack.crud.show_save_action_change', true) &&
68 67
             session('save_action', 'save_and_back') !== $saveAction) {
Please login to merge, or discard this patch.
src/PanelTraits/HeadingsAndTitle.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
      */
22 22
     public function getTitle($action = false)
23 23
     {
24
-        if (! $action) {
24
+        if (!$action) {
25 25
             $action = $this->getActionMethod();
26 26
         }
27 27
 
@@ -38,7 +38,7 @@  discard block
 block discarded – undo
38 38
      */
39 39
     public function setTitle($string, $action = false)
40 40
     {
41
-        if (! $action) {
41
+        if (!$action) {
42 42
             $action = $this->getActionMethod();
43 43
         }
44 44
 
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
      */
59 59
     public function getHeading($action = false)
60 60
     {
61
-        if (! $action) {
61
+        if (!$action) {
62 62
             $action = $this->getActionMethod();
63 63
         }
64 64
 
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
      */
76 76
     public function setHeading($string, $action = false)
77 77
     {
78
-        if (! $action) {
78
+        if (!$action) {
79 79
             $action = $this->getActionMethod();
80 80
         }
81 81
 
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
      */
96 96
     public function getSubheading($action = false)
97 97
     {
98
-        if (! $action) {
98
+        if (!$action) {
99 99
             $action = $this->getActionMethod();
100 100
         }
101 101
 
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
      */
113 113
     public function setSubheading($string, $action = false)
114 114
     {
115
-        if (! $action) {
115
+        if (!$action) {
116 116
             $action = $this->getActionMethod();
117 117
         }
118 118
 
Please login to merge, or discard this patch.
src/ModelTraits/SpatieTranslatable/Sluggable.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@
 block discarded – undo
48 48
         $separator = $config['separator'];
49 49
         $attribute = $attribute.'->'.$this->getLocale();
50 50
 
51
-        return $query->where(function (Builder $q) use ($attribute, $slug, $separator) {
51
+        return $query->where(function(Builder $q) use ($attribute, $slug, $separator) {
52 52
             $q->where($attribute, '=', $slug)
53 53
                 ->orWhere($attribute, 'LIKE', $slug.$separator.'%')
54 54
                 // Fixes issues with Json data types in MySQL where data is sourrounded by "
Please login to merge, or discard this patch.
src/CrudUsageStats.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
     private function sendUsageStats()
39 39
     {
40 40
         // only send usage stats in production
41
-        if (! $this->runningInProduction()) {
41
+        if (!$this->runningInProduction()) {
42 42
             return;
43 43
         }
44 44
 
Please login to merge, or discard this patch.
src/PanelTraits/Buttons.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
          */
108 108
         $button = $this->buttons()->firstWhere('name', $name);
109 109
 
110
-        if (! $button) {
110
+        if (!$button) {
111 111
             abort(500, 'CRUD Button "'.$name.'" not found. Please check the button exists before you modify it.');
112 112
         }
113 113
 
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
      */
129 129
     public function removeButton($name, $stack = null)
130 130
     {
131
-        $this->buttons = $this->buttons->reject(function ($button) use ($name, $stack) {
131
+        $this->buttons = $this->buttons->reject(function($button) use ($name, $stack) {
132 132
             return $stack == null ? $button->name == $name : ($button->stack == $stack) && ($button->name == $name);
133 133
         });
134 134
     }
@@ -140,14 +140,14 @@  discard block
 block discarded – undo
140 140
 
141 141
     public function removeAllButtonsFromStack($stack)
142 142
     {
143
-        $this->buttons = $this->buttons->reject(function ($button) use ($stack) {
143
+        $this->buttons = $this->buttons->reject(function($button) use ($stack) {
144 144
             return $button->stack == $stack;
145 145
         });
146 146
     }
147 147
 
148 148
     public function removeButtonFromStack($name, $stack)
149 149
     {
150
-        $this->buttons = $this->buttons->reject(function ($button) use ($name, $stack) {
150
+        $this->buttons = $this->buttons->reject(function($button) use ($name, $stack) {
151 151
             return $button->name == $name && $button->stack == $stack;
152 152
         });
153 153
     }
Please login to merge, or discard this patch.
src/PanelTraits/Filters.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -11,7 +11,7 @@  discard block
 block discarded – undo
11 11
      */
12 12
     public function filtersEnabled()
13 13
     {
14
-        return ! is_array($this->filters);
14
+        return !is_array($this->filters);
15 15
     }
16 16
 
17 17
     /**
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
         $this->enableFilters();
60 60
 
61 61
         // check if another filter with the same name exists
62
-        if (! isset($options['name'])) {
62
+        if (!isset($options['name'])) {
63 63
             abort(500, 'All your filters need names.');
64 64
         }
65 65
         if ($this->filters->contains('name', $options['name'])) {
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
     {
161 161
         $filter = $this->filters->firstWhere('name', $name);
162 162
 
163
-        if (! $filter) {
163
+        if (!$filter) {
164 164
             abort(500, 'CRUD Filter "'.$name.'" not found. Please check the filter exists before you modify it.');
165 165
         }
166 166
 
@@ -175,7 +175,7 @@  discard block
 block discarded – undo
175 175
 
176 176
     public function removeFilter($name)
177 177
     {
178
-        $this->filters = $this->filters->reject(function ($filter) use ($name) {
178
+        $this->filters = $this->filters->reject(function($filter) use ($name) {
179 179
             return $filter->name == $name;
180 180
         });
181 181
     }
@@ -259,16 +259,16 @@  discard block
 block discarded – undo
259 259
 
260 260
     public function checkOptionsIntegrity($options)
261 261
     {
262
-        if (! isset($options['name'])) {
262
+        if (!isset($options['name'])) {
263 263
             abort(500, 'Please make sure all your filters have names.');
264 264
         }
265
-        if (! isset($options['type'])) {
265
+        if (!isset($options['type'])) {
266 266
             abort(500, 'Please make sure all your filters have types.');
267 267
         }
268
-        if (! \View::exists('crud::filters.'.$options['type'])) {
268
+        if (!\View::exists('crud::filters.'.$options['type'])) {
269 269
             abort(500, 'No filter view named "'.$options['type'].'.blade.php" was found.');
270 270
         }
271
-        if (! isset($options['label'])) {
271
+        if (!isset($options['label'])) {
272 272
             abort(500, 'Please make sure all your filters have labels.');
273 273
         }
274 274
     }
Please login to merge, or discard this patch.
src/PanelTraits/Tabs.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
      */
36 36
     public function tabsDisabled()
37 37
     {
38
-        return ! $this->tabsEnabled;
38
+        return !$this->tabsEnabled;
39 39
     }
40 40
 
41 41
     public function setTabsType($type)
@@ -116,8 +116,8 @@  discard block
 block discarded – undo
116 116
     {
117 117
         $all_fields = $this->getCurrentFields();
118 118
 
119
-        $fields_without_a_tab = collect($all_fields)->filter(function ($value, $key) {
120
-            return ! isset($value['tab']);
119
+        $fields_without_a_tab = collect($all_fields)->filter(function($value, $key) {
120
+            return !isset($value['tab']);
121 121
         });
122 122
 
123 123
         return $fields_without_a_tab;
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
         if ($this->tabExists($label)) {
134 134
             $all_fields = $this->getCurrentFields();
135 135
 
136
-            $fields_for_current_tab = collect($all_fields)->filter(function ($value, $key) use ($label) {
136
+            $fields_for_current_tab = collect($all_fields)->filter(function($value, $key) use ($label) {
137 137
                 return isset($value['tab']) && $value['tab'] == $label;
138 138
             });
139 139
 
@@ -152,11 +152,11 @@  discard block
 block discarded – undo
152 152
         $fields = $this->getCurrentFields();
153 153
 
154 154
         $fields_with_tabs = collect($fields)
155
-            ->filter(function ($value, $key) {
155
+            ->filter(function($value, $key) {
156 156
                 return isset($value['tab']);
157 157
             })
158
-            ->each(function ($value, $key) use (&$tabs) {
159
-                if (! in_array($value['tab'], $tabs)) {
158
+            ->each(function($value, $key) use (&$tabs) {
159
+                if (!in_array($value['tab'], $tabs)) {
160 160
                     $tabs[] = $value['tab'];
161 161
                 }
162 162
             });
Please login to merge, or discard this patch.
src/CrudPanel.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -90,11 +90,11 @@  discard block
 block discarded – undo
90 90
      */
91 91
     public function setModel($model_namespace)
92 92
     {
93
-        if (! class_exists($model_namespace)) {
93
+        if (!class_exists($model_namespace)) {
94 94
             throw new \Exception('The model does not exist.', 500);
95 95
         }
96 96
 
97
-        if (! method_exists($model_namespace, 'hasCrudTrait')) {
97
+        if (!method_exists($model_namespace, 'hasCrudTrait')) {
98 98
             throw new \Exception('Please use CrudTrait on the model.', 500);
99 99
         }
100 100
 
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
     {
148 148
         $complete_route = $route.'.index';
149 149
 
150
-        if (! \Route::has($complete_route)) {
150
+        if (!\Route::has($complete_route)) {
151 151
             throw new \Exception('There are no routes for this route name.', 404);
152 152
         }
153 153
 
@@ -248,7 +248,7 @@  discard block
 block discarded – undo
248 248
      */
249 249
     public function getFirstOfItsTypeInArray($type, $array)
250 250
     {
251
-        return array_first($array, function ($item) use ($type) {
251
+        return array_first($array, function($item) use ($type) {
252 252
             return $item['type'] == $type;
253 253
         });
254 254
     }
@@ -265,8 +265,8 @@  discard block
 block discarded – undo
265 265
 
266 266
     public function sync($type, $fields, $attributes)
267 267
     {
268
-        if (! empty($this->{$type})) {
269
-            $this->{$type} = array_map(function ($field) use ($fields, $attributes) {
268
+        if (!empty($this->{$type})) {
269
+            $this->{$type} = array_map(function($field) use ($fields, $attributes) {
270 270
                 if (in_array($field['name'], (array) $fields)) {
271 271
                     $field = array_merge($field, $attributes);
272 272
                 }
@@ -298,8 +298,8 @@  discard block
 block discarded – undo
298 298
                 }
299 299
             }
300 300
 
301
-            return $this->{$items} = array_merge($elements, array_filter($this->{$items}, function ($item) use ($items) {
302
-                return ! in_array($item['name'], $this->sort[$items]);
301
+            return $this->{$items} = array_merge($elements, array_filter($this->{$items}, function($item) use ($items) {
302
+                return !in_array($item['name'], $this->sort[$items]);
303 303
             }));
304 304
         }
305 305
 
@@ -328,15 +328,15 @@  discard block
 block discarded – undo
328 328
     {
329 329
         $relationArray = explode('.', $relationString);
330 330
 
331
-        if (! isset($length)) {
331
+        if (!isset($length)) {
332 332
             $length = count($relationArray);
333 333
         }
334 334
 
335
-        if (! isset($model)) {
335
+        if (!isset($model)) {
336 336
             $model = $this->model;
337 337
         }
338 338
 
339
-        $result = array_reduce(array_splice($relationArray, 0, $length), function ($obj, $method) {
339
+        $result = array_reduce(array_splice($relationArray, 0, $length), function($obj, $method) {
340 340
             return $obj->$method()->getRelated();
341 341
         }, $model);
342 342
 
@@ -386,7 +386,7 @@  discard block
 block discarded – undo
386 386
         $relation = $model->{$firstRelationName};
387 387
 
388 388
         $results = [];
389
-        if (! empty($relation)) {
389
+        if (!empty($relation)) {
390 390
             if ($relation instanceof Collection) {
391 391
                 $currentResults = $relation->toArray();
392 392
             } else {
@@ -395,7 +395,7 @@  discard block
 block discarded – undo
395 395
 
396 396
             array_shift($relationArray);
397 397
 
398
-            if (! empty($relationArray)) {
398
+            if (!empty($relationArray)) {
399 399
                 foreach ($currentResults as $currentResult) {
400 400
                     $results = array_merge($results, $this->getRelationModelInstances($currentResult, implode('.', $relationArray)));
401 401
                 }
Please login to merge, or discard this patch.
src/resources/views/fields/select2_nested.blade.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -2,7 +2,7 @@
 block discarded – undo
2 2
 <div @include('crud::inc.field_wrapper_attributes') >
3 3
     <label>{!! $field['label'] !!}</label>
4 4
     @include('crud::inc.field_translatable_icon')
5
-    <?php $entity_model = $crud->getRelationModel($field['entity'],  - 1); ?>
5
+    <?php $entity_model = $crud->getRelationModel($field['entity'], - 1); ?>
6 6
     <select
7 7
         name="{{ $field['name'] }}"
8 8
         style="width: 100%"
Please login to merge, or discard this patch.