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 (#1388)
by Thomas
08:55 queued 05:26
created
src/CrudTrait.php 3 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -69,7 +69,7 @@
 block discarded – undo
69 69
      *
70 70
      * @param array $columns - the database columns that contain the JSONs
71 71
      *
72
-     * @return Model
72
+     * @return CrudTrait
73 73
      */
74 74
     public function withFakes($columns = [])
75 75
     {
Please login to merge, or discard this patch.
Unused Use Statements   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -3,9 +3,9 @@
 block discarded – undo
3 3
 namespace Backpack\CRUD;
4 4
 
5 5
 use DB;
6
-use Traversable;
7
-use Illuminate\Support\Facades\Config;
8 6
 use Illuminate\Database\Eloquent\Model;
7
+use Illuminate\Support\Facades\Config;
8
+use Traversable;
9 9
 
10 10
 trait CrudTrait
11 11
 {
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
         $conn->getDoctrineSchemaManager()->getDatabasePlatform()->registerDoctrineTypeMapping('json', 'json_array');
58 58
         $conn->getDoctrineSchemaManager()->getDatabasePlatform()->registerDoctrineTypeMapping('jsonb', 'json_array');
59 59
 
60
-        return ! $conn->getDoctrineColumn($table, $column_name)->getNotnull();
60
+        return !$conn->getDoctrineColumn($table, $column_name)->getNotnull();
61 61
     }
62 62
 
63 63
     /*
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
         foreach ($columns as $key => $column) {
77 77
             $column_contents = $this->{$column};
78 78
 
79
-            if (! is_object($this->{$column})) {
79
+            if (!is_object($this->{$column})) {
80 80
                 $column_contents = json_decode($this->{$column});
81 81
             }
82 82
 
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
     {
100 100
         $model = '\\'.get_class($this);
101 101
 
102
-        if (! count($columns)) {
102
+        if (!count($columns)) {
103 103
             $columns = (property_exists($model, 'fakeColumns')) ? $this->fakeColumns : ['extras'];
104 104
         }
105 105
 
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
      */
117 117
     public function shouldEncodeFake($column)
118 118
     {
119
-        return ! in_array($column, array_keys($this->casts));
119
+        return !in_array($column, array_keys($this->casts));
120 120
     }
121 121
 
122 122
     /*
@@ -199,7 +199,7 @@  discard block
 block discarded – undo
199 199
             $attribute_value = (array) $this->{$attribute_name};
200 200
             foreach ($files_to_clear as $key => $filename) {
201 201
                 \Storage::disk($disk)->delete($filename);
202
-                $attribute_value = array_where($attribute_value, function ($value, $key) use ($filename) {
202
+                $attribute_value = array_where($attribute_value, function($value, $key) use ($filename) {
203 203
                     return $value != $filename;
204 204
                 });
205 205
             }
Please login to merge, or discard this patch.
src/resources/views-elfinder/filepicker.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@
 block discarded – undo
26 26
     <!-- Include jQuery, jQuery UI, elFinder (REQUIRED) -->
27 27
 
28 28
     <?php
29
-    $mimeTypes = implode(',', array_map(function ($t) {
29
+    $mimeTypes = implode(',', array_map(function($t) {
30 30
         return "'".$t."'";
31 31
     }, explode(',', $type)));
32 32
     ?>
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.
src/app/Http/Controllers/CrudFeatures/Reorder.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -174,7 +174,7 @@
 block discarded – undo
174 174
     {
175 175
         $this->crud->hasAccessOrFail('reorder');
176 176
 
177
-        if (! $this->crud->isReorderEnabled()) {
177
+        if (!$this->crud->isReorderEnabled()) {
178 178
             abort(403, 'Reorder is disabled.');
179 179
         }
180 180
 
Please login to merge, or discard this patch.
src/PanelTraits/AutoFocus.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -11,6 +11,9 @@
 block discarded – undo
11 11
         return $this->autoFocusOnFirstField;
12 12
     }
13 13
 
14
+    /**
15
+     * @param boolean $value
16
+     */
14 17
     public function setAutoFocusOnFirstField($value)
15 18
     {
16 19
         return $this->autoFocusOnFirstField = (bool) $value;
Please login to merge, or discard this patch.
src/resources/views/fields/video.blade.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1,11 +1,11 @@
 block discarded – undo
1 1
 <!-- text input -->
2 2
 <?php
3 3
 
4
-$value = old($field['name']) ? (old($field['name'])) : (isset($field['value']) ? ($field['value']) : (isset($field['default']) ? ($field['default']) : '' ));
4
+$value = old($field['name']) ? (old($field['name'])) : (isset($field['value']) ? ($field['value']) : (isset($field['default']) ? ($field['default']) : ''));
5 5
 
6 6
 // if attribute casting is used, convert to JSON
7 7
 if (is_array($value)) {
8
-    $value = json_encode((object)$value);
8
+    $value = json_encode((object) $value);
9 9
 } elseif (is_object($value)) {
10 10
     $value = json_encode($value);
11 11
 } else {
Please login to merge, or discard this patch.
src/PanelTraits/Filters.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -124,7 +124,7 @@
 block discarded – undo
124 124
 
125 125
     /**
126 126
      * Determine if the current CRUD action is a list operation (using standard or ajax DataTables).
127
-     * @return bool
127
+     * @return boolean|null
128 128
      */
129 129
     public function doingListOperation()
130 130
     {
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -10,7 +10,7 @@  discard block
 block discarded – undo
10 10
 
11 11
     public function filtersEnabled()
12 12
     {
13
-        return ! is_array($this->filters);
13
+        return !is_array($this->filters);
14 14
     }
15 15
 
16 16
     public function filtersDisabled()
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
         $this->enableFilters();
55 55
 
56 56
         // check if another filter with the same name exists
57
-        if (! isset($options['name'])) {
57
+        if (!isset($options['name'])) {
58 58
             abort(500, 'All your filters need names.');
59 59
         }
60 60
         if ($this->filters->contains('name', $options['name'])) {
@@ -131,7 +131,7 @@  discard block
 block discarded – undo
131 131
 
132 132
     public function removeFilter($name)
133 133
     {
134
-        $this->filters = $this->filters->reject(function ($filter) use ($name) {
134
+        $this->filters = $this->filters->reject(function($filter) use ($name) {
135 135
             return $filter->name == $name;
136 136
         });
137 137
     }
@@ -201,7 +201,7 @@  discard block
 block discarded – undo
201 201
         $this->type = $options['type'];
202 202
         $this->label = $options['label'];
203 203
 
204
-        if (! isset($options['placeholder'])) {
204
+        if (!isset($options['placeholder'])) {
205 205
             $this->placeholder = '';
206 206
         } else {
207 207
             $this->placeholder = $options['placeholder'];
@@ -218,16 +218,16 @@  discard block
 block discarded – undo
218 218
 
219 219
     public function checkOptionsIntegrity($options)
220 220
     {
221
-        if (! isset($options['name'])) {
221
+        if (!isset($options['name'])) {
222 222
             abort(500, 'Please make sure all your filters have names.');
223 223
         }
224
-        if (! isset($options['type'])) {
224
+        if (!isset($options['type'])) {
225 225
             abort(500, 'Please make sure all your filters have types.');
226 226
         }
227
-        if (! \View::exists('crud::filters.'.$options['type'])) {
227
+        if (!\View::exists('crud::filters.'.$options['type'])) {
228 228
             abort(500, 'No filter view named "'.$options['type'].'.blade.php" was found.');
229 229
         }
230
-        if (! isset($options['label'])) {
230
+        if (!isset($options['label'])) {
231 231
             abort(500, 'Please make sure all your filters have labels.');
232 232
         }
233 233
     }
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
@@ -3,7 +3,7 @@
 block discarded – undo
3 3
 <?php
4 4
 // if the column has been cast to Carbon or Date (using attribute casting)
5 5
 // get the value as a date string
6
-if (isset($field['value']) && ( $field['value'] instanceof \Carbon\Carbon || $field['value'] instanceof \Jenssegers\Date\Date )) {
6
+if (isset($field['value']) && ($field['value'] instanceof \Carbon\Carbon || $field['value'] instanceof \Jenssegers\Date\Date)) {
7 7
     $field['value'] = $field['value']->toDateString();
8 8
 }
9 9
 ?>
Please login to merge, or discard this patch.
src/resources/views/fields/datetime_picker.blade.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -3,11 +3,11 @@
 block discarded – undo
3 3
 <?php
4 4
 // if the column has been cast to Carbon or Date (using attribute casting)
5 5
 // get the value as a date string
6
-if (isset($field['value']) && ( $field['value'] instanceof \Carbon\Carbon || $field['value'] instanceof \Jenssegers\Date\Date )) {
6
+if (isset($field['value']) && ($field['value'] instanceof \Carbon\Carbon || $field['value'] instanceof \Jenssegers\Date\Date)) {
7 7
     $field['value'] = $field['value']->format('Y-m-d H:i:s');
8 8
 }
9 9
 
10
-    $field_language = isset($field['datetime_picker_options']['language'])?$field['datetime_picker_options']['language']:\App::getLocale();
10
+    $field_language = isset($field['datetime_picker_options']['language']) ? $field['datetime_picker_options']['language'] : \App::getLocale();
11 11
 ?>
12 12
 
13 13
 <div @include('crud::inc.field_wrapper_attributes') >
Please login to merge, or discard this patch.