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 (#124)
by Owen
03:12
created
src/PanelTraits/FakeFields.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
                     // remove the fake field
42 42
                     array_pull($request, $fields[$k]['name']);
43 43
 
44
-                    if (! in_array($fields[$k]['store_in'], $fake_field_columns_to_encode, true)) {
44
+                    if (!in_array($fields[$k]['store_in'], $fake_field_columns_to_encode, true)) {
45 45
                         array_push($fake_field_columns_to_encode, $fields[$k]['store_in']);
46 46
                     }
47 47
                 } else {
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
                     // remove the fake field
53 53
                     array_pull($request, $fields[$k]['name']);
54 54
 
55
-                    if (! in_array('extras', $fake_field_columns_to_encode, true)) {
55
+                    if (!in_array('extras', $fake_field_columns_to_encode, true)) {
56 56
                         array_push($fake_field_columns_to_encode, 'extras');
57 57
                     }
58 58
                 }
Please login to merge, or discard this patch.
src/PanelTraits/AutoSet.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
     {
17 17
         $this->getDbColumnTypes();
18 18
 
19
-        array_map(function ($field) {
19
+        array_map(function($field) {
20 20
             // $this->labels[$field] = $this->makeLabel($field);
21 21
 
22 22
             $new_field = [
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
             $this->create_fields[$field] = $new_field;
31 31
             $this->update_fields[$field] = $new_field;
32 32
 
33
-            if (! in_array($field, $this->model->getHidden())) {
33
+            if (!in_array($field, $this->model->getHidden())) {
34 34
                 $this->columns[$field] = [
35 35
                     'name'  => $field,
36 36
                     'label' => ucfirst($field),
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
      */
64 64
     public function getFieldTypeFromDbColumnType($field)
65 65
     {
66
-        if (! array_key_exists($field, $this->db_column_types)) {
66
+        if (!array_key_exists($field, $this->db_column_types)) {
67 67
             return 'text';
68 68
         }
69 69
 
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
         $columns = \Schema::getColumnListing($this->model->getTable());
145 145
         $fillable = $this->model->getFillable();
146 146
 
147
-        if (! empty($fillable)) {
147
+        if (!empty($fillable)) {
148 148
             $columns = array_intersect($columns, $fillable);
149 149
         }
150 150
 
Please login to merge, or discard this patch.
src/resources/views/fields/date.blade.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@
 block discarded – undo
4 4
 <?php
5 5
     // if the column has been cast to Carbon or Date (using attribute casting)
6 6
     // get the value as a date string
7
-    if (isset($field['value']) && ( $field['value'] instanceof \Carbon\Carbon || $field['value'] instanceof \Jenssegers\Date\Date ))
7
+    if (isset($field['value']) && ($field['value'] instanceof \Carbon\Carbon || $field['value'] instanceof \Jenssegers\Date\Date))
8 8
     {
9 9
         $field['value'] = $field['value']->toDateString();
10 10
     }
Please login to merge, or discard this patch.
src/resources/views/fields/datetime.blade.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@
 block discarded – undo
4 4
 <?php
5 5
     // if the column has been cast to Carbon or Date (using attribute casting)
6 6
     // get the value as a date string
7
-    if (isset($field['value']) && ( $field['value'] instanceof \Carbon\Carbon || $field['value'] instanceof \Jenssegers\Date\Date ))
7
+    if (isset($field['value']) && ($field['value'] instanceof \Carbon\Carbon || $field['value'] instanceof \Jenssegers\Date\Date))
8 8
     {
9 9
         $field['value'] = $field['value']->toDateString();
10 10
     }
Please login to merge, or discard this patch.
src/CrudPanel.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
      */
71 71
     public function setModel($model_namespace)
72 72
     {
73
-        if (! class_exists($model_namespace)) {
73
+        if (!class_exists($model_namespace)) {
74 74
             throw new \Exception('This model does not exist.', 404);
75 75
         }
76 76
 
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
     {
113 113
         $complete_route = $route.'.index';
114 114
 
115
-        if (! \Route::has($complete_route)) {
115
+        if (!\Route::has($complete_route)) {
116 116
             throw new \Exception('There are no routes for this route name.', 404);
117 117
         }
118 118
 
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
      */
163 163
     public function getFirstOfItsTypeInArray($type, $array)
164 164
     {
165
-        return array_first($array, function ($item) use ($type) {
165
+        return array_first($array, function($item) use ($type) {
166 166
             return $item['type'] == $type;
167 167
         });
168 168
     }
@@ -179,8 +179,8 @@  discard block
 block discarded – undo
179 179
 
180 180
     public function sync($type, $fields, $attributes)
181 181
     {
182
-        if (! empty($this->{$type})) {
183
-            $this->{$type} = array_map(function ($field) use ($fields, $attributes) {
182
+        if (!empty($this->{$type})) {
183
+            $this->{$type} = array_map(function($field) use ($fields, $attributes) {
184 184
                 if (in_array($field['name'], (array) $fields)) {
185 185
                     $field = array_merge($field, $attributes);
186 186
                 }
@@ -206,8 +206,8 @@  discard block
 block discarded – undo
206 206
                 }
207 207
             }
208 208
 
209
-            return $this->{$items} = array_merge($elements, array_filter($this->{$items}, function ($item) use ($items) {
210
-                return ! in_array($item['name'], $this->sort[$items]);
209
+            return $this->{$items} = array_merge($elements, array_filter($this->{$items}, function($item) use ($items) {
210
+                return !in_array($item['name'], $this->sort[$items]);
211 211
             }));
212 212
         }
213 213
 
Please login to merge, or discard this patch.
src/PanelTraits/Read.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
     public function hasUploadFields($form)
76 76
     {
77 77
         $fields = $this->getFields($form);
78
-        $upload_fields = array_where($fields, function ($value, $key) {
78
+        $upload_fields = array_where($fields, function($value, $key) {
79 79
             return isset($value['upload']) && $value['upload'] == true;
80 80
         });
81 81
 
@@ -173,7 +173,7 @@  discard block
 block discarded – undo
173 173
      */
174 174
     public function getCellView($column, $entry)
175 175
     {
176
-        if (! isset($column['type'])) {
176
+        if (!isset($column['type'])) {
177 177
             return \View::make('crud::columns.text')->with('crud', $this)->with('column', $column)->with('entry', $entry)->render();
178 178
         } else {
179 179
             if (view()->exists('vendor.backpack.crud.columns.'.$column['type'])) {
Please login to merge, or discard this patch.
src/CrudTrait.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
         foreach ($columns as $key => $column) {
51 51
             $column_contents = $this->{$column};
52 52
 
53
-            if (! is_object($this->{$column})) {
53
+            if (!is_object($this->{$column})) {
54 54
                 $column_contents = json_decode($this->{$column});
55 55
             }
56 56
 
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
     {
74 74
         $model = '\\'.get_class($this);
75 75
 
76
-        if (! count($columns)) {
76
+        if (!count($columns)) {
77 77
             $columns = (property_exists($model, 'fakeColumns')) ? $this->fakeColumns : ['extras'];
78 78
         }
79 79
 
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
             $attribute_value = (array) $this->{$attribute_name};
164 164
             foreach ($files_to_clear as $key => $filename) {
165 165
                 \Storage::disk($disk)->delete($filename);
166
-                $attribute_value = array_where($attribute_value, function ($value, $key) use ($filename) {
166
+                $attribute_value = array_where($attribute_value, function($value, $key) use ($filename) {
167 167
                     return $value != $filename;
168 168
                 });
169 169
             }
Please login to merge, or discard this patch.
src/app/Http/Controllers/CrudController.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
         $this->data['title'] = ucfirst($this->crud->entity_name_plural);
38 38
 
39 39
         // get all entries if AJAX is not enabled
40
-        if (! $this->data['crud']->ajaxTable()) {
40
+        if (!$this->data['crud']->ajaxTable()) {
41 41
             $this->data['entries'] = $this->data['crud']->getEntries();
42 42
         }
43 43
 
@@ -220,7 +220,7 @@  discard block
 block discarded – undo
220 220
         // is somewhat superfluous.. however if we are POSTing, it makes sense to actually have data to post.
221 221
         // Perhaps the route shoud be better named to reflect this (e.g. just /model/{id}/revisions) (??)
222 222
         $revisionId = \Request::input('revision_id', false);
223
-        if (! $revisionId) {
223
+        if (!$revisionId) {
224 224
             abort(500, 'Can\'t restore revision without revision_id');
225 225
         } else {
226 226
             $this->crud->restoreRevision($id, $revisionId); // do the update
@@ -260,7 +260,7 @@  discard block
 block discarded – undo
260 260
     {
261 261
         $this->crud->hasAccessOrFail('reorder');
262 262
 
263
-        if (! $this->crud->isReorderEnabled()) {
263
+        if (!$this->crud->isReorderEnabled()) {
264 264
             abort(403, 'Reorder is disabled.');
265 265
         }
266 266
 
@@ -331,7 +331,7 @@  discard block
 block discarded – undo
331 331
         $dataTable = new DataTable($this->crud->query, $columns);
332 332
 
333 333
         // make the datatable use the column types instead of just echoing the text
334
-        $dataTable->setFormatRowFunction(function ($entry) {
334
+        $dataTable->setFormatRowFunction(function($entry) {
335 335
             // get the actual HTML for each row's cell
336 336
             $row_items = $this->crud->getRowViews($entry, $this->crud);
337 337
 
Please login to merge, or discard this patch.
src/PanelTraits/ViewsAndRestoresRevisions.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@
 block discarded – undo
21 21
             $revisionDate = date('Y-m-d', strtotime((string) $history->created_at));
22 22
 
23 23
             // Be sure to instantiate the initial grouping array
24
-            if (! array_key_exists($revisionDate, $revisions)) {
24
+            if (!array_key_exists($revisionDate, $revisions)) {
25 25
                 $revisions[$revisionDate] = [];
26 26
             }
27 27
 
Please login to merge, or discard this patch.