Laravel-Backpack / CRUD

src/app/Library/CrudPanel/Traits/Settings.php

Doc Comments +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
      * Defaults to the current operation.
      *
      * @param  string   $key   Has no operation prepended. (ex: exportButtons)
-     * @return mixed      [description]
+     * @return boolean      [description]
      */
     public function hasOperationSetting(string $key, $operation = null)
     {

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
      */
     public function settings()
     {
-        return \Arr::sort($this->settings, function ($value, $key) {
+        return \Arr::sort($this->settings, function($value, $key) {
             return $key;
         });
     }

src/app/Library/CrudPanel/Traits/Tabs.php

Doc Comments +3 added lines

@@ -35,6 +35,9 @@
         return ! $this->tabsEnabled();
     }
 
+    /**
+     * @param string $type
+     */
     public function setTabsType($type)
     {
         $this->enableTabs();

Spacing +7 added lines, -7 removed lines

@@ -32,7 +32,7 @@ 
      */
     public function tabsDisabled()
     {
-        return ! $this->tabsEnabled();
+        return !$this->tabsEnabled();
     }
 
     public function setTabsType($type)
@@ -114,8 +114,8 @@ 
     {
         $all_fields = $this->getCurrentFields();
 
-        $fields_without_a_tab = collect($all_fields)->filter(function ($value, $key) {
-            return ! isset($value['tab']);
+        $fields_without_a_tab = collect($all_fields)->filter(function($value, $key) {
+            return !isset($value['tab']);
         });
 
         return $fields_without_a_tab;
@@ -131,7 +131,7 @@ 
         if ($this->tabExists($label)) {
             $all_fields = $this->getCurrentFields();
 
-            $fields_for_current_tab = collect($all_fields)->filter(function ($value, $key) use ($label) {
+            $fields_for_current_tab = collect($all_fields)->filter(function($value, $key) use ($label) {
                 return isset($value['tab']) && $value['tab'] == $label;
             });
 
@@ -150,11 +150,11 @@ 
         $fields = $this->getCurrentFields();
 
         $fields_with_tabs = collect($fields)
-            ->filter(function ($value, $key) {
+            ->filter(function($value, $key) {
                 return isset($value['tab']);
             })
-            ->each(function ($value, $key) use (&$tabs) {
-                if (! in_array($value['tab'], $tabs)) {
+            ->each(function($value, $key) use (&$tabs) {
+                if (!in_array($value['tab'], $tabs)) {
                     $tabs[] = $value['tab'];
                 }
             });

src/app/Library/CrudPanel/Traits/Validation.php

Doc Comments -1 removed lines

@@ -88,7 +88,6 @@
      * for the given operation.
      *
      * @param string $inputKey Field or input name.
-     * @param string $operation create / update
      *
      * @return bool
      */

Spacing +1 added lines, -1 removed lines

@@ -94,7 +94,7 @@
      */
     public function isRequired($inputKey)
     {
-        if (! $this->hasOperationSetting('requiredFields')) {
+        if (!$this->hasOperationSetting('requiredFields')) {
             return false;
         }
 

src/app/Models/Traits/CrudTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
      *
      * @param array $columns - the database columns that contain the JSONs
      *
-     * @return Model
+     * @return CrudTrait
      */
     public function withFakes($columns = [])
     {

Unused Use Statements +2 added lines, -2 removed lines

@@ -3,9 +3,9 @@
 namespace Backpack\CRUD\app\Models\Traits;
 
 use DB;
-use Traversable;
-use Illuminate\Support\Facades\Config;
 use Illuminate\Database\Eloquent\Model;
+use Illuminate\Support\Facades\Config;
+use Traversable;
 
 trait CrudTrait
 {

Spacing +6 added lines, -6 removed lines

@@ -69,7 +69,7 @@ 
         $conn->getDoctrineSchemaManager()->getDatabasePlatform()->registerDoctrineTypeMapping('json', 'json_array');
         $conn->getDoctrineSchemaManager()->getDatabasePlatform()->registerDoctrineTypeMapping('jsonb', 'json_array');
 
-        return ! $conn->getDoctrineColumn($table, $column_name)->getNotnull();
+        return !$conn->getDoctrineColumn($table, $column_name)->getNotnull();
     }
 
     /*
@@ -86,7 +86,7 @@ 
     public function addFakes($columns = ['extras'])
     {
         foreach ($columns as $key => $column) {
-            if (! isset($this->attributes[$column])) {
+            if (!isset($this->attributes[$column])) {
                 continue;
             }
 
@@ -134,7 +134,7 @@ 
      */
     public function shouldDecodeFake($column)
     {
-        return ! in_array($column, array_keys($this->casts));
+        return !in_array($column, array_keys($this->casts));
     }
 
     /**
@@ -145,7 +145,7 @@ 
      */
     public function shouldEncodeFake($column)
     {
-        return ! in_array($column, array_keys($this->casts));
+        return !in_array($column, array_keys($this->casts));
     }
 
     /*
@@ -219,7 +219,7 @@ 
     public function uploadMultipleFilesToDisk($value, $attribute_name, $disk, $destination_path)
     {
         $request = \Request::instance();
-        if (! is_array($this->{$attribute_name})) {
+        if (!is_array($this->{$attribute_name})) {
             $attribute_value = json_decode($this->{$attribute_name}, true) ?? [];
         } else {
             $attribute_value = $this->{$attribute_name};
@@ -231,7 +231,7 @@ 
         if ($files_to_clear) {
             foreach ($files_to_clear as $key => $filename) {
                 \Storage::disk($disk)->delete($filename);
-                $attribute_value = array_where($attribute_value, function ($value, $key) use ($filename) {
+                $attribute_value = array_where($attribute_value, function($value, $key) use ($filename) {
                     return $value != $filename;
                 });
             }

src/app/Models/Traits/InheritsRelationsFromParentModel.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,8 +2,8 @@
 
 namespace Backpack\CRUD\app\Models\Traits;
 
-use ReflectionClass;
 use Illuminate\Support\Str;
+use ReflectionClass;
 
 /**
  * This trait helps a child model (ex: BackpackUser) inherit all relationships of its parent model (ex: User).

Spacing +3 added lines, -3 removed lines

@@ -18,14 +18,14 @@ 
 
     public static function bootHasParent()
     {
-        static::creating(function ($model) {
+        static::creating(function($model) {
             if ($model->parentHasHasChildrenTrait()) {
                 $model->forceFill(
                     [$model->getInheritanceColumn() => $model->classToAlias(get_class($model))]
                 );
             }
         });
-        static::addGlobalScope(function ($query) {
+        static::addGlobalScope(function($query) {
             $instance = new static();
             if ($instance->parentHasHasChildrenTrait()) {
                 $query->where($instance->getTable().'.'.$instance->getInheritanceColumn(), $instance->classToAlias(get_class($instance)));
@@ -40,7 +40,7 @@ 
 
     public function getTable()
     {
-        if (! isset($this->table)) {
+        if (!isset($this->table)) {
             return str_replace('\\', '', Str::snake(Str::plural(class_basename($this->getParentClass()))));
         }
 

src/app/Models/Traits/SpatieTranslatable/Sluggable.php

Doc Comments -1 removed lines

@@ -37,7 +37,6 @@
      * Query scope for finding "similar" slugs, used to determine uniqueness.
      *
      * @param \Illuminate\Database\Eloquent\Builder $query
-     * @param \Illuminate\Database\Eloquent\Model $model
      * @param string $attribute
      * @param array $config
      * @param string $slug

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
         $separator = $config['separator'];
         $attribute = $attribute.'->'.$this->getLocale();
 
-        return $query->where(function (Builder $q) use ($attribute, $slug, $separator) {
+        return $query->where(function(Builder $q) use ($attribute, $slug, $separator) {
             $q->where($attribute, '=', $slug)
                 ->orWhere($attribute, 'LIKE', $slug.$separator.'%')
                 // Fixes issues with Json data types in MySQL where data is sourrounded by "

Unused Use Statements +2 added lines, -2 removed lines

@@ -2,9 +2,9 @@
 
 namespace Backpack\CRUD\app\Models\Traits\SpatieTranslatable;
 
-use Illuminate\Database\Eloquent\Model;
-use Illuminate\Database\Eloquent\Builder;
 use Cviebrock\EloquentSluggable\Sluggable as OriginalSluggable;
+use Illuminate\Database\Eloquent\Builder;
+use Illuminate\Database\Eloquent\Model;
 
 trait Sluggable
 {

src/helpers.php

Doc Comments +2 added lines, -1 removed lines

@@ -5,7 +5,7 @@ 
      * Appends the configured backpack prefix and returns
      * the URL using the standard Laravel helpers.
      *
-     * @param $path
+     * @param string $path
      *
      * @return string
      */
@@ -156,6 +156,7 @@ 
      * If that view doesn't exist, it will load the one from the original theme.
      *
      * @param string (see config/backpack/base.php)
+     * @param string $view
      *
      * @return string
      */

Spacing +14 added lines, -14 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if (! function_exists('backpack_url')) {
+if (!function_exists('backpack_url')) {
     /**
      * Appends the configured backpack prefix and returns
      * the URL using the standard Laravel helpers.
@@ -11,13 +11,13 @@ 
      */
     function backpack_url($path = null, $parameters = [], $secure = null)
     {
-        $path = ! $path || (substr($path, 0, 1) == '/') ? $path : '/'.$path;
+        $path = !$path || (substr($path, 0, 1) == '/') ? $path : '/'.$path;
 
         return url(config('backpack.base.route_prefix', 'admin').$path, $parameters = [], $secure = null);
     }
 }
 
-if (! function_exists('backpack_authentication_column')) {
+if (!function_exists('backpack_authentication_column')) {
     /**
      * Return the username column name.
      * The Laravel default (and Backpack default) is 'email'.
@@ -30,7 +30,7 @@ 
     }
 }
 
-if (! function_exists('backpack_users_have_email')) {
+if (!function_exists('backpack_users_have_email')) {
     /**
      * Check if the email column is present on the user table.
      *
@@ -45,7 +45,7 @@ 
     }
 }
 
-if (! function_exists('backpack_avatar_url')) {
+if (!function_exists('backpack_avatar_url')) {
     /**
      * Returns the avatar URL of a user.
      *
@@ -78,7 +78,7 @@ 
     }
 }
 
-if (! function_exists('backpack_middleware')) {
+if (!function_exists('backpack_middleware')) {
     /**
      * Return the key of the middleware used across Backpack.
      * That middleware checks if the visitor is an admin.
@@ -93,7 +93,7 @@ 
     }
 }
 
-if (! function_exists('backpack_guard_name')) {
+if (!function_exists('backpack_guard_name')) {
     /*
      * Returns the name of the guard defined
      * by the application config
@@ -104,7 +104,7 @@ 
     }
 }
 
-if (! function_exists('backpack_auth')) {
+if (!function_exists('backpack_auth')) {
     /*
      * Returns the user instance if it exists
      * of the currently authenticated admin
@@ -116,7 +116,7 @@ 
     }
 }
 
-if (! function_exists('backpack_user')) {
+if (!function_exists('backpack_user')) {
     /*
      * Returns back a user instance without
      * the admin guard, however allows you
@@ -128,7 +128,7 @@ 
     }
 }
 
-if (! function_exists('mb_ucfirst')) {
+if (!function_exists('mb_ucfirst')) {
     /**
      * Capitalize the first letter of a string,
      * even if that string is multi-byte (non-latin alphabet).
@@ -150,7 +150,7 @@ 
     }
 }
 
-if (! function_exists('backpack_view')) {
+if (!function_exists('backpack_view')) {
     /**
      * Returns a new displayable view based on the configured backpack view namespace.
      * If that view doesn't exist, it will load the one from the original theme.
@@ -169,7 +169,7 @@ 
 
         $returnView = $theme.$view;
 
-        if (! view()->exists($returnView)) {
+        if (!view()->exists($returnView)) {
             $returnView = $originalTheme.$view;
         }
 
@@ -177,7 +177,7 @@ 
     }
 }
 
-if (! function_exists('square_brackets_to_dots')) {
+if (!function_exists('square_brackets_to_dots')) {
     /**
      * Turns a string from bracket-type array to dot-notation array.
      * Ex: array[0][property] turns into array.0.property.
@@ -194,7 +194,7 @@ 
     }
 }
 
-if (! function_exists('is_countable')) {
+if (!function_exists('is_countable')) {
     /**
      * We need this because is_countable was only introduced in PHP 7.3,
      * and in PHP 7.2 you should check if count() argument is really countable.

src/routes/backpack/base.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
     'middleware' => 'web',
     'prefix'     => config('backpack.base.route_prefix'),
 ],
-function () {
+function() {
     // if not otherwise configured, setup the auth routes
     if (config('backpack.base.setup_auth_routes')) {
         // Authentication Routes...

src/routes/backpack/custom.php

Spacing +1 added lines, -1 removed lines

@@ -10,5 +10,5 @@
     'prefix'     => config('backpack.base.route_prefix', 'admin'),
     'middleware' => ['web', config('backpack.base.middleware_key', 'admin')],
     'namespace'  => 'App\Http\Controllers\Admin',
-], function () { // custom admin routes
+], function() { // custom admin routes
 }); // this should be the absolute last line of this file