z-song / laravel-admin

src/Grid.php

Doc Comments +5 added lines, -5 removed lines

@@ -250,7 +250,7 @@ 
      * Get or set option for grid.
      *
      * @param string $key
-     * @param mixed  $value
+     * @param boolean  $value
      *
      * @return $this|mixed
      */
@@ -440,7 +440,7 @@ 
     /**
      * Get the grid paginator.
      *
-     * @return mixed
+     * @return Tools\Paginator
      */
     public function paginator()
     {
@@ -622,7 +622,7 @@ 
      *
      * @param bool $toArray
      *
-     * @return array|Collection|mixed
+     * @return Collection
      */
     public function processFilter($toArray = true)
     {
@@ -872,7 +872,7 @@ 
      *
      * @param Closure|null $closure
      *
-     * @return $this|Closure
+     * @return callable
      */
     public function header(Closure $closure = null)
     {
@@ -904,7 +904,7 @@ 
      *
      * @param Closure|null $closure
      *
-     * @return $this|Closure
+     * @return callable
      */
     public function footer(Closure $closure = null)
     {

Spacing +9 added lines, -9 removed lines

@@ -330,7 +330,7 @@ 
 
             $label = empty($label) ? ucfirst($relationColumn) : $label;
 
-            $name = snake_case($relationName) . '.' . $relationColumn;
+            $name = snake_case($relationName).'.'.$relationColumn;
         }
 
         $column = $this->addColumn($name, $label);
@@ -388,7 +388,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);
         });
     }
@@ -408,7 +408,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);
         });
     }
@@ -426,7 +426,7 @@ 
         $column = new Column($column, $label);
         $column->setGrid($this);
 
-        return tap($column, function ($value) {
+        return tap($column, function($value) {
             $this->columns->push($value);
         });
     }
@@ -444,7 +444,7 @@ 
         $column = new Column($column, $label);
         $column->setGrid($this);
 
-        return tap($column, function ($value) {
+        return tap($column, function($value) {
             $this->columns->prepend($value);
         });
     }
@@ -601,7 +601,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();
@@ -705,7 +705,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);
         });
 
@@ -782,7 +782,7 @@ 
             $input = array_merge($input, $constraints);
         }
 
-        return $this->resource() . '?' . http_build_query($input);
+        return $this->resource().'?'.http_build_query($input);
     }
 
     /**
@@ -801,7 +801,7 @@ 
         return sprintf(
             '%s/create%s',
             $this->resource(),
-            $queryString ? ('?' . $queryString) : ''
+            $queryString ? ('?'.$queryString) : ''
         );
     }
 

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/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/Grid/Filter.php

Spacing +13 added lines, -13 removed lines

@@ -271,7 +271,7 @@ 
      */
     protected function removeFilterByID($id)
     {
-        $this->filters = array_filter($this->filters, function (AbstractFilter $filter) use ($id) {
+        $this->filters = array_filter($this->filters, function(AbstractFilter $filter) use ($id) {
             return $filter->getId() != $id;
         });
     }
@@ -285,7 +285,7 @@ 
     {
         $inputs = array_dot(Input::all());
 
-        $inputs = array_filter($inputs, function ($input) {
+        $inputs = array_filter($inputs, function($input) {
             return $input !== '' && !is_null($input);
         });
 
@@ -313,7 +313,7 @@ 
             }
         }
 
-        return tap(array_filter($conditions), function ($conditions) {
+        return tap(array_filter($conditions), function($conditions) {
             if (!empty($conditions)) {
                 $this->expand();
             }
@@ -331,9 +331,9 @@ 
             return $inputs;
         }
 
-        $inputs = collect($inputs)->filter(function ($input, $key) {
+        $inputs = collect($inputs)->filter(function($input, $key) {
             return starts_with($key, "{$this->name}_");
-        })->mapWithKeys(function ($val, $key) {
+        })->mapWithKeys(function($val, $key) {
             $key = str_replace("{$this->name}_", '', $key);
 
             return [$key => $val];
@@ -403,7 +403,7 @@ 
      */
     public function scope($key, $label = '')
     {
-        return tap(new Scope($key, $label), function (Scope $scope) {
+        return tap(new Scope($key, $label), function(Scope $scope) {
             return $this->scopes->push($scope);
         });
     }
@@ -427,7 +427,7 @@ 
     {
         $key = request(Scope::QUERY_NAME);
 
-        return $this->scopes->first(function ($scope) use ($key) {
+        return $this->scopes->first(function($scope) use ($key) {
             return $scope->key == $key;
         });
     }
@@ -547,9 +547,9 @@ 
 
         $columns->push($pageKey);
 
-        $groupNames = collect($this->filters)->filter(function ($filter) {
+        $groupNames = collect($this->filters)->filter(function($filter) {
             return $filter instanceof Group;
-        })->map(function (AbstractFilter $filter) {
+        })->map(function(AbstractFilter $filter) {
             return "{$filter->getId()}_group";
         });
 
@@ -581,17 +581,17 @@ 
             $keys = $keys->toArray();
         }
 
-        $keys = (array)$keys;
+        $keys = (array) $keys;
 
         $request = request();
 
         $query = $request->query();
         array_forget($query, $keys);
 
-        $question = $request->getBaseUrl() . $request->getPathInfo() == '/' ? '/?' : '?';
+        $question = $request->getBaseUrl().$request->getPathInfo() == '/' ? '/?' : '?';
 
         return count($request->query()) > 0
-            ? $request->url() . $question . http_build_query($query)
+            ? $request->url().$question.http_build_query($query)
             : $request->fullUrl();
     }
 
@@ -602,7 +602,7 @@ 
     public static function extend($name, $filterClass)
     {
         if (!is_subclass_of($filterClass, AbstractFilter::class)) {
-            throw new \InvalidArgumentException("The class [$filterClass] must be a type of " . AbstractFilter::class . '.');
+            throw new \InvalidArgumentException("The class [$filterClass] must be a type of ".AbstractFilter::class.'.');
         }
 
         static::$supports[$name] = $filterClass;