Laravel-Backpack / CRUD

src/app/Console/Commands/PublishBackpackUserModel.php

Spacing +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
     {
         $stub = str_replace('Backpack\CRUD\app\Models;', $this->laravel->getNamespace().'Models;', $stub);
 
-        if (! $this->files->exists($this->laravel['path'].'/User.php') && $this->files->exists($this->laravel['path'].'/Models/User.php')) {
+        if (!$this->files->exists($this->laravel['path'].'/User.php') && $this->files->exists($this->laravel['path'].'/Models/User.php')) {
             $stub = str_replace($this->laravel->getNamespace().'User', $this->laravel->getNamespace().'Models\User', $stub);
         }
 

Doc Comments +2 added lines, -3 removed lines

@@ -33,7 +33,7 @@ 
     /**
      * Execute the console command.
      *
-     * @return bool|null
+     * @return false|null
      */
     public function handle()
     {
@@ -71,9 +71,8 @@ 
      * Replace the User model, if it was moved to App\Models\User.
      *
      * @param string $stub
-     * @param string $name
      *
-     * @return $this
+     * @return string
      */
     protected function makeReplacements(&$stub)
     {

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

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
      */
     public function getAttributeValue($key)
     {
-        if (! $this->isTranslatableAttribute($key)) {
+        if (!$this->isTranslatableAttribute($key)) {
             return parent::getAttributeValue($key);
         }
 
@@ -72,7 +72,7 @@ 
      */
     public function update(array $attributes = [], array $options = [])
     {
-        if (! $this->exists) {
+        if (!$this->exists) {
             return false;
         }
 

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

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
 {

Doc Comments -1 removed lines

@@ -38,7 +38,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

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

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
      */
     protected function makeSlugUnique(string $slug, array $config, string $attribute): string
     {
-        if (! $config['unique']) {
+        if (!$config['unique']) {
             return $slug;
         }
 

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

Spacing +5 added lines, -5 removed lines

@@ -92,7 +92,7 @@ 
          */
         $button = $this->buttons()->firstWhere('name', $name);
 
-        if (! $button) {
+        if (!$button) {
             abort(500, 'CRUD Button "'.$name.'" not found. Please check the button exists before you modify it.');
         }
 
@@ -113,7 +113,7 @@ 
      */
     public function removeButton($name, $stack = null)
     {
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($name, $stack) {
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($name, $stack) {
             return $stack == null ? $button->name == $name : ($button->stack == $stack) && ($button->name == $name);
         }));
     }
@@ -124,7 +124,7 @@ 
      */
     public function removeButtons($names, $stack = null)
     {
-        if (! empty($names)) {
+        if (!empty($names)) {
             foreach ($names as $name) {
                 $this->removeButton($name, $stack);
             }
@@ -138,14 +138,14 @@ 
 
     public function removeAllButtonsFromStack($stack)
     {
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($stack) {
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($stack) {
             return $button->stack == $stack;
         }));
     }
 
     public function removeButtonFromStack($name, $stack)
     {
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($name, $stack) {
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($name, $stack) {
             return $button->name == $name && $button->stack == $stack;
         }));
     }

Doc Comments +13 added lines, -1 removed lines

@@ -62,6 +62,9 @@ 
         $this->addButton($stack, $name, 'model_function', $model_function_name, $position);
     }
 
+    /**
+     * @param string $view
+     */
     public function addButtonFromView($stack, $name, $view, $position = false)
     {
         $view = 'crud::buttons.'.$view;
@@ -119,7 +122,7 @@ 
     }
 
     /**
-     * @param array       $names Button names
+     * @param string[]       $names Button names
      * @param string|null $stack Optional stack name.
      */
     public function removeButtons($names, $stack = null)
@@ -136,6 +139,9 @@ 
         $this->setOperationSetting('buttons', collect());
     }
 
+    /**
+     * @param string $stack
+     */
     public function removeAllButtonsFromStack($stack)
     {
         $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($stack) {
@@ -158,6 +164,12 @@ 
     public $type = 'view';
     public $content;
 
+    /**
+     * @param string $stack
+     * @param string $name
+     * @param string $type
+     * @param string $content
+     */
     public function __construct($stack, $name, $type, $content)
     {
         $this->stack = $stack;

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

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
             $revisionDate = date('Y-m-d', strtotime((string) $history->created_at));
 
             // Be sure to instantiate the initial grouping array
-            if (! array_key_exists($revisionDate, $revisions)) {
+            if (!array_key_exists($revisionDate, $revisions)) {
                 $revisions[$revisionDate] = [];
             }
 

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

Spacing +6 added lines, -6 removed lines

@@ -21,7 +21,7 @@ 
      */
     public function getTitle($action = false)
     {
-        if (! $action) {
+        if (!$action) {
             $action = $this->getActionMethod();
         }
 
@@ -38,7 +38,7 @@ 
      */
     public function setTitle($string, $action = false)
     {
-        if (! $action) {
+        if (!$action) {
             $action = $this->getActionMethod();
         }
 
@@ -58,7 +58,7 @@ 
      */
     public function getHeading($action = false)
     {
-        if (! $action) {
+        if (!$action) {
             $action = $this->getActionMethod();
         }
 
@@ -75,7 +75,7 @@ 
      */
     public function setHeading($string, $action = false)
     {
-        if (! $action) {
+        if (!$action) {
             $action = $this->getActionMethod();
         }
 
@@ -95,7 +95,7 @@ 
      */
     public function getSubheading($action = false)
     {
-        if (! $action) {
+        if (!$action) {
             $action = $this->getActionMethod();
         }
 
@@ -112,7 +112,7 @@ 
      */
     public function setSubheading($string, $action = false)
     {
-        if (! $action) {
+        if (!$action) {
             $action = $this->getActionMethod();
         }
 

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

Spacing +4 added lines, -4 removed lines

@@ -38,7 +38,7 @@ 
     {
         $id = $this->getCurrentEntryId();
 
-        if (! $id) {
+        if (!$id) {
             return false;
         }
 
@@ -54,7 +54,7 @@ 
      */
     public function getEntry($id)
     {
-        if (! $this->entry) {
+        if (!$this->entry) {
             $this->entry = $this->model->findOrFail($id);
             $this->entry = $this->entry->withFakes();
         }
@@ -199,7 +199,7 @@ 
     {
         // If the default Page Length isn't in the menu's values, Add it the beginnin and resort all to show a croissant list.
         // assume both arrays are the same length.
-        if (! in_array($this->getDefaultPageLength(), $this->getOperationSetting('pageLengthMenu')[0])) {
+        if (!in_array($this->getDefaultPageLength(), $this->getOperationSetting('pageLengthMenu')[0])) {
             // Loop through 2 arrays of prop. page_length_menu
             foreach ($this->getOperationSetting('pageLengthMenu') as $key => &$page_length_choices) {
                 // This is a condition that should be always true.
@@ -230,7 +230,7 @@ 
     public function getPageLengthMenu()
     {
         // if already set, use that
-        if (! $this->getOperationSetting('pageLengthMenu')) {
+        if (!$this->getOperationSetting('pageLengthMenu')) {
             // try to get the menu settings from the config file
             $this->setOperationSetting('pageLengthMenu', config('backpack.crud.page_length_menu') ?? [[10, 25, 50, 100, -1], [10, 25, 50, 100, 'backpack::crud.all']]);
 

Doc Comments +1 added lines

@@ -175,6 +175,7 @@
 
     /**
      * Set the number of rows that should be show on the list view.
+     * @param integer $value
      */
     public function setDefaultPageLength($value)
     {

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

Spacing +3 added lines, -4 removed lines

@@ -24,11 +24,11 @@ 
 
         $saveOptions = collect($permissions)
             // Restrict list to allowed actions.
-            ->filter(function ($action, $permission) {
+            ->filter(function($action, $permission) {
                 return $this->hasAccess($permission);
             })
             // Generate list of possible actions.
-            ->mapWithKeys(function ($action, $permission) {
+            ->mapWithKeys(function($action, $permission) {
                 return [$action => $this->getSaveActionButtonName($action)];
             })
             ->all();
@@ -63,8 +63,7 @@ 
      */
     public function setSaveAction($forceSaveAction = null)
     {
-        $saveAction = $forceSaveAction ?:
-            \Request::input('save_action', config('backpack.crud.default_save_action', 'save_and_back'));
+        $saveAction = $forceSaveAction ?: \Request::input('save_action', config('backpack.crud.default_save_action', 'save_and_back'));
 
         if (config('backpack.crud.show_save_action_change', true) &&
             session('save_action', 'save_and_back') !== $saveAction) {