z-song / laravel-admin

src/Grid/Filter.php

Spacing +9 added lines, -9 removed lines

@@ -285,7 +285,7 @@ 
      */
     public 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;
         });
     }
@@ -299,7 +299,7 @@ 
     {
         $inputs = Arr::dot(request()->all());
 
-        $inputs = array_filter($inputs, function ($input) {
+        $inputs = array_filter($inputs, function($input) {
             return $input !== '' && !is_null($input);
         });
 
@@ -327,7 +327,7 @@ 
             }
         }
 
-        return tap(array_filter($conditions), function ($conditions) {
+        return tap(array_filter($conditions), function($conditions) {
             if (!empty($conditions)) {
                 $this->expand();
             }
@@ -345,9 +345,9 @@ 
             return $inputs;
         }
 
-        $inputs = collect($inputs)->filter(function ($input, $key) {
+        $inputs = collect($inputs)->filter(function($input, $key) {
             return Str::startsWith($key, "{$this->name}_");
-        })->mapWithKeys(function ($val, $key) {
+        })->mapWithKeys(function($val, $key) {
             $key = str_replace("{$this->name}_", '', $key);
 
             return [$key => $val];
@@ -417,7 +417,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);
         });
     }
@@ -441,7 +441,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;
         });
     }
@@ -566,9 +566,9 @@ 
 
         $columns->push($pageKey);
 
-        $groupNames = collect($this->filters)->filter(function ($filter) {
+        $groupNames = collect($this->filters)->filter(function($filter) {
             return $filter instanceof Filter\Group;
-        })->map(function (AbstractFilter $filter) {
+        })->map(function(AbstractFilter $filter) {
             return "{$filter->getId()}_group";
         });
 

src/Extension.php

Doc Comments +1 added lines, -1 removed lines

@@ -390,7 +390,7 @@
      *
      * @param       $name
      * @param       $slug
-     * @param       $path
+     * @param       string $path
      * @param array $methods
      */
     protected static function createPermission($name, $slug, $path, $methods = [])

Spacing +1 added lines, -1 removed lines

@@ -236,7 +236,7 @@
     public static function import()
     {
         $extension = static::getInstance();
-        DB::transaction(function () use ($extension) {
+        DB::transaction(function() use ($extension) {
             if ($menu = $extension->menu()) {
                 if ($extension->validateMenu($menu)) {
                     extract($menu);

src/Form/Layout/Layout.php

Doc Comments +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
     /**
      * Remove reserved fields from form layout.
      *
-     * @param array $fields
+     * @param string[] $fields
      */
     public function removeReservedFields(array $fields)
     {

src/Form/Layout/Column.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
      */
     public function removeFields($fields)
     {
-        $this->fields = $this->fields->reject(function (Field $field) use ($fields) {
+        $this->fields = $this->fields->reject(function(Field $field) use ($fields) {
             return in_array($field->column(), $fields);
         });
     }

src/Console/CreateUserCommand.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
         if (empty($selectedOption)) {
             $selected = $this->choice('Please choose a role for the user', $selectedOption, null, null, true);
 
-            $roles = $roles->filter(function ($role) use ($selected) {
+            $roles = $roles->filter(function($role) use ($selected) {
                 return in_array($role->name, $selected);
             });
         }

src/Actions/GridAction.php

Doc Comments +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     protected function getModelClass()
     {

src/Grid/Column.php

Doc Comments +2 added lines, -2 removed lines

@@ -88,7 +88,7 @@ 
     /**
      * Get name of this column.
      *
-     * @return mixed
+     * @return string
      */
     public function getName()
     {
@@ -112,7 +112,7 @@ 
     /**
      * Get label of the column.
      *
-     * @return mixed
+     * @return string
      */
     public function getLabel()
     {

Spacing +21 added lines, -21 removed lines

@@ -490,7 +490,7 @@ 
         $name = $this->getName();
         $query = request()->query();
 
-        $this->prefix(function ($_, $original) use ($name, $query) {
+        $this->prefix(function($_, $original) use ($name, $query) {
             Arr::set($query, $name, $original);
 
             $url = request()->fullUrlWithQuery($query);
@@ -541,7 +541,7 @@ 
 
         $column = $this;
 
-        return $this->display(function ($value) use ($grid, $column, $abstract, $arguments) {
+        return $this->display(function($value) use ($grid, $column, $abstract, $arguments) {
             /** @var AbstractDisplayer $displayer */
             $displayer = new $abstract($value, $grid, $column, $this);
 
@@ -559,7 +559,7 @@ 
      */
     public function using(array $values, $default = null)
     {
-        return $this->display(function ($value) use ($values, $default) {
+        return $this->display(function($value) use ($values, $default) {
             if (is_null($value)) {
                 return $default;
             }
@@ -577,7 +577,7 @@ 
      */
     public function replace(array $replacements)
     {
-        return $this->display(function ($value) use ($replacements) {
+        return $this->display(function($value) use ($replacements) {
             if (isset($replacements[$value])) {
                 return $replacements[$value];
             }
@@ -595,13 +595,13 @@ 
     public function repeat($input, $seperator = '')
     {
         if (is_string($input)) {
-            $input = function () use ($input) {
+            $input = function() use ($input) {
                 return $input;
             };
         }
 
         if ($input instanceof Closure) {
-            return $this->display(function ($value) use ($input, $seperator) {
+            return $this->display(function($value) use ($input, $seperator) {
                 return join($seperator, array_fill(0, (int) $value, $input->call($this, [$value])));
             });
         }
@@ -618,7 +618,7 @@ 
      */
     public function view($view)
     {
-        return $this->display(function ($value) use ($view) {
+        return $this->display(function($value) use ($view) {
             $model = $this;
 
             return view($view, compact('model', 'value'))->render();
@@ -658,7 +658,7 @@ 
      */
     public function filesize()
     {
-        return $this->display(function ($value) {
+        return $this->display(function($value) {
             return file_size($value);
         });
     }
@@ -672,7 +672,7 @@ 
      */
     public function gravatar($size = 30)
     {
-        return $this->display(function ($value) use ($size) {
+        return $this->display(function($value) use ($size) {
             $src = sprintf(
                 'https://www.gravatar.com/avatar/%s?s=%d',
                 md5(strtolower($value)),
@@ -693,7 +693,7 @@ 
      */
     public function loading($values = [], $others = [])
     {
-        return $this->display(function ($value) use ($values, $others) {
+        return $this->display(function($value) use ($values, $others) {
             $values = (array) $values;
 
             if (in_array($value, $values)) {
@@ -714,7 +714,7 @@ 
      */
     public function icon(array $setting, $default = '')
     {
-        return $this->display(function ($value) use ($setting, $default) {
+        return $this->display(function($value) use ($setting, $default) {
             $fa = '';
 
             if (isset($setting[$value])) {
@@ -740,7 +740,7 @@ 
             Carbon::setLocale($locale);
         }
 
-        return $this->display(function ($value) {
+        return $this->display(function($value) {
             return Carbon::parse($value)->diffForHumans();
         });
     }
@@ -769,7 +769,7 @@ 
      */
     public function bool(array $map = [], $default = false)
     {
-        return $this->display(function ($value) use ($map, $default) {
+        return $this->display(function($value) use ($map, $default) {
             $bool = empty($map) ? boolval($value) : Arr::get($map, $value, $default);
 
             return $bool ? '<i class="fa fa-check text-green"></i>' : '<i class="fa fa-close text-red"></i>';
@@ -784,7 +784,7 @@ 
      */
     public function default($default = '-')
     {
-        return $this->display(function ($value) use ($default) {
+        return $this->display(function($value) use ($default) {
             return $value ?: $default;
         });
     }
@@ -804,7 +804,7 @@ 
 
         $grid = $this->grid;
 
-        return $this->display(function ($_, $column) use ($action, $grid) {
+        return $this->display(function($_, $column) use ($action, $grid) {
             /** @var RowAction $action */
             $action = new $action();
 
@@ -826,7 +826,7 @@ 
      */
     public function dot($options = [], $default = '')
     {
-        return $this->prefix(function ($_, $original) use ($options, $default) {
+        return $this->prefix(function($_, $original) use ($options, $default) {
             if (is_null($original)) {
                 $style = $default;
             } else {
@@ -1037,7 +1037,7 @@ 
         $grid = $this->grid;
         $column = $this;
 
-        $this->display(function ($value) use ($grid, $column, $class) {
+        $this->display(function($value) use ($grid, $column, $class) {
             /** @var AbstractDisplayer $definition */
             $definition = new $class($value, $grid, $column, $this);
 
@@ -1055,7 +1055,7 @@ 
     protected function htmlEntityEncode($item)
     {
         if (is_array($item)) {
-            array_walk_recursive($item, function (&$value) {
+            array_walk_recursive($item, function(&$value) {
                 $value = htmlentities($value);
             });
         } else {
@@ -1092,7 +1092,7 @@ 
      */
     protected function callSupportDisplayer($abstract, $arguments)
     {
-        return $this->display(function ($value) use ($abstract, $arguments) {
+        return $this->display(function($value) use ($abstract, $arguments) {
             if (is_array($value) || $value instanceof Arrayable) {
                 return call_user_func_array([collect($value), $abstract], $arguments);
             }
@@ -1116,7 +1116,7 @@ 
     protected function callBuiltinDisplayer($abstract, $arguments)
     {
         if ($abstract instanceof Closure) {
-            return $this->display(function ($value) use ($abstract, $arguments) {
+            return $this->display(function($value) use ($abstract, $arguments) {
                 return $abstract->call($this, ...array_merge([$value], $arguments));
             });
         }
@@ -1125,7 +1125,7 @@ 
             $grid = $this->grid;
             $column = $this;
 
-            return $this->display(function ($value) use ($abstract, $grid, $column, $arguments) {
+            return $this->display(function($value) use ($abstract, $grid, $column, $arguments) {
                 /** @var AbstractDisplayer $displayer */
                 $displayer = new $abstract($value, $grid, $column, $this);
 

src/Auth/Database/HasPermissions.php

Spacing +1 added lines, -1 removed lines

@@ -112,7 +112,7 @@
      */
     protected static function bootHasPermissions()
     {
-        static::deleting(function ($model) {
+        static::deleting(function($model) {
             $model->roles()->detach();
 
             $model->permissions()->detach();

Doc Comments +5 added lines, -5 removed lines

@@ -9,7 +9,7 @@ 
     /**
      * Get all permissions of user.
      *
-     * @return mixed
+     * @return Collection
      */
     public function allPermissions(): Collection
     {
@@ -19,7 +19,7 @@ 
     /**
      * Check if user has permission.
      *
-     * @param $ability
+     * @param string $ability
      * @param array $arguments
      *
      * @return bool
@@ -56,7 +56,7 @@ 
     /**
      * Check if user is administrator.
      *
-     * @return mixed
+     * @return boolean
      */
     public function isAdministrator(): bool
     {
@@ -68,7 +68,7 @@ 
      *
      * @param string $role
      *
-     * @return mixed
+     * @return boolean
      */
     public function isRole(string $role): bool
     {
@@ -80,7 +80,7 @@ 
      *
      * @param array $roles
      *
-     * @return mixed
+     * @return boolean
      */
     public function inRoles(array $roles = []): bool
     {

src/Grid/Model.php

Spacing +10 added lines, -10 removed lines

@@ -347,9 +347,9 @@ 
 
         $this->setSort();
 
-        $this->queries->reject(function ($query) {
+        $this->queries->reject(function($query) {
             return $query['method'] == 'paginate';
-        })->each(function ($query) {
+        })->each(function($query) {
             $this->model = $this->model->{$query['method']}(...$query['arguments']);
         });
 
@@ -400,7 +400,7 @@ 
         $this->setSort();
         $this->setPaginate();
 
-        $this->queries->unique()->each(function ($query) {
+        $this->queries->unique()->each(function($query) {
             $this->model = call_user_func_array([$this->model, $query['method']], $query['arguments']);
         });
 
@@ -430,9 +430,9 @@ 
 
         $queryBuilder = $this->originalModel;
 
-        $this->queries->reject(function ($query) {
+        $this->queries->reject(function($query) {
             return in_array($query['method'], ['get', 'paginate']);
-        })->each(function ($query) use (&$queryBuilder) {
+        })->each(function($query) use (&$queryBuilder) {
             $queryBuilder = $queryBuilder->{$query['method']}(...$query['arguments']);
         });
 
@@ -466,7 +466,7 @@ 
     {
         $paginate = $this->findQueryByMethod('paginate');
 
-        $this->queries = $this->queries->reject(function ($query) {
+        $this->queries = $this->queries->reject(function($query) {
             return $query['method'] == 'paginate';
         });
 
@@ -524,7 +524,7 @@ 
      */
     protected function findQueryByMethod($method)
     {
-        return $this->queries->first(function ($query) use ($method) {
+        return $this->queries->first(function($query) use ($method) {
             return $query['method'] == $method;
         });
     }
@@ -547,7 +547,7 @@ 
         }
 
         $columnNameContainsDots = Str::contains($columnName, '.');
-        $isRelation = $this->queries->contains(function ($query) use ($columnName) {
+        $isRelation = $this->queries->contains(function($query) use ($columnName) {
             return $query['method'] === 'with' && in_array($columnName, $query['arguments'], true);
         });
         if ($columnNameContainsDots === true && $isRelation) {
@@ -593,7 +593,7 @@ 
     {
         list($relationName, $relationColumn) = explode('.', $column);
 
-        if ($this->queries->contains(function ($query) use ($relationName) {
+        if ($this->queries->contains(function($query) use ($relationName) {
             return $query['method'] == 'with' && in_array($relationName, $query['arguments']);
         })) {
             $relation = $this->model->$relationName();
@@ -627,7 +627,7 @@ 
      */
     public function resetOrderBy()
     {
-        $this->queries = $this->queries->reject(function ($query) {
+        $this->queries = $this->queries->reject(function($query) {
             return $query['method'] == 'orderBy' || $query['method'] == 'orderByDesc';
         });
     }

Doc Comments +3 added lines, -3 removed lines

@@ -467,7 +467,7 @@ 
     /**
      * Resolve perPage for pagination.
      *
-     * @param array|null $paginate
+     * @param Model $paginate
      *
      * @return array
      */
@@ -497,7 +497,7 @@ 
     /**
      * Find query by method name.
      *
-     * @param $method
+     * @param string $method
      *
      * @return static
      */
@@ -651,7 +651,7 @@ 
 
     /**
      * @param string $method
-     * @param array  $arguments
+     * @param integer[]  $arguments
      *
      * @return $this
      */