z-song / laravel-admin

src/Form/Builder.php

Doc Comments +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
     /**
      * Returns builder is $mode.
      *
-     * @param $mode
+     * @param string $mode
      *
      * @return bool
      */

Spacing +6 added lines, -6 removed lines

@@ -336,7 +336,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;
         });
     }
@@ -348,7 +348,7 @@ 
      */
     public function hasRows(): bool
     {
-        return ! empty($this->form->rows);
+        return !empty($this->form->rows);
     }
 
     /**
@@ -459,7 +459,7 @@ 
     {
         $previous = URL::previous();
 
-        if (! $previous || $previous === URL::current()) {
+        if (!$previous || $previous === URL::current()) {
             return;
         }
 
@@ -523,7 +523,7 @@ 
      */
     protected function removeReservedFields()
     {
-        if (! $this->isCreating()) {
+        if (!$this->isCreating()) {
             return;
         }
 
@@ -535,7 +535,7 @@ 
 
         $this->form->getLayout()->removeReservedFields($reservedColumns);
 
-        $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, true);
         });
     }
@@ -571,7 +571,7 @@ 
 
         $tabObj = $this->form->setTab();
 
-        if (! $tabObj->isEmpty()) {
+        if (!$tabObj->isEmpty()) {
             $script = <<<'SCRIPT'
 
 var hash = document.location.hash;

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/Auth/Permission.php

Doc Comments +3 added lines, -3 removed lines

@@ -12,7 +12,7 @@ 
      *
      * @param $permission
      *
-     * @return true
+     * @return boolean|null
      */
     public static function check($permission)
     {
@@ -30,7 +30,7 @@ 
      *
      * @param $roles
      *
-     * @return true
+     * @return boolean|null
      */
     public static function allow($roles)
     {
@@ -48,7 +48,7 @@ 
      *
      * @param $roles
      *
-     * @return true
+     * @return boolean|null
      */
     public static function deny($roles)
     {

Spacing +3 added lines, -3 removed lines

@@ -21,7 +21,7 @@ 
         }
 
         if (is_array($permission)) {
-            collect($permission)->each(function ($permission) {
+            collect($permission)->each(function($permission) {
                 call_user_func([self::class, 'check'], $permission);
             });
 
@@ -46,7 +46,7 @@ 
             return true;
         }
 
-        if (! Admin::user()->inRoles($roles)) {
+        if (!Admin::user()->inRoles($roles)) {
             static::error();
         }
     }
@@ -86,7 +86,7 @@ 
     {
         $response = response(Admin::content()->withError(trans('admin.deny')));
 
-        if (! request()->pjax() && request()->ajax()) {
+        if (!request()->pjax() && request()->ajax()) {
             abort(403, trans('admin.deny'));
         }
 

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 +13 added lines, -13 removed lines

@@ -331,9 +331,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']);
         });
 
@@ -384,7 +384,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']);
         });
 
@@ -414,9 +414,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']);
         });
 
@@ -450,11 +450,11 @@ 
     {
         $paginate = $this->findQueryByMethod('paginate');
 
-        $this->queries = $this->queries->reject(function ($query) {
+        $this->queries = $this->queries->reject(function($query) {
             return $query['method'] == 'paginate';
         });
 
-        if (! $this->usePaginate) {
+        if (!$this->usePaginate) {
             $query = [
                 'method'    => 'get',
                 'arguments' => [],
@@ -508,7 +508,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;
         });
     }
@@ -521,7 +521,7 @@ 
     protected function setSort()
     {
         $this->sort = \request($this->sortName, []);
-        if (! is_array($this->sort)) {
+        if (!is_array($this->sort)) {
             return;
         }
 
@@ -531,7 +531,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) {
@@ -549,7 +549,7 @@ 
             }
 
             // get column. if contains "cast", set set column as cast
-            if (! empty($this->sort['cast'])) {
+            if (!empty($this->sort['cast'])) {
                 $column = "CAST({$columnName} AS {$this->sort['cast']}) {$this->sort['type']}";
                 $method = 'orderByRaw';
                 $arguments = [$column];
@@ -577,7 +577,7 @@ 
     {
         [$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();
@@ -611,7 +611,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';
         });
     }

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

Spacing +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
      */
     public function handle()
     {
-        if (! $this->modelExists()) {
+        if (!$this->modelExists()) {
             $this->error('Model does not exists !');
 
             return false;
@@ -47,7 +47,7 @@ 
 
         $stub = $this->option('stub');
 
-        if ($stub and ! is_file($stub)) {
+        if ($stub and !is_file($stub)) {
             $this->error('The stub file dose not exist.');
 
             return false;

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 +12 added lines, -12 removed lines

@@ -32,8 +32,8 @@ 
         $grid->column('slug', trans('admin.slug'));
         $grid->column('name', trans('admin.name'));
 
-        $grid->column('http_path', trans('admin.route'))->display(function ($path) {
-            return collect(explode("\n", $path))->map(function ($path) {
+        $grid->column('http_path', trans('admin.route'))->display(function($path) {
+            return collect(explode("\n", $path))->map(function($path) {
                 $method = $this->http_method ?: ['ANY'];
 
                 if (Str::contains($path, ':')) {
@@ -41,13 +41,13 @@ 
                     $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;');
 
-                if (! empty(config('admin.route.prefix'))) {
+                if (!empty(config('admin.route.prefix'))) {
                     $path = '/'.trim(config('admin.route.prefix'), '/').$path;
                 }
 
@@ -58,8 +58,8 @@ 
         $grid->column('created_at', trans('admin.created_at'));
         $grid->column('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();
             });
         });
@@ -84,8 +84,8 @@ 
         $show->field('slug', trans('admin.slug'));
         $show->field('name', trans('admin.name'));
 
-        $show->field('http_path', trans('admin.route'))->unescape()->as(function ($path) {
-            return collect(explode("\r\n", $path))->map(function ($path) {
+        $show->field('http_path', trans('admin.route'))->unescape()->as(function($path) {
+            return collect(explode("\r\n", $path))->map(function($path) {
                 $method = $this->http_method ?: ['ANY'];
 
                 if (Str::contains($path, ':')) {
@@ -93,13 +93,13 @@ 
                     $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;');
 
-                if (! empty(config('admin.route.prefix'))) {
+                if (!empty(config('admin.route.prefix'))) {
                     $path = '/'.trim(config('admin.route.prefix'), '/').$path;
                 }
 

src/Form/Field/Checkbox.php

Doc Comments +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
      *
      * @param array|callable|string $options
      *
-     * @return $this|mixed
+     * @return Checkbox
      */
     public function options($options = [])
     {

src/Form/Field/ImageField.php

Doc Comments +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      *
      * @param string $target
      *
-     * @return mixed
+     * @return string
      */
     public function callInterventionMethods($target)
     {

Spacing +4 added lines, -4 removed lines

@@ -43,7 +43,7 @@ 
      */
     public function callInterventionMethods($target)
     {
-        if (! empty($this->interventionCalls)) {
+        if (!empty($this->interventionCalls)) {
             $image = ImageManagerStatic::make($target);
 
             foreach ($this->interventionCalls as $call) {
@@ -73,7 +73,7 @@ 
             return $this;
         }
 
-        if (! class_exists(ImageManagerStatic::class)) {
+        if (!class_exists(ImageManagerStatic::class)) {
             throw new \Exception('To use image handling and manipulation, please install [intervention/image] first.');
         }
 
@@ -170,11 +170,11 @@ 
 
             $action = $size[2] ?? 'resize';
             // Resize image with aspect ratio
-            $image->$action($size[0], $size[1], function (Constraint $constraint) {
+            $image->$action($size[0], $size[1], function(Constraint $constraint) {
                 $constraint->aspectRatio();
             })->resizeCanvas($size[0], $size[1], 'center', false, '#ffffff');
 
-            if (! is_null($this->storagePermission)) {
+            if (!is_null($this->storagePermission)) {
                 $this->storage->put("{$this->getDirectory()}/{$path}", $image->encode(), $this->storagePermission);
             } else {
                 $this->storage->put("{$this->getDirectory()}/{$path}", $image->encode());

src/Form/NestedForm.php

Doc Comments +1 added lines, -1 removed lines

@@ -292,7 +292,7 @@
     /**
      * Get the html and script of template.
      *
-     * @return array
+     * @return string[]
      */
     public function getTemplateHtmlAndScript()
     {

Spacing +3 added lines, -3 removed lines

@@ -133,7 +133,7 @@ 
             $key = $this->model->getKey();
         }
 
-        if (! is_null($this->key)) {
+        if (!is_null($this->key)) {
             $key = $this->key;
         }
 
@@ -241,7 +241,7 @@ 
             $values = $this->original[$key];
         }
 
-        $this->fields->each(function (Field $field) use ($values) {
+        $this->fields->each(function(Field $field) use ($values) {
             $field->setOriginal($values);
         });
     }
@@ -308,7 +308,7 @@ 
         if (is_array($columns)) {
             $value = [];
             foreach ($columns as $name => $column) {
-                if (! Arr::has($data, $column)) {
+                if (!Arr::has($data, $column)) {
                     continue;
                 }
                 $value[$name] = Arr::get($data, $column);