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 (#1935)
by Cristian
04:27 queued 13s
created
src/app/Library/CrudPanel/Traits/FakeColumns.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -24,19 +24,19 @@
 block discarded – undo
24 24
             if (isset($field['fake']) && $field['fake'] == true) {
25 25
                 // add it to the request in its appropriate variable - the one defined, if defined
26 26
                 if (isset($field['store_in'])) {
27
-                    if (! in_array($field['store_in'], $fakeFieldsColumnsArray, true)) {
27
+                    if (!in_array($field['store_in'], $fakeFieldsColumnsArray, true)) {
28 28
                         array_push($fakeFieldsColumnsArray, $field['store_in']);
29 29
                     }
30 30
                 } else {
31 31
                     //otherwise in the one defined in the $crud variable
32
-                    if (! in_array('extras', $fakeFieldsColumnsArray, true)) {
32
+                    if (!in_array('extras', $fakeFieldsColumnsArray, true)) {
33 33
                         array_push($fakeFieldsColumnsArray, 'extras');
34 34
                     }
35 35
                 }
36 36
             }
37 37
         }
38 38
 
39
-        if (! count($fakeFieldsColumnsArray)) {
39
+        if (!count($fakeFieldsColumnsArray)) {
40 40
             $fakeFieldsColumnsArray = ['extras'];
41 41
         }
42 42
 
Please login to merge, or discard this patch.
src/config/backpack/base.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@
 block discarded – undo
43 43
     ],
44 44
 
45 45
     // CSS files that are loaded in all pages, using Laravel's mix() helper
46
-    'mix_styles' => [ // file_path => manifest_directory_path
46
+    'mix_styles' => [// file_path => manifest_directory_path
47 47
         // 'css/app.css' => '',
48 48
     ],
49 49
 
Please login to merge, or discard this patch.
src/app/Models/Traits/SpatieTranslatable/SluggableObserver.php 1 patch
Unused Use Statements   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -2,8 +2,8 @@
 block discarded – undo
2 2
 
3 3
 namespace Backpack\CRUD\app\Models\Traits\SpatieTranslatable;
4 4
 
5
-use Illuminate\Database\Eloquent\Model;
6 5
 use Illuminate\Contracts\Events\Dispatcher;
6
+use Illuminate\Database\Eloquent\Model;
7 7
 
8 8
 class SluggableObserver extends \Cviebrock\EloquentSluggable\SluggableObserver
9 9
 {
Please login to merge, or discard this patch.
src/Stats.php 2 patches
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.
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -8,7 +8,7 @@  discard block
 block discarded – undo
8 8
      * Check if the application is running in normal conditions
9 9
      * (production env, not in console, not in unit tests).
10 10
      *
11
-     * @return void
11
+     * @return boolean
12 12
      */
13 13
     private function runningInProduction()
14 14
     {
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
      * @param string $url     URL to point the request at.
86 86
      * @param array  $payload The data you want sent to the URL.
87 87
      *
88
-     * @return void
88
+     * @return boolean
89 89
      */
90 90
     private function makeCurlRequest($method, $url, $payload)
91 91
     {
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/Traits/Views.php 1 patch
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -301,6 +301,9 @@  discard block
 block discarded – undo
301 301
         return $this->getShowView();
302 302
     }
303 303
 
304
+    /**
305
+     * @param string $view
306
+     */
304 307
     public function setPreviewView($view)
305 308
     {
306 309
         return $this->setShowView($view);
@@ -311,6 +314,9 @@  discard block
 block discarded – undo
311 314
         return $this->getEditView();
312 315
     }
313 316
 
317
+    /**
318
+     * @param string $view
319
+     */
314 320
     public function setUpdateView($view)
315 321
     {
316 322
         return $this->setEditView($view);
Please login to merge, or discard this patch.
src/BackpackServiceProvider.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -54,13 +54,13 @@  discard block
 block discarded – undo
54 54
     public function register()
55 55
     {
56 56
         // Bind the CrudPanel object to Laravel's service container
57
-        $this->app->singleton('crud', function ($app) {
57
+        $this->app->singleton('crud', function($app) {
58 58
             return new \Backpack\CRUD\app\Library\CrudPanel\CrudPanel($app);
59 59
         });
60 60
 
61 61
         // load a macro for Route,
62 62
         // helps developers load all routes for a CRUD resource in one line
63
-        if (! Route::hasMacro('crud')) {
63
+        if (!Route::hasMacro('crud')) {
64 64
             $this->addRouteMacro();
65 65
         }
66 66
 
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
         // }
82 82
 
83 83
         // map the elfinder prefix
84
-        if (! \Config::get('elfinder.route.prefix')) {
84
+        if (!\Config::get('elfinder.route.prefix')) {
85 85
             \Config::set('elfinder.route.prefix', \Config::get('backpack.base.route_prefix').'/elfinder');
86 86
         }
87 87
     }
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
         $middleware_key = config('backpack.base.middleware_key');
92 92
         $middleware_class = config('backpack.base.middleware_class');
93 93
 
94
-        if (! is_array($middleware_class)) {
94
+        if (!is_array($middleware_class)) {
95 95
             $router->pushMiddlewareToGroup($middleware_key, $middleware_class);
96 96
 
97 97
             return;
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
      */
196 196
     private function addRouteMacro()
197 197
     {
198
-        Route::macro('crud', function ($name, $controller) {
198
+        Route::macro('crud', function($name, $controller) {
199 199
             // put together the route name prefix,
200 200
             // as passed to the Route::group() statements
201 201
             $routeName = '';
Please login to merge, or discard this patch.
src/LicenseCheck.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@
 block discarded – undo
17 17
             return;
18 18
         }
19 19
 
20
-        if (! $this->validCode(config('backpack.base.license_code'))) {
20
+        if (!$this->validCode(config('backpack.base.license_code'))) {
21 21
             \Alert::add('warning', "<strong>You're using unlicensed software.</strong> Please ask your web developer to <a target='_blank' class='alert-link text-white' href='http://backpackforlaravel.com'>purchase a license code</a> to hide this message.");
22 22
         }
23 23
     }
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/Traits/Delete.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@
 block discarded – undo
19 19
      *
20 20
      * TODO: should this delete items with relations to it too?
21 21
      *
22
-     * @return bool True if the item was deleted.
22
+     * @return string True if the item was deleted.
23 23
      */
24 24
     public function delete($id)
25 25
     {
Please login to merge, or discard this patch.
src/app/Library/CrudPanel/Traits/Operations.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
     /**
57 57
      * Convenience method to make sure all calls are made to a particular operation.
58 58
      *
59
-     * @param string|array  $operation Operation name in string form
59
+     * @param string|array  $operations Operation name in string form
60 60
      * @param bool|\Closure $closure   Code that calls CrudPanel methods.
61 61
      *
62 62
      * @return void
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
      * Allc configurations are put inside that operation's namespace.
72 72
      * Ex: show.configuration.
73 73
      *
74
-     * @param string|array  $operation Operation name in string form
74
+     * @param string|array  $operations Operation name in string form
75 75
      * @param bool|\Closure $closure   Code that calls CrudPanel methods.
76 76
      *
77 77
      * @return void
Please login to merge, or discard this patch.