Completed
Pull Request — master (#2588)
by Muhlis
02:22
created
src/Form/Tab.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
         $all = $fields->toArray();
71 71
 
72 72
         foreach ($this->form->rows as $row) {
73
-            $rowFields = array_map(function ($field) {
73
+            $rowFields = array_map(function($field) {
74 74
                 return $field['element'];
75 75
             }, $row->getFields());
76 76
 
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
     public function getTabs()
105 105
     {
106 106
         // If there is no active tab, then active the first.
107
-        if ($this->tabs->filter(function ($tab) {
107
+        if ($this->tabs->filter(function($tab) {
108 108
             return $tab['active'];
109 109
         })->isEmpty()) {
110 110
             $first = $this->tabs->first();
Please login to merge, or discard this patch.
src/Form.php 2 patches
Doc Comments   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -265,7 +265,7 @@  discard block
 block discarded – undo
265 265
      *
266 266
      * @param $id
267 267
      *
268
-     * @return mixed
268
+     * @return boolean
269 269
      */
270 270
     public function destroy($id)
271 271
     {
@@ -369,7 +369,7 @@  discard block
 block discarded – undo
369 369
      *
370 370
      * @param array $data
371 371
      *
372
-     * @return mixed
372
+     * @return Response|null
373 373
      */
374 374
     protected function prepare($data = [])
375 375
     {
@@ -429,7 +429,7 @@  discard block
 block discarded – undo
429 429
     /**
430 430
      * Call submitted callback.
431 431
      *
432
-     * @return mixed
432
+     * @return Response|null
433 433
      */
434 434
     protected function callSubmitted()
435 435
     {
@@ -443,7 +443,7 @@  discard block
 block discarded – undo
443 443
     /**
444 444
      * Call saving callback.
445 445
      *
446
-     * @return mixed
446
+     * @return Response|null
447 447
      */
448 448
     protected function callSaving()
449 449
     {
@@ -457,7 +457,7 @@  discard block
 block discarded – undo
457 457
     /**
458 458
      * Callback after saving a Model.
459 459
      *
460
-     * @return mixed|null
460
+     * @return Response|null
461 461
      */
462 462
     protected function callSaved()
463 463
     {
@@ -551,7 +551,7 @@  discard block
 block discarded – undo
551 551
     /**
552 552
      * Get RedirectResponse after update.
553 553
      *
554
-     * @param mixed $key
554
+     * @param integer $key
555 555
      *
556 556
      * @return \Illuminate\Http\RedirectResponse
557 557
      */
Please login to merge, or discard this patch.
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -291,7 +291,7 @@  discard block
 block discarded – undo
291 291
      */
292 292
     public function destroy($id)
293 293
     {
294
-        collect(explode(',', $id))->filter()->each(function ($id) {
294
+        collect(explode(',', $id))->filter()->each(function($id) {
295 295
 
296 296
             $builder = $this->model()->newQuery();
297 297
 
@@ -330,9 +330,9 @@  discard block
 block discarded – undo
330 330
 
331 331
         $data = $model->toArray();
332 332
 
333
-        $this->builder->fields()->filter(function ($field) {
333
+        $this->builder->fields()->filter(function($field) {
334 334
             return $field instanceof Field\File;
335
-        })->each(function (Field\File $file) use ($data) {
335
+        })->each(function(Field\File $file) use ($data) {
336 336
             $file->setOriginal($data);
337 337
 
338 338
             $file->destroy();
@@ -357,7 +357,7 @@  discard block
 block discarded – undo
357 357
             return $response;
358 358
         }
359 359
 
360
-        DB::transaction(function () {
360
+        DB::transaction(function() {
361 361
             $inserts = $this->prepareInsert($this->updates);
362 362
 
363 363
             foreach ($inserts as $column => $value) {
@@ -560,7 +560,7 @@  discard block
 block discarded – undo
560 560
             return $response;
561 561
         }
562 562
 
563
-        DB::transaction(function () {
563
+        DB::transaction(function() {
564 564
             $updates = $this->prepareUpdate($this->updates);
565 565
 
566 566
             foreach ($updates as $column => $value) {
@@ -1029,7 +1029,7 @@  discard block
 block discarded – undo
1029 1029
     protected function getFieldByColumn($column)
1030 1030
     {
1031 1031
         return $this->builder->fields()->first(
1032
-            function (Field $field) use ($column) {
1032
+            function(Field $field) use ($column) {
1033 1033
                 if (is_array($field->column())) {
1034 1034
                     return in_array($column, $field->column());
1035 1035
                 }
@@ -1050,7 +1050,7 @@  discard block
 block discarded – undo
1050 1050
 
1051 1051
         $values = $this->model->toArray();
1052 1052
 
1053
-        $this->builder->fields()->each(function (Field $field) use ($values) {
1053
+        $this->builder->fields()->each(function(Field $field) use ($values) {
1054 1054
             $field->setOriginal($values);
1055 1055
         });
1056 1056
     }
@@ -1080,7 +1080,7 @@  discard block
 block discarded – undo
1080 1080
 
1081 1081
         $data = $this->model->toArray();
1082 1082
 
1083
-        $this->builder->fields()->each(function (Field $field) use ($data) {
1083
+        $this->builder->fields()->each(function(Field $field) use ($data) {
1084 1084
             if (!in_array($field->column(), $this->ignored)) {
1085 1085
                 $field->fill($data);
1086 1086
             }
@@ -1203,7 +1203,7 @@  discard block
 block discarded – undo
1203 1203
      */
1204 1204
     public function setWidth($fieldWidth = 8, $labelWidth = 2)
1205 1205
     {
1206
-        $this->builder()->fields()->each(function ($field) use ($fieldWidth, $labelWidth) {
1206
+        $this->builder()->fields()->each(function($field) use ($fieldWidth, $labelWidth) {
1207 1207
             /* @var Field $field  */
1208 1208
             $field->setWidth($fieldWidth, $labelWidth);
1209 1209
         });
Please login to merge, or discard this patch.
src/Admin.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -212,10 +212,10 @@
 block discarded – undo
212 212
             'middleware' => config('admin.route.middleware'),
213 213
         ];
214 214
 
215
-        app('router')->group($attributes, function ($router) {
215
+        app('router')->group($attributes, function($router) {
216 216
 
217 217
             /* @var \Illuminate\Routing\Router $router */
218
-            $router->group([], function ($router) {
218
+            $router->group([], function($router) {
219 219
 
220 220
                 /* @var \Illuminate\Routing\Router $router */
221 221
                 $router->resource('auth/users', 'UserController');
Please login to merge, or discard this patch.
src/Grid.php 2 patches
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -230,7 +230,7 @@  discard block
 block discarded – undo
230 230
      * Get or set option for grid.
231 231
      *
232 232
      * @param string $key
233
-     * @param mixed  $value
233
+     * @param boolean  $value
234 234
      *
235 235
      * @return $this|mixed
236 236
      */
@@ -360,7 +360,7 @@  discard block
 block discarded – undo
360 360
     /**
361 361
      * Get the grid paginator.
362 362
      *
363
-     * @return mixed
363
+     * @return Tools\Paginator
364 364
      */
365 365
     public function paginator()
366 366
     {
@@ -547,7 +547,7 @@  discard block
 block discarded – undo
547 547
      *
548 548
      * @param bool $toArray
549 549
      *
550
-     * @return array|Collection|mixed
550
+     * @return Collection
551 551
      */
552 552
     public function processFilter($toArray = true)
553 553
     {
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -335,7 +335,7 @@  discard block
 block discarded – undo
335 335
         $column = new Column($column, $label);
336 336
         $column->setGrid($this);
337 337
 
338
-        return tap($column, function ($value) {
338
+        return tap($column, function($value) {
339 339
             $this->columns->push($value);
340 340
         });
341 341
     }
@@ -352,7 +352,7 @@  discard block
 block discarded – undo
352 352
         $column = new Column($column, $label);
353 353
         $column->setGrid($this);
354 354
 
355
-        return tap($column, function ($value) {
355
+        return tap($column, function($value) {
356 356
             $this->columns->prepend($value);
357 357
         });
358 358
     }
@@ -471,7 +471,7 @@  discard block
 block discarded – undo
471 471
      */
472 472
     public function disableRowSelector()
473 473
     {
474
-        $this->tools(function ($tools) {
474
+        $this->tools(function($tools) {
475 475
             /* @var Grid\Tools $tools */
476 476
             $tools->disableBatchActions();
477 477
         });
@@ -514,7 +514,7 @@  discard block
 block discarded – undo
514 514
 
515 515
         Column::setOriginalGridModels($collection);
516 516
 
517
-        $this->columns->map(function (Column $column) use (&$data) {
517
+        $this->columns->map(function(Column $column) use (&$data) {
518 518
             $data = $column->fill($data);
519 519
 
520 520
             $this->columnNames[] = $column->getName();
@@ -610,7 +610,7 @@  discard block
 block discarded – undo
610 610
      */
611 611
     protected function buildRows(array $data)
612 612
     {
613
-        $this->rows = collect($data)->map(function ($model, $number) {
613
+        $this->rows = collect($data)->map(function($model, $number) {
614 614
             return new Row($number, $model);
615 615
         });
616 616
 
Please login to merge, or discard this patch.
src/Grid/Filter/Layout/Column.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -73,7 +73,7 @@
 block discarded – undo
73 73
      */
74 74
     public function removeFilterByID($id)
75 75
     {
76
-        $this->filters = $this->filters->reject(function (AbstractFilter $filter) use ($id) {
76
+        $this->filters = $this->filters->reject(function(AbstractFilter $filter) use ($id) {
77 77
             return $filter->getId() == $id;
78 78
         });
79 79
     }
Please login to merge, or discard this patch.
src/Grid/Filter.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -240,7 +240,7 @@  discard block
 block discarded – undo
240 240
      */
241 241
     protected function removeFilterByID($id)
242 242
     {
243
-        $this->filters = array_filter($this->filters, function (AbstractFilter $filter) use ($id) {
243
+        $this->filters = array_filter($this->filters, function(AbstractFilter $filter) use ($id) {
244 244
             return $filter->getId() != $id;
245 245
         });
246 246
     }
@@ -254,7 +254,7 @@  discard block
 block discarded – undo
254 254
     {
255 255
         $inputs = array_dot(Input::all());
256 256
 
257
-        $inputs = array_filter($inputs, function ($input) {
257
+        $inputs = array_filter($inputs, function($input) {
258 258
             return $input !== '' && !is_null($input);
259 259
         });
260 260
 
@@ -278,7 +278,7 @@  discard block
 block discarded – undo
278 278
             $conditions[] = $filter->condition($params);
279 279
         }
280 280
 
281
-        return tap(array_filter($conditions), function ($conditions) {
281
+        return tap(array_filter($conditions), function($conditions) {
282 282
             if (!empty($conditions)) {
283 283
                 $this->expand();
284 284
             }
@@ -296,9 +296,9 @@  discard block
 block discarded – undo
296 296
             return $inputs;
297 297
         }
298 298
 
299
-        $inputs = collect($inputs)->filter(function ($input, $key) {
299
+        $inputs = collect($inputs)->filter(function($input, $key) {
300 300
             return starts_with($key, "{$this->name}_");
301
-        })->mapWithKeys(function ($val, $key) {
301
+        })->mapWithKeys(function($val, $key) {
302 302
             $key = str_replace("{$this->name}_", '', $key);
303 303
 
304 304
             return [$key => $val];
@@ -351,7 +351,7 @@  discard block
 block discarded – undo
351 351
      */
352 352
     public function scope($key, $label = '')
353 353
     {
354
-        return tap(new Scope($key, $label), function (Scope $scope) {
354
+        return tap(new Scope($key, $label), function(Scope $scope) {
355 355
             return $this->scopes->push($scope);
356 356
         });
357 357
     }
@@ -375,7 +375,7 @@  discard block
 block discarded – undo
375 375
     {
376 376
         $key = request(Scope::QUERY_NAME);
377 377
 
378
-        return $this->scopes->first(function ($scope) use ($key) {
378
+        return $this->scopes->first(function($scope) use ($key) {
379 379
             return $scope->key == $key;
380 380
         });
381 381
     }
@@ -495,9 +495,9 @@  discard block
 block discarded – undo
495 495
 
496 496
         $columns->push($pageKey);
497 497
 
498
-        $groupNames = collect($this->filters)->filter(function ($filter) {
498
+        $groupNames = collect($this->filters)->filter(function($filter) {
499 499
             return $filter instanceof Group;
500
-        })->map(function (AbstractFilter $filter) {
500
+        })->map(function(AbstractFilter $filter) {
501 501
             return "{$filter->getId()}_group";
502 502
         });
503 503
 
Please login to merge, or discard this patch.
src/Middleware/LogOperation.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -66,7 +66,7 @@
 block discarded – undo
66 66
             return true;
67 67
         }
68 68
 
69
-        return $allowedMethods->map(function ($method) {
69
+        return $allowedMethods->map(function($method) {
70 70
             return strtoupper($method);
71 71
         })->contains($method);
72 72
     }
Please login to merge, or discard this patch.
src/Grid/Displayers/SwitchDisplay.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@
 block discarded – undo
30 30
         if ($keys->isEmpty()) {
31 31
             $key = $name;
32 32
         } else {
33
-            $key = $keys->shift().$keys->reduce(function ($carry, $val) {
33
+            $key = $keys->shift().$keys->reduce(function($carry, $val) {
34 34
                 return $carry."[$val]";
35 35
             });
36 36
         }
Please login to merge, or discard this patch.
src/Grid/Displayers/SwitchGroup.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@
 block discarded – undo
30 30
         if ($keys->isEmpty()) {
31 31
             $key = $name;
32 32
         } else {
33
-            $key = $keys->shift().$keys->reduce(function ($carry, $val) {
33
+            $key = $keys->shift().$keys->reduce(function($carry, $val) {
34 34
                 return $carry."[$val]";
35 35
             });
36 36
         }
Please login to merge, or discard this patch.