z-song / laravel-admin

src/Grid/Tools/Paginator.php

Doc Comments +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
     /**
      * Get Pagination links.
      *
-     * @return string
+     * @return \Illuminate\Contracts\View\View
      */
     protected function paginationLinks()
     {

Spacing +2 added lines, -2 removed lines

@@ -71,7 +71,7 @@ 
             'total' => $this->paginator->total(),
         ];
 
-        $parameters = collect($parameters)->flatMap(function ($parameter, $key) {
+        $parameters = collect($parameters)->flatMap(function($parameter, $key) {
             return [$key => "<b>$parameter</b>"];
         });
 
@@ -85,7 +85,7 @@ 
      */
     public function render()
     {
-        if (! $this->grid->showPagination()) {
+        if (!$this->grid->showPagination()) {
             return '';
         }
 

src/Show/Panel.php

Doc Comments +2 added lines, -2 removed lines

@@ -128,7 +128,7 @@ 
     /**
      * Build panel tools.
      *
-     * @param $callable
+     * @param \Closure $callable
      */
     public function tools($callable)
     {
@@ -138,7 +138,7 @@ 
     /**
      * Fill fields to panel.
      *
-     * @param []Field $fields
+     * @param Collection $fields
      *
      * @return $this
      */

src/Controllers/UserController.php

Spacing +8 added lines, -8 removed lines

@@ -34,14 +34,14 @@ 
         $grid->column('created_at', trans('admin.created_at'));
         $grid->column('updated_at', trans('admin.updated_at'));
 
-        $grid->actions(function (Grid\Displayers\Actions $actions) {
+        $grid->actions(function(Grid\Displayers\Actions $actions) {
             if ($actions->getKey() == 1) {
                 $actions->disableDelete();
             }
         });
 
-        $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();
             });
         });
@@ -65,15 +65,15 @@ 
         $show->field('id', 'ID');
         $show->field('username', trans('admin.username'));
         $show->field('name', trans('admin.name'));
-        $show->field('roles', trans('admin.roles'))->as(function ($roles) {
+        $show->field('roles', trans('admin.roles'))->as(function($roles) {
             return $roles->pluck('name');
         })->label();
-        $show->field('permissions', trans('admin.permissions'))->as(function ($permission) {
+        $show->field('permissions', trans('admin.permissions'))->as(function($permission) {
             return $permission->pluck('name');
         })->label();
         $show->field('created_at', trans('admin.created_at'));
         $show->field('updated_at', trans('admin.updated_at'));
-        $show->panel()->tools(function ($tools) use ($id) {
+        $show->panel()->tools(function($tools) use ($id) {
             if ($id == 1) {
                 $tools->disableDelete();
             }
@@ -107,7 +107,7 @@ 
         $form->image('avatar', trans('admin.avatar'));
         $form->password('password', trans('admin.password'))->rules('required|confirmed');
         $form->password('password_confirmation', trans('admin.password_confirmation'))->rules('required')
-            ->default(function ($form) {
+            ->default(function($form) {
                 return $form->model()->password;
             });
 
@@ -119,7 +119,7 @@ 
         $form->display('created_at', trans('admin.created_at'));
         $form->display('updated_at', trans('admin.updated_at'));
 
-        $form->saving(function (Form $form) {
+        $form->saving(function(Form $form) {
             if ($form->password && $form->model()->password != $form->password) {
                 $form->password = bcrypt($form->password);
             }

src/Grid/Exporters/CsvExporter.php

Spacing +3 added lines, -3 removed lines

@@ -22,12 +22,12 @@ 
             'Content-Disposition' => "attachment; filename=\"$filename\"",
         ];
 
-        response()->stream(function () {
+        response()->stream(function() {
             $handle = fopen('php://output', 'w');
 
             $titles = [];
 
-            $this->chunk(function ($records) use ($handle, &$titles) {
+            $this->chunk(function($records) use ($handle, &$titles) {
                 if (empty($titles)) {
                     $titles = $this->getHeaderRowFromRecords($records);
 
@@ -55,7 +55,7 @@ 
     public function getHeaderRowFromRecords(Collection $records): array
     {
         $titles = collect(Arr::dot($records->first()->getAttributes()))->keys()->map(
-            function ($key) {
+            function($key) {
                 $key = str_replace('.', ' ', $key);
 
                 return Str::ucfirst($key);

src/Form/Field.php

Doc Comments +6 added lines, -6 removed lines

@@ -281,7 +281,7 @@ 
      *
      * @param string $column
      *
-     * @return mixed|string
+     * @return string
      */
     protected function formatColumn($column = '')
     {
@@ -574,7 +574,7 @@ 
     }
 
     /**
-     * @param string|array|Closure $input
+     * @param callable|null $input
      * @param string|array         $original
      *
      * @return array|Closure
@@ -785,7 +785,7 @@ 
     /**
      * Set or get value of the field.
      *
-     * @param $value
+     * @param string $value
      *
      * @return mixed
      */
@@ -956,7 +956,7 @@ 
     /**
      * Add html attributes to elements.
      *
-     * @param array|string $attribute
+     * @param string $attribute
      * @param mixed        $value
      *
      * @return $this
@@ -1334,7 +1334,7 @@ 
     }
 
     /**
-     * @param array $labelClass
+     * @param string[] $labelClass
      *
      * @return self
      */
@@ -1389,7 +1389,7 @@ 
      *
      * @param string $view
      *
-     * @return string
+     * @return Field
      */
     public function setView($view): self
     {

Spacing +14 added lines, -14 removed lines

@@ -520,18 +520,18 @@ 
      */
     protected function addRequiredAttribute($rules)
     {
-        if (! is_array($rules)) {
+        if (!is_array($rules)) {
             return;
         }
 
-        if (! in_array('required', $rules, true)) {
+        if (!in_array('required', $rules, true)) {
             return;
         }
 
         $this->setLabelClass(['asterisk']);
 
         // Only text field has `required` attribute.
-        if (! $this instanceof Form\Field\Text) {
+        if (!$this instanceof Form\Field\Text) {
             return;
         }
 
@@ -586,7 +586,7 @@ 
         if ($input instanceof Closure) {
             $rules = $input;
         } else {
-            if (! empty($original)) {
+            if (!empty($original)) {
                 $original = $this->formatRules($original);
             }
 
@@ -704,11 +704,11 @@ 
             $rules = array_filter(explode('|', $rules));
         }
 
-        if (! $this->form) {
+        if (!$this->form) {
             return $rules;
         }
 
-        if (! $id = $this->form->model()->getKey()) {
+        if (!$id = $this->form->model()->getKey()) {
             return $rules;
         }
 
@@ -738,7 +738,7 @@ 
             return;
         }
 
-        if (! is_string($this->rules)) {
+        if (!is_string($this->rules)) {
             return;
         }
 
@@ -908,12 +908,12 @@ 
 
         $rules = $attributes = [];
 
-        if (! $fieldRules = $this->getRules()) {
+        if (!$fieldRules = $this->getRules()) {
             return false;
         }
 
         if (is_string($this->column)) {
-            if (! Arr::has($input, $this->column)) {
+            if (!Arr::has($input, $this->column)) {
                 return false;
             }
 
@@ -925,7 +925,7 @@ 
 
         if (is_array($this->column)) {
             foreach ($this->column as $key => $column) {
-                if (! array_key_exists($column, $input)) {
+                if (!array_key_exists($column, $input)) {
                     continue;
                 }
                 $input[$column.$key] = Arr::get($input, $column);
@@ -1170,7 +1170,7 @@ 
      */
     public function getElementClass(): array
     {
-        if (! $this->elementClass) {
+        if (!$this->elementClass) {
             $name = $this->elementName ?: $this->formatName($this->column);
 
             $this->elementClass = (array) str_replace(['[', ']'], '_', $name);
@@ -1373,7 +1373,7 @@ 
      */
     public function getView(): string
     {
-        if (! empty($this->view)) {
+        if (!empty($this->view)) {
             return $this->view;
         }
 
@@ -1441,7 +1441,7 @@ 
      */
     protected function shouldRender(): bool
     {
-        if (! $this->display) {
+        if (!$this->display) {
             return false;
         }
 
@@ -1467,7 +1467,7 @@ 
      */
     public function render()
     {
-        if (! $this->shouldRender()) {
+        if (!$this->shouldRender()) {
             return '';
         }
 

src/Auth/Database/HasPermissions.php

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
     {

Spacing +2 added lines, -2 removed lines

@@ -50,7 +50,7 @@ 
      */
     public function cannot(string $permission): bool
     {
-        return ! $this->can($permission);
+        return !$this->can($permission);
     }
 
     /**
@@ -112,7 +112,7 @@ 
      */
     protected static function bootHasPermissions()
     {
-        static::deleting(function ($model) {
+        static::deleting(function($model) {
             $model->roles()->detach();
 
             $model->permissions()->detach();

src/Grid/Filter/Presenter/Select.php

Doc Comments +1 added lines, -1 removed lines

@@ -303,7 +303,7 @@
      *
      * @param string $target
      *
-     * @return mixed
+     * @return string
      */
     protected function getClass($target): string
     {

Spacing +3 added lines, -3 removed lines

@@ -112,13 +112,13 @@
      */
     public function model($model, $idField = 'id', $textField = 'name')
     {
-        if (! class_exists($model)
-            || ! in_array(Model::class, class_parents($model))
+        if (!class_exists($model)
+            || !in_array(Model::class, class_parents($model))
         ) {
             throw new \InvalidArgumentException("[$model] must be a valid model class");
         }
 
-        $this->options = function ($value) use ($model, $idField, $textField) {
+        $this->options = function($value) use ($model, $idField, $textField) {
             if (empty($value)) {
                 return [];
             }

src/Widgets/Form.php

Doc Comments +2 added lines, -2 removed lines

@@ -119,7 +119,7 @@ 
     /**
      * Get form title.
      *
-     * @return mixed
+     * @return string
      */
     public function title()
     {
@@ -183,7 +183,7 @@ 
     /**
      * Add form attributes.
      *
-     * @param string|array $attr
+     * @param string $attr
      * @param string       $value
      *
      * @return $this

Spacing +7 added lines, -7 removed lines

@@ -147,7 +147,7 @@ 
             $data = $data->toArray();
         }
 
-        if (! empty($data)) {
+        if (!empty($data)) {
             $this->data = $data;
         }
 
@@ -300,7 +300,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);
         });
@@ -401,11 +401,11 @@ 
 
         /** @var Field $field */
         foreach ($this->fields() as $field) {
-            if (! $validator = $field->getValidator($request->all())) {
+            if (!$validator = $field->getValidator($request->all())) {
                 continue;
             }
 
-            if (($validator instanceof Validator) && ! $validator->passes()) {
+            if (($validator instanceof Validator) && !$validator->passes()) {
                 $failedValidators[] = $validator;
             }
         }
@@ -493,7 +493,7 @@ 
 
         $form = view('admin::widgets.form', $this->getVariables())->render();
 
-        if (! ($title = $this->title()) || ! $this->inbox) {
+        if (!($title = $this->title()) || !$this->inbox) {
             return $form;
         }
 
@@ -510,7 +510,7 @@ 
      */
     public function __call($method, $arguments)
     {
-        if (! $this->hasField($method)) {
+        if (!$this->hasField($method)) {
             return $this;
         }
 
@@ -518,7 +518,7 @@ 
 
         $field = new $class(Arr::get($arguments, 0), array_slice($arguments, 1));
 
-        return tap($field, function ($field) {
+        return tap($field, function($field) {
             $this->pushField($field);
         });
     }

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

Spacing +2 added lines, -2 removed lines

@@ -32,12 +32,12 @@
     {
         $active = $active ?: request($this->activeName);
 
-        if (! isset($forms[$active])) {
+        if (!isset($forms[$active])) {
             $active = key($forms);
         }
 
         foreach ($forms as $name => $class) {
-            if (! is_subclass_of($class, Form::class)) {
+            if (!is_subclass_of($class, Form::class)) {
                 admin_error("Class [{$class}] must be a sub-class of [Encore\Admin\Widgets\Form].");
                 continue;
             }