z-song / laravel-admin

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()
     {

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace Encore\Admin\Grid;
 
-use Carbon\Carbon;
 use Closure;
 use Encore\Admin\Actions\RowAction;
 use Encore\Admin\Grid;

Spacing +7 added lines, -7 removed lines

@@ -431,7 +431,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);
 
@@ -480,7 +480,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();
 
@@ -608,7 +608,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);
 
@@ -626,7 +626,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 {
@@ -663,7 +663,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);
             }
@@ -687,7 +687,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));
             });
         }
@@ -696,7 +696,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
      */

src/Widgets/ContainsForms.php

Doc Comments +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
      * @param array $forms
      * @param null  $active
      *
-     * @return mixed
+     * @return ContainsForms
      */
     public static function forms($forms, $active = null)
     {

src/Controllers/MenuController.php

Spacing +3 added lines, -3 removed lines

@@ -26,10 +26,10 @@ 
         return $content
             ->title(trans('admin.menu'))
             ->description(trans('admin.list'))
-            ->row(function (Row $row) {
+            ->row(function(Row $row) {
                 $row->column(6, $this->treeView()->render());
 
-                $row->column(6, function (Column $column) {
+                $row->column(6, function(Column $column) {
                     $form = new \Encore\Admin\Widgets\Form();
                     $form->action(admin_url('auth/menu'));
 
@@ -75,7 +75,7 @@ 
 
         $tree->disableCreate();
 
-        $tree->branch(function ($branch) {
+        $tree->branch(function($branch) {
             $payload = "<i class='fa {$branch['icon']}'></i>&nbsp;<strong>{$branch['title']}</strong>";
 
             if (!isset($branch['children'])) {

src/Console/GenerateMenuCommand.php

Spacing +4 added lines, -4 removed lines

@@ -48,7 +48,7 @@ 
      */
     public function handle()
     {
-        $routes = collect($this->router->getRoutes())->filter(function (Route $route) {
+        $routes = collect($this->router->getRoutes())->filter(function(Route $route) {
             $uri = $route->uri();
             // built-in, parameterized and no-GET are ignored
             return Str::startsWith($uri, 'admin/')
@@ -57,7 +57,7 @@ 
                 && !Str::contains($uri, '{')
                 && in_array('GET', $route->methods());
         })
-            ->map(function (Route $route) {
+            ->map(function(Route $route) {
                 $uri = substr($route->uri(), strlen('admin/'));
 
                 return [
@@ -71,7 +71,7 @@ 
 
         $menus = Menu::all()->pluck('title', 'uri');
         // exclude exist ones
-        $news = $routes->diffKeys($menus)->map(function ($item, $key) {
+        $news = $routes->diffKeys($menus)->map(function($item, $key) {
             return [
                 'title' => $item,
                 'uri'   => $key,
@@ -85,7 +85,7 @@ 
         } else {
             if ($this->hasOption('dry-run') && $this->option('dry-run')) {
                 $this->line('<info>The following menu items will be created</info>: ');
-                $this->table(['Title', 'Uri'], array_map(function ($item) {
+                $this->table(['Title', 'Uri'], array_map(function($item) {
                     return [
                         $item['title'],
                         $item['uri'],

src/Form/Field/MultipleFile.php

Spacing +2 added lines, -2 removed lines

@@ -154,7 +154,7 @@ 
 
         // for create or update
         if ($this->pathColumn) {
-            $targets = array_map(function ($target) {
+            $targets = array_map(function($target) {
                 return [$this->pathColumn => $target];
             }, $targets);
         }
@@ -185,7 +185,7 @@ 
     {
         $this->name = $this->getStoreName($file);
 
-        return tap($this->upload($file), function () {
+        return tap($this->upload($file), function() {
             $this->name = null;
         });
     }