z-song / laravel-admin

src/Grid/Filter/Between.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
 
         $this->value = array_get($inputs, $this->column);
 
-        $value = array_filter($this->value, function ($val) {
+        $value = array_filter($this->value, function($val) {
             return $val !== '';
         });
 

src/Grid/Row.php

Spacing +1 added lines, -1 removed lines

@@ -134,7 +134,7 @@
     public function style($style)
     {
         if (is_array($style)) {
-            $style = implode('', array_map(function ($key, $val) {
+            $style = implode('', array_map(function($key, $val) {
                 return "$key:$val";
             }, array_keys($style), array_values($style)));
         }

src/Widgets/Box.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
             return $this->style([$styles]);
         }
 
-        $styles = array_map(function ($style) {
+        $styles = array_map(function($style) {
             return 'box-'.$style;
         }, $styles);
 

src/Widgets/Form.php

Doc Comments +1 added lines, -1 removed lines

@@ -117,7 +117,7 @@
     /**
      * Add form attributes.
      *
-     * @param string|array $attr
+     * @param string $attr
      * @param string       $value
      *
      * @return $this

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 use Illuminate\Contracts\Validation\Validator;
 use Illuminate\Support\Facades\Input;
 use Illuminate\Support\MessageBag;
-use phpDocumentor\Reflection\Types\This;
 
 /**
  * Class Form.

Spacing +3 added lines, -3 removed lines

@@ -172,7 +172,7 @@ 
      */
     public function setWidth($fieldWidth = 8, $labelWidth = 2)
     {
-        collect($this->fields)->each(function ($field) use ($fieldWidth, $labelWidth) {
+        collect($this->fields)->each(function($field) use ($fieldWidth, $labelWidth) {
             /* @var Field $field */
             $field->setWidth($fieldWidth, $labelWidth);
         });
@@ -341,7 +341,7 @@ 
     {
         $rules = [];
         foreach ($this->fields as $item) {
-            if(!empty($item->getRules())){
+            if (!empty($item->getRules())) {
                 $rules[$item->id] = $item->getRules();
             }
         }
@@ -357,7 +357,7 @@ 
     public function getRuleMessages()
     {
         $rules = [];
-        foreach ($this->fields as $item ) {
+        foreach ($this->fields as $item) {
             foreach ($item->validationMessages as $key => $value) {
                 $rules[$key] = $value;
             }

src/Grid/Model.php

Doc Comments +2 added lines, -2 removed lines

@@ -240,7 +240,7 @@ 
     /**
      * Resolve perPage for pagination.
      *
-     * @param array|null $paginate
+     * @param Model $paginate
      *
      * @return array
      */
@@ -262,7 +262,7 @@ 
     /**
      * Find query by method name.
      *
-     * @param $method
+     * @param string $method
      *
      * @return static
      */

Spacing +7 added lines, -7 removed lines

@@ -225,9 +225,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']);
         });
 
@@ -274,7 +274,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']);
         });
 
@@ -318,7 +318,7 @@ 
     {
         $paginate = $this->findQueryByMethod('paginate');
 
-        $this->queries = $this->queries->reject(function ($query) {
+        $this->queries = $this->queries->reject(function($query) {
             return $query['method'] == 'paginate';
         });
 
@@ -372,7 +372,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;
         });
     }
@@ -416,7 +416,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();
@@ -445,7 +445,7 @@ 
      */
     public function resetOrderBy()
     {
-        $this->queries = $this->queries->reject(function ($query) {
+        $this->queries = $this->queries->reject(function($query) {
             return $query['method'] == 'orderBy';
         });
     }

src/Console/MakeCommand.php

Doc Comments +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
     /**
      * Execute the console command.
      *
-     * @return void
+     * @return false|null
      */
     public function handle()
     {

src/Controllers/PermissionController.php

Doc Comments +1 added lines, -1 removed lines

@@ -131,7 +131,7 @@
     /**
      * Get options of HTTP methods select field.
      *
-     * @return array
+     * @return callable
      */
     protected function getHttpMethodsOptions()
     {

Spacing +11 added lines, -11 removed lines

@@ -21,7 +21,7 @@ 
      */
     public function index()
     {
-        return Admin::content(function (Content $content) {
+        return Admin::content(function(Content $content) {
             $content->header(trans('admin.permissions'));
             $content->description(trans('admin.list'));
             $content->body($this->grid()->render());
@@ -37,7 +37,7 @@ 
      */
     public function edit($id)
     {
-        return Admin::content(function (Content $content) use ($id) {
+        return Admin::content(function(Content $content) use ($id) {
             $content->header(trans('admin.permissions'));
             $content->description(trans('admin.edit'));
             $content->body($this->form()->edit($id));
@@ -51,7 +51,7 @@ 
      */
     public function create()
     {
-        return Admin::content(function (Content $content) {
+        return Admin::content(function(Content $content) {
             $content->header(trans('admin.permissions'));
             $content->description(trans('admin.create'));
             $content->body($this->form());
@@ -65,13 +65,13 @@ 
      */
     protected function grid()
     {
-        return Admin::grid(Permission::class, function (Grid $grid) {
+        return Admin::grid(Permission::class, function(Grid $grid) {
             $grid->id('ID')->sortable();
             $grid->slug(trans('admin.slug'));
             $grid->name(trans('admin.name'));
 
-            $grid->http_path(trans('admin.route'))->display(function ($path) {
-                return collect(explode("\r\n", $path))->map(function ($path) {
+            $grid->http_path(trans('admin.route'))->display(function($path) {
+                return collect(explode("\r\n", $path))->map(function($path) {
                     $method = $this->http_method ?: ['ANY'];
 
                     if (Str::contains($path, ':')) {
@@ -79,9 +79,9 @@ 
                         $method = explode(',', $method);
                     }
 
-                    $method = collect($method)->map(function ($name) {
+                    $method = collect($method)->map(function($name) {
                         return strtoupper($name);
-                    })->map(function ($name) {
+                    })->map(function($name) {
                         return "<span class='label label-primary'>{$name}</span>";
                     })->implode('&nbsp;');
 
@@ -94,8 +94,8 @@ 
             $grid->created_at(trans('admin.created_at'));
             $grid->updated_at(trans('admin.updated_at'));
 
-            $grid->tools(function (Grid\Tools $tools) {
-                $tools->batch(function (Grid\Tools\BatchActions $actions) {
+            $grid->tools(function(Grid\Tools $tools) {
+                $tools->batch(function(Grid\Tools\BatchActions $actions) {
                     $actions->disableDelete();
                 });
             });
@@ -109,7 +109,7 @@ 
      */
     public function form()
     {
-        return Admin::form(Permission::class, function (Form $form) {
+        return Admin::form(Permission::class, function(Form $form) {
             $form->display('id', 'ID');
 
             $form->text('slug', trans('admin.slug'))->rules('required');

src/Form.php

Doc Comments +4 added lines, -4 removed lines

@@ -278,7 +278,7 @@ 
      *
      * @param $id
      *
-     * @return mixed
+     * @return boolean
      */
     public function destroy($id)
     {
@@ -317,7 +317,7 @@ 
     /**
      * Store a new record.
      *
-     * @return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector|\Illuminate\Http\JsonResponse
+     * @return Response
      */
     public function store()
     {
@@ -396,7 +396,7 @@ 
      *
      * @param array $data
      *
-     * @return mixed
+     * @return Response|null
      */
     protected function prepare($data = [])
     {
@@ -854,7 +854,7 @@ 
     /**
      * Set saved callback.
      *
-     * @param callable $callback
+     * @param Closure $callback
      *
      * @return void
      */

Spacing +10 added lines, -10 removed lines

@@ -328,9 +328,9 @@ 
         $data = $this->model->with($this->getRelations())
             ->findOrFail($id)->toArray();
 
-        $this->builder->fields()->filter(function ($field) {
+        $this->builder->fields()->filter(function($field) {
             return $field instanceof Field\File;
-        })->each(function (File $file) use ($data) {
+        })->each(function(File $file) use ($data) {
             $file->setOriginal($data);
 
             $file->destroy();
@@ -355,7 +355,7 @@ 
             return $response;
         }
 
-        DB::transaction(function () {
+        DB::transaction(function() {
             $inserts = $this->prepareInsert($this->updates);
 
             foreach ($inserts as $column => $value) {
@@ -559,7 +559,7 @@ 
             return $response;
         }
 
-        DB::transaction(function () {
+        DB::transaction(function() {
             $updates = $this->prepareUpdate($this->updates);
 
             foreach ($updates as $column => $value) {
@@ -807,7 +807,7 @@ 
      */
     protected function invalidColumn($columns, $oneToOneRelation = false)
     {
-        foreach ((array)$columns as $column) {
+        foreach ((array) $columns as $column) {
             if ((!$oneToOneRelation && Str::contains($column, '.')) ||
                 ($oneToOneRelation && !Str::contains($column, '.'))
             ) {
@@ -916,7 +916,7 @@ 
      */
     public function ignore($fields)
     {
-        $this->ignored = array_merge($this->ignored, (array)$fields);
+        $this->ignored = array_merge($this->ignored, (array) $fields);
 
         return $this;
     }
@@ -956,7 +956,7 @@ 
     protected function getFieldByColumn($column)
     {
         return $this->builder->fields()->first(
-            function (Field $field) use ($column) {
+            function(Field $field) use ($column) {
                 if (is_array($field->column())) {
                     return in_array($column, $field->column());
                 }
@@ -977,7 +977,7 @@ 
 
         $values = $this->model->toArray();
 
-        $this->builder->fields()->each(function (Field $field) use ($values) {
+        $this->builder->fields()->each(function(Field $field) use ($values) {
             $field->setOriginal($values);
         });
     }
@@ -999,7 +999,7 @@ 
 
         $data = $this->model->toArray();
 
-        $this->builder->fields()->each(function (Field $field) use ($data) {
+        $this->builder->fields()->each(function(Field $field) use ($data) {
             $field->fill($data);
         });
     }
@@ -1118,7 +1118,7 @@ 
      */
     public function setWidth($fieldWidth = 8, $labelWidth = 2)
     {
-        $this->builder()->fields()->each(function ($field) use ($fieldWidth, $labelWidth) {
+        $this->builder()->fields()->each(function($field) use ($fieldWidth, $labelWidth) {
             /* @var Field $field */
             $field->setWidth($fieldWidth, $labelWidth);
         });

Braces +3 added lines, -1 removed lines

@@ -228,7 +228,9 @@
     public function edit($id, $show_mode = false)
     {
         $mode = Builder::MODE_EDIT;
-        if ($show_mode) $mode = Builder::MODE_SHOW;
+        if ($show_mode) {
+            $mode = Builder::MODE_SHOW;
+        }
         $this->builder->setMode($mode);
         $this->builder->setResourceId($id);
 

src/Form/Builder.php

Doc Comments +2 added lines, -1 removed lines

@@ -132,7 +132,7 @@ 
     /**
      * Returns builder is $mode.
      *
-     * @param $mode
+     * @param string $mode
      *
      * @return bool
      */
@@ -154,6 +154,7 @@ 
     }
 
     /**
+     * @param integer $slice
      * @return string
      */
     public function getResource($slice = null)

Spacing +4 added lines, -4 removed lines

@@ -259,7 +259,7 @@ 
      */
     public function field($name)
     {
-        return $this->fields()->first(function (Field $field) use ($name) {
+        return $this->fields()->first(function(Field $field) use ($name) {
             return $field->column() == $name;
         });
     }
@@ -502,7 +502,7 @@ 
             $this->form->model()->getUpdatedAtColumn(),
         ];
 
-        $this->fields = $this->fields()->reject(function (Field $field) use ($reservedColumns) {
+        $this->fields = $this->fields()->reject(function(Field $field) use ($reservedColumns) {
             return in_array($field->column(), $reservedColumns);
         });
 
@@ -528,7 +528,7 @@ 
     {
         $rules = [];
         foreach ($this->fields() as $item) {
-            if(!empty($item->getRules())){
+            if (!empty($item->getRules())) {
                 $rules[$item->id] = $item->getRules();
             }
         }
@@ -544,7 +544,7 @@ 
     public function getRuleMessages()
     {
         $rules = [];
-        foreach ($this->fields() as $item ) {
+        foreach ($this->fields() as $item) {
             foreach ($item->validationMessages as $key => $value) {
                 $rules[$key] = $value;
             }

Braces +3 added lines, -1 removed lines

@@ -366,7 +366,9 @@
      */
     public function title()
     {
-        if ($this->Title != "") return $this->Title;
+        if ($this->Title != "") {
+            return $this->Title;
+        }
 
         if ($this->mode == static::MODE_CREATE) {
             return trans('admin.create');