z-song / laravel-admin

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;
         });
     }

src/Grid/Actions/Delete.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
         ];
 
         try {
-            DB::transaction(function () use ($model) {
+            DB::transaction(function() use ($model) {
                 $model->delete();
             });
         } catch (\Exception $exception) {

src/Grid/Exporters/CsvExporter.php

Spacing +4 added lines, -4 removed lines

@@ -139,16 +139,16 @@ 
             call_user_func($this->callback, $this);
         }
 
-        $response = function () {
+        $response = function() {
             $handle = fopen('php://output', 'w');
             $titles = [];
 
-            $this->chunk(function ($collection) use ($handle, &$titles) {
+            $this->chunk(function($collection) use ($handle, &$titles) {
                 Column::setOriginalGridModels($collection);
 
                 $original = $current = $collection->toArray();
 
-                $this->grid->getColumns()->map(function (Column $column) use (&$current) {
+                $this->grid->getColumns()->map(function(Column $column) use (&$current) {
                     $current = $column->fill($current);
                     $this->grid->columnNames[] = $column->getName();
                 });
@@ -177,7 +177,7 @@ 
     protected function getVisiableTitles()
     {
         $titles = $this->grid->visibleColumns()
-            ->mapWithKeys(function (Column $column) {
+            ->mapWithKeys(function(Column $column) {
                 return [$column->getName() => $column->getLabel()];
             });
 

src/Grid/Column.php

Doc Comments +3 added lines, -3 removed lines

@@ -359,7 +359,7 @@ 
     /**
      * Get name of this column.
      *
-     * @return mixed
+     * @return string
      */
     public function getName()
     {
@@ -397,7 +397,7 @@ 
     /**
      * Get label of the column.
      *
-     * @return mixed
+     * @return string
      */
     public function getLabel()
     {
@@ -535,7 +535,7 @@ 
      * Display using display abstract.
      *
      * @param string $abstract
-     * @param array  $arguments
+     * @param string[]  $arguments
      *
      * @return $this
      */

Spacing +22 added lines, -22 removed lines

@@ -492,7 +492,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);
@@ -543,7 +543,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);
 
@@ -561,7 +561,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;
             }
@@ -579,7 +579,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];
             }
@@ -597,13 +597,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) {
                 $callable = $input->bindTo($this);
                 return join($seperator, array_fill(0, (int) $value, $callable($value)));
             });
@@ -621,7 +621,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();
@@ -661,7 +661,7 @@ 
      */
     public function filesize()
     {
-        return $this->display(function ($value) {
+        return $this->display(function($value) {
             return file_size($value);
         });
     }
@@ -675,7 +675,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)),
@@ -696,7 +696,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)) {
@@ -717,7 +717,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])) {
@@ -743,7 +743,7 @@ 
             Carbon::setLocale($locale);
         }
 
-        return $this->display(function ($value) {
+        return $this->display(function($value) {
             return Carbon::parse($value)->diffForHumans();
         });
     }
@@ -757,7 +757,7 @@ 
      */
     public function date($format)
     {
-        return $this->display(function ($value) use ($format) {
+        return $this->display(function($value) use ($format) {
             return date($format, strtotime($value));
         });
     }
@@ -772,7 +772,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>';
@@ -787,7 +787,7 @@ 
      */
     public function default($default = '-')
     {
-        return $this->display(function ($value) use ($default) {
+        return $this->display(function($value) use ($default) {
             return $value ?: $default;
         });
     }
@@ -807,7 +807,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();
 
@@ -829,7 +829,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 {
@@ -984,7 +984,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);
 
@@ -1002,7 +1002,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 {
@@ -1039,7 +1039,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);
             }
@@ -1063,7 +1063,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));
             });
         }
@@ -1072,7 +1072,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/Grid/Selectable.php

Spacing +2 added lines, -2 removed lines

@@ -86,7 +86,7 @@ 
             $this->disablePagination();
         }
 
-        $this->tools(function (Tools $tools) {
+        $this->tools(function(Tools $tools) {
             $tools->append(new Grid\Selectable\BrowserBtn());
         });
 
@@ -98,7 +98,7 @@ 
         $hide = $hide ? 'hide' : '';
         $key = $this->key;
 
-        $this->column('__remove__', ' ')->display(function () use ($hide, $key) {
+        $this->column('__remove__', ' ')->display(function() use ($hide, $key) {
             return <<<BTN
 <a href="javascript:void(0);" class="grid-row-remove {$hide}" data-key="{$this->getAttribute($key)}">
     <i class="fa fa-trash"></i>

Doc Comments -1 removed lines

@@ -67,7 +67,6 @@
     }
 
     /**
-     * @param bool $multiple
      *
      * @return string
      */

src/Grid/Displayers/AbstractDisplayer.php

Doc Comments +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
     }
 
     /**
-     * @param mixed $key
+     * @param string $key
      *
      * @return mixed
      */

src/Console/ConfigCommand.php

Doc Comments +4 added lines

@@ -57,6 +57,10 @@
             });
     }
 
+    /**
+     * @param string $title
+     * @param \Illuminate\Support\Collection $diff
+     */
     protected function printDiff($title, $diff, $error = false)
     {
         if ($diff->isEmpty()) {

Spacing +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
     {
         return collect(Arr::dot($from))
             ->keys()
-            ->reject(function ($key) use ($to) {
+            ->reject(function($key) use ($to) {
                 return Arr::has($to, $key);
             });
     }
@@ -66,7 +66,7 @@ 
         $this->line('');
         $this->comment("{$title}:");
 
-        $diff->each(function ($key) use ($error) {
+        $diff->each(function($key) use ($error) {
             if ($error) {
                 $this->error("    {$key}");
             } else {