Completed
Pull Request — master (#4674)
by
unknown
02:34
created
src/Grid/Tools/FixColumns.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -78,7 +78,7 @@
 block discarded – undo
78 78
     {
79 79
         $this->grid->setView($this->view);
80 80
 
81
-        return function (Grid $grid) {
81
+        return function(Grid $grid) {
82 82
             if ($this->head > 0) {
83 83
                 $this->left = $grid->visibleColumns()->slice(0, $this->head);
84 84
             }
Please login to merge, or discard this patch.
src/Actions/Action.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
     /**
123 123
      * @param string $prefix
124 124
      *
125
-     * @return mixed|string
125
+     * @return string
126 126
      */
127 127
     public function selector($prefix)
128 128
     {
@@ -212,7 +212,7 @@  discard block
 block discarded – undo
212 212
     }
213 213
 
214 214
     /**
215
-     * @return mixed
215
+     * @return string
216 216
      */
217 217
     public function getCalledClass()
218 218
     {
@@ -403,7 +403,7 @@  discard block
 block discarded – undo
403 403
     }
404 404
 
405 405
     /**
406
-     * @return mixed
406
+     * @return string
407 407
      */
408 408
     public function render()
409 409
     {
Please login to merge, or discard this patch.
src/Grid/Tools/QuickCreate.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -34,6 +34,9 @@
 block discarded – undo
34 34
         $this->fields = Collection::make();
35 35
     }
36 36
 
37
+    /**
38
+     * @param string $placeholder
39
+     */
37 40
     protected function formatPlaceholder($placeholder)
38 41
     {
39 42
         return array_filter((array) $placeholder);
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
@@ -268,7 +268,7 @@  discard block
 block discarded – undo
268 268
      * Get or set option for grid.
269 269
      *
270 270
      * @param string $key
271
-     * @param mixed  $value
271
+     * @param boolean  $value
272 272
      *
273 273
      * @return $this|mixed
274 274
      */
@@ -453,7 +453,7 @@  discard block
 block discarded – undo
453 453
     /**
454 454
      * Get the grid paginator.
455 455
      *
456
-     * @return mixed
456
+     * @return Tools\Paginator
457 457
      */
458 458
     public function paginator()
459 459
     {
@@ -807,7 +807,7 @@  discard block
 block discarded – undo
807 807
     /**
808 808
      * Dynamically add columns to the grid view.
809 809
      *
810
-     * @param $method
810
+     * @param string $method
811 811
      * @param $arguments
812 812
      *
813 813
      * @return Column
Please login to merge, or discard this patch.
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -312,7 +312,7 @@  discard block
 block discarded – undo
312 312
         $column = new Column($column, $label);
313 313
         $column->setGrid($this);
314 314
 
315
-        return tap($column, function ($value) {
315
+        return tap($column, function($value) {
316 316
             $this->columns->push($value);
317 317
         });
318 318
     }
@@ -386,7 +386,7 @@  discard block
 block discarded – undo
386 386
         $column = new Column($column, $label);
387 387
         $column->setGrid($this);
388 388
 
389
-        return tap($column, function ($value) {
389
+        return tap($column, function($value) {
390 390
             $this->columns->prepend($value);
391 391
         });
392 392
     }
@@ -564,7 +564,7 @@  discard block
 block discarded – undo
564 564
 
565 565
         $data = $collection->toArray();
566 566
 
567
-        $this->columns->map(function (Column $column) use (&$data) {
567
+        $this->columns->map(function(Column $column) use (&$data) {
568 568
             $data = $column->fill($data);
569 569
 
570 570
             $this->columnNames[] = $column->getName();
@@ -584,7 +584,7 @@  discard block
 block discarded – undo
584 584
      */
585 585
     protected function buildRows(array $data)
586 586
     {
587
-        $this->rows = collect($data)->map(function ($model, $number) {
587
+        $this->rows = collect($data)->map(function($model, $number) {
588 588
             return new Row($number, $model, $this->keyName);
589 589
         });
590 590
 
@@ -614,7 +614,7 @@  discard block
 block discarded – undo
614 614
      *
615 615
      * @param array $params
616 616
      */
617
-    public function setCreateUrlQuery(array $params){
617
+    public function setCreateUrlQuery(array $params) {
618 618
         $this->variables['create'] = $params;
619 619
     }
620 620
 
@@ -636,7 +636,7 @@  discard block
 block discarded – undo
636 636
             $queryData += $this->variables['create'];
637 637
         }
638 638
 
639
-        if(!empty($queryData)) {
639
+        if (!empty($queryData)) {
640 640
             $queryString = http_build_query($queryData);
641 641
         }
642 642
 
Please login to merge, or discard this patch.
src/Grid/Concerns/HasActions.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@
 block discarded – undo
24 24
     /**
25 25
      * Set grid action callback.
26 26
      *
27
-     * @param Closure|string $actions
27
+     * @param Closure $actions
28 28
      *
29 29
      * @return $this
30 30
      */
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -88,7 +88,7 @@
 block discarded – undo
88 88
      */
89 89
     public function batchActions(Closure $closure)
90 90
     {
91
-        $this->tools(function (Grid\Tools $tools) use ($closure) {
91
+        $this->tools(function(Grid\Tools $tools) use ($closure) {
92 92
             $tools->batch($closure);
93 93
         });
94 94
 
Please login to merge, or discard this patch.
src/Grid/Tools/PerPageSelector.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -67,7 +67,7 @@
 block discarded – undo
67 67
     {
68 68
         Admin::script($this->script());
69 69
 
70
-        $options = $this->getOptions()->map(function ($option) {
70
+        $options = $this->getOptions()->map(function($option) {
71 71
             $selected = ($option == $this->perPage) ? 'selected' : '';
72 72
             $url = \request()->fullUrlWithQuery([$this->perPageName => $option]);
73 73
 
Please login to merge, or discard this patch.
src/Grid/Tools/Selector.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -106,7 +106,7 @@
 block discarded – undo
106 106
             return [];
107 107
         }
108 108
 
109
-        $selected = array_filter($selected, function ($value) {
109
+        $selected = array_filter($selected, function($value) {
110 110
             return !is_null($value);
111 111
         });
112 112
 
Please login to merge, or discard this patch.
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@
 block discarded – undo
57 57
      * @param string $column
58 58
      * @param string $label
59 59
      * @param array  $options
60
-     * @param null   $query
60
+     * @param null|\Closure   $query
61 61
      * @param string $type
62 62
      *
63 63
      * @return $this
Please login to merge, or discard this patch.
src/Grid/Concerns/HasSelector.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
 
27 27
         call_user_func($closure, $this->selector);
28 28
 
29
-        $this->header(function () {
29
+        $this->header(function() {
30 30
             return $this->renderSelector();
31 31
         });
32 32
 
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
 
47 47
         $active = Selector::parseSelected();
48 48
 
49
-        $this->selector->getSelectors()->each(function ($selector, $column) use ($active) {
49
+        $this->selector->getSelectors()->each(function($selector, $column) use ($active) {
50 50
             if (!array_key_exists($column, $active)) {
51 51
                 return;
52 52
             }
Please login to merge, or discard this patch.
src/Grid/Displayers/Label.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@
 block discarded – undo
13 13
             $this->value = $this->value->toArray();
14 14
         }
15 15
 
16
-        return collect((array) $this->value)->map(function ($item) use ($style) {
16
+        return collect((array) $this->value)->map(function($item) use ($style) {
17 17
             if (is_array($style)) {
18 18
                 if (is_string($this->getColumn()->getOriginal()) || is_int($this->getColumn()->getOriginal())) {
19 19
                     $style = Arr::get($style, $this->getColumn()->getOriginal(), 'success');
Please login to merge, or discard this patch.