z-song / laravel-admin

src/Layout/Column.php

Spacing +1 added lines, -1 removed lines

@@ -112,7 +112,7 @@
     protected function startColumn()
     {
         // get class name using width array
-        $classnName = collect($this->width)->map(function ($value, $key) {
+        $classnName = collect($this->width)->map(function($value, $key) {
             return "col-$key-$value";
         })->implode(' ');
 

src/Form/Field.php

Doc Comments +5 added lines, -5 removed lines

@@ -222,7 +222,7 @@ 
     /**
      * Field constructor.
      *
-     * @param       $column
+     * @param       string $column
      * @param array $arguments
      */
     public function __construct($column, $arguments = [])
@@ -556,7 +556,7 @@ 
     /**
      * Set or get value of the field.
      *
-     * @param null $value
+     * @param string $value
      *
      * @return mixed
      */
@@ -727,7 +727,7 @@ 
     /**
      * Add html attributes to elements.
      *
-     * @param array|string $attribute
+     * @param string $attribute
      * @param mixed        $value
      *
      * @return $this
@@ -1019,7 +1019,7 @@ 
     }
 
     /**
-     * @param array $labelClass
+     * @param string[] $labelClass
      *
      * @return self
      */
@@ -1072,7 +1072,7 @@ 
     /**
      * Set view of current field.
      *
-     * @return string
+     * @return Field
      */
     public function setView($view)
     {

src/Grid/Tools/ColumnSelector.php

Spacing +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
 
         $this->setupScript($show, $columns);
 
-        $lists = $columns->map(function ($val, $key) use ($show) {
+        $lists = $columns->map(function($val, $key) use ($show) {
             if (empty($show)) {
                 $checked = 'checked';
             } else {
@@ -73,7 +73,7 @@ 
      */
     protected function getGridColumns()
     {
-        return $this->grid->columns()->map(function (Grid\Column $column) {
+        return $this->grid->columns()->map(function(Grid\Column $column) {
             $name = $column->getName();
 
             if (in_array($name, ['__row_selector__', '__actions__'])) {

src/Grid.php

Spacing +6 added lines, -6 removed lines

@@ -371,7 +371,7 @@ 
 
         array_push($visible, '__row_selector__', '__actions__');
 
-        return $this->columns->filter(function (Column $column) use ($visible) {
+        return $this->columns->filter(function(Column $column) use ($visible) {
             return in_array($column->getName(), $visible);
         });
     }
@@ -391,7 +391,7 @@ 
 
         array_push($visible, '__row_selector__', '__actions__');
 
-        return collect($this->columnNames)->filter(function ($column) use ($visible) {
+        return collect($this->columnNames)->filter(function($column) use ($visible) {
             return in_array($column, $visible);
         });
     }
@@ -409,7 +409,7 @@ 
         $column = new Column($column, $label);
         $column->setGrid($this);
 
-        return tap($column, function ($value) {
+        return tap($column, function($value) {
             $this->columns->push($value);
         });
     }
@@ -427,7 +427,7 @@ 
         $column = new Column($column, $label);
         $column->setGrid($this);
 
-        return tap($column, function ($value) {
+        return tap($column, function($value) {
             $this->columns->prepend($value);
         });
     }
@@ -584,7 +584,7 @@ 
 
         Column::setOriginalGridModels($collection);
 
-        $this->columns->map(function (Column $column) use (&$data) {
+        $this->columns->map(function(Column $column) use (&$data) {
             $data = $column->fill($data);
 
             $this->columnNames[] = $column->getName();
@@ -688,7 +688,7 @@ 
      */
     protected function buildRows(array $data)
     {
-        $this->rows = collect($data)->map(function ($model, $number) {
+        $this->rows = collect($data)->map(function($model, $number) {
             return new Row($number, $model);
         });
 

Doc Comments +6 added lines, -6 removed lines

@@ -276,7 +276,7 @@ 
      * Get or set option for grid.
      *
      * @param string $key
-     * @param mixed  $value
+     * @param boolean  $value
      *
      * @return $this|mixed
      */
@@ -466,7 +466,7 @@ 
     /**
      * Get the grid paginator.
      *
-     * @return mixed
+     * @return Tools\Paginator
      */
     public function paginator()
     {
@@ -518,7 +518,7 @@ 
     /**
      * Set grid action callback.
      *
-     * @param Closure|string $actions
+     * @param Closure $actions
      *
      * @return $this
      */
@@ -645,7 +645,7 @@ 
      *
      * @param bool $toArray
      *
-     * @return array|Collection|mixed
+     * @return Collection
      */
     public function processFilter($toArray = true)
     {
@@ -895,7 +895,7 @@ 
      *
      * @param Closure|null $closure
      *
-     * @return $this|Closure
+     * @return callable
      */
     public function header(Closure $closure = null)
     {
@@ -927,7 +927,7 @@ 
      *
      * @param Closure|null $closure
      *
-     * @return $this|Closure
+     * @return callable
      */
     public function footer(Closure $closure = null)
     {

src/Grid/Tools.php

Spacing +5 added lines, -5 removed lines

@@ -85,7 +85,7 @@ 
      */
     public function disableFilterButton(bool $disable = true)
     {
-        $this->tools = $this->tools->map(function (AbstractTool $tool) use ($disable) {
+        $this->tools = $this->tools->map(function(AbstractTool $tool) use ($disable) {
             if ($tool instanceof FilterButton) {
                 return $tool->disable($disable);
             }
@@ -100,7 +100,7 @@ 
      */
     public function disableRefreshButton(bool $disable = true)
     {
-        $this->tools = $this->tools->map(function (AbstractTool $tool) use ($disable) {
+        $this->tools = $this->tools->map(function(AbstractTool $tool) use ($disable) {
             if ($tool instanceof RefreshButton) {
                 return $tool->disable($disable);
             }
@@ -115,7 +115,7 @@ 
      */
     public function disableBatchActions(bool $disable = true)
     {
-        $this->tools = $this->tools->map(function ($tool) use ($disable) {
+        $this->tools = $this->tools->map(function($tool) use ($disable) {
             if ($tool instanceof BatchActions) {
                 return $tool->disable($disable);
             }
@@ -128,7 +128,7 @@ 
      */
     public function batch(\Closure $closure)
     {
-        call_user_func($closure, $this->tools->first(function ($tool) {
+        call_user_func($closure, $this->tools->first(function($tool) {
             return $tool instanceof BatchActions;
         }));
     }
@@ -140,7 +140,7 @@ 
      */
     public function render()
     {
-        return $this->tools->map(function ($tool) {
+        return $this->tools->map(function($tool) {
             if ($tool instanceof AbstractTool) {
                 if (!$tool->allowed()) {
                     return '';

src/Form/Field/Embeds.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Encore\Admin\Form\EmbeddedForm;
 use Encore\Admin\Form\Field;
 use Illuminate\Support\Facades\Validator;
-use Illuminate\Support\Str;
 
 if (!function_exists('Encore\Admin\Form\Field\array_key_attach_str')) {
     function array_key_attach_str(array $a, string $b, string $c = '.')

Spacing +16 added lines, -16 removed lines

@@ -12,7 +12,7 @@ 
     {
         return call_user_func_array(
             'array_merge',
-            array_map(function ($u, $v) use ($b, $c) {
+            array_map(function($u, $v) use ($b, $c) {
                 return ["{$b}{$c}{$u}" => $v];
             }, array_keys($a), array_values($a))
         );
@@ -44,7 +44,7 @@ 
 if (!function_exists('Encore\Admin\Form\Field\array_key_clean')) {
     function array_key_clean(array $a)
     {
-        $a = count($a) ? call_user_func_array('array_merge', array_map(function ($k, $v) {
+        $a = count($a) ? call_user_func_array('array_merge', array_map(function($k, $v) {
             return [str_replace(':', '', $k) => $v];
         }, array_keys($a), array_values($a))) : $a;
 
@@ -157,13 +157,13 @@ 
              *      'extra.end_atend'     => "$label[end_at]"
              * ]
              */
-            $newColumn = array_map(function ($k, $v) use ($rel) {
+            $newColumn = array_map(function($k, $v) use ($rel) {
                 //Fix ResetInput Function! A Headache Implementation!
                 return !$k ? "{$rel}.{$v}" : "{$rel}.{$v}:{$k}";
             }, array_keys($columns), array_values($columns));
 
             $fieldRules = is_array($fieldRules) ? implode('|', $fieldRules) : $fieldRules;
-            $newRules = array_map(function ($v) use ($fieldRules, $availInput) {
+            $newRules = array_map(function($v) use ($fieldRules, $availInput) {
                 list($k, $c) = explode('.', $v);
                 //Fix ResetInput Function! A Headache Implementation!
                 $col = explode(':', $c)[0];
@@ -244,20 +244,20 @@ 
              *      'extra.end_atend'     => "$label[end_at]"
              * ]
              */
-            $newColumn = array_map(function ($k, $v) use ($rel) {
+            $newColumn = array_map(function($k, $v) use ($rel) {
                 //Fix ResetInput Function! A Headache Implementation!
                 return !$k ? "{$rel}.{$v}" : "{$rel}.{$v}:{$k}";
             }, array_keys($columns), array_values($columns));
 
-            $newAttributes = array_map(function ($v) use ($field, $availInput) {
+            $newAttributes = array_map(function($v) use ($field, $availInput) {
                 list($k, $c) = explode('.', $v);
                 //Fix ResetInput Function! A Headache Implementation!
                 $col = explode(':', $c)[0];
                 if (array_key_exists($col, $availInput[$k]) && is_array($availInput[$k][$col])) {
-                    return call_user_func_array('array_merge', array_map(function ($u) use ($v, $field) {
+                    return call_user_func_array('array_merge', array_map(function($u) use ($v, $field) {
                         $w = $field->label();
                         //Fix ResetInput Function! A Headache Implementation!
-                        $w .= is_array($field->column()) ? '[' . explode(':', explode('.', $v)[2])[0] . ']' : '';
+                        $w .= is_array($field->column()) ? '['.explode(':', explode('.', $v)[2])[0].']' : '';
 
                         return ["{$v}:{$u}" => $w];
                     }, array_keys($availInput[$k][$col])));
@@ -266,7 +266,7 @@ 
                 //May Have Problem in Dealing with File Upload in Edit Mode
                 $w = $field->label();
                 //Fix ResetInput Function! A Headache Implementation!
-                $w .= is_array($field->column()) ? '[' . explode(':', explode('.', $v)[2])[0] . ']' : '';
+                $w .= is_array($field->column()) ? '['.explode(':', explode('.', $v)[2])[0].']' : '';
 
                 return [$v => $w];
             }, $newColumn);
@@ -333,12 +333,12 @@ 
              *     'extra.end_atend' => $value
              * ]
              */
-            $newColumn = array_map(function ($k, $v) use ($rel) {
+            $newColumn = array_map(function($k, $v) use ($rel) {
                 //Fix ResetInput Function! A Headache Implementation!
                 return !$k ? "{$rel}.{$v}" : "{$rel}.{$v}:{$k}";
             }, array_keys($columns), array_values($columns));
 
-            $newInput = array_map(function ($v) use ($availInput) {
+            $newInput = array_map(function($v) use ($availInput) {
                 list($k, $c) = explode('.', $v);
                 //Fix ResetInput Function! A Headache Implementation!
                 $col = explode(':', $c)[0];
@@ -396,20 +396,20 @@ 
                 $availInput[$column] = $availInput[$column] ?: null;
             }
 
-            $newColumn = array_map(function ($k, $v) use ($rel) {
+            $newColumn = array_map(function($k, $v) use ($rel) {
                 //Fix ResetInput Function! A Headache Implementation!
                 return !$k ? "{$rel}.{$v}" : "{$rel}.{$v}:{$k}";
             }, array_keys($columns), array_values($columns));
 
             if ($field->validationMessages) {
-                $newMessages = array_map(function ($v) use ($field, $availInput) {
+                $newMessages = array_map(function($v) use ($field, $availInput) {
                     list($k, $c) = explode('.', $v);
                     //Fix ResetInput Function! A Headache Implementation!
                     $col = explode(':', $c)[0];
                     if (array_key_exists($col, $availInput[$k]) && is_array($availInput[$k][$col])) {
                         return call_user_func_array(
                             'array_merge',
-                            array_map(function ($u) use (
+                            array_map(function($u) use (
                                 $v,
                                 $field
                             ) {
@@ -477,7 +477,7 @@ 
             return json_decode($this->value, true);
         }
 
-        return (array)$this->value;
+        return (array) $this->value;
     }
 
     /**
@@ -496,7 +496,7 @@ 
         //Fix the Bug of Embeds Fields Cannot be used within HasMany
         if ($this->elementName) {
             list($rel, $key, $col) = explode('.', $this->errorKey);
-            $form->fields()->each(function (Field $field) use ($rel, $key, $col) {
+            $form->fields()->each(function(Field $field) use ($rel, $key, $col) {
                 $column = $field->column();
                 $elementName = $elementClass = $errorKey = [];
                 if (is_array($column)) {