View GitHub Repository CRUD

Schedule Inspection
Subscribe

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 (#2220)
by Cristian
03:10
created
Status
Category
Toggle Patches
src/app/Http/Controllers/Operations/ListOperation.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -41,7 +41,7 @@  discard block
 
 block discarded – undo
41 41 
     {
42 42 
         $this->crud->allowAccess('list');
43 43
44 
-        $this->crud->operation('list', function () {
44 
+        $this->crud->operation('list', function() {
45 45 
             $this->crud->loadDefaultOperationSettingsFromConfig();
46 46 
         });
47 47 
     }
 
@@ -112,8 +112,8 @@  discard block
 
 block discarded – undo
112 112 
         // if there was an order set, this will be the last one (after all others were applied)
113 113 
         $orderBy = $this->crud->query->getQuery()->orders;
114 114 
         $hasOrderByPrimaryKey = false;
115 
-        collect($orderBy)->each(function ($item, $key) use ($hasOrderByPrimaryKey) {
116 
-            if (! isset($item['column'])) {
115 
+        collect($orderBy)->each(function($item, $key) use ($hasOrderByPrimaryKey) {
116 
+            if (!isset($item['column'])) {
117 117 
                 return false;
118 118 
             }
119 119 
 
@@ -123,7 +123,7 @@  discard block
 
 block discarded – undo
123 123 
                 return false;
124 124 
             }
125 125 
         });
126 
-        if (! $hasOrderByPrimaryKey) {
126 
+        if (!$hasOrderByPrimaryKey) {
127 127 
             $this->crud->query->orderByDesc($this->crud->model->getKeyName());
128 128 
         }
129 129
Please login to merge, or discard this patch.