Laravel-Backpack / CRUD

src/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;
 
 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/resources/views-elfinder/filepicker.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
     <!-- Include jQuery, jQuery UI, elFinder (REQUIRED) -->
 
     <?php
-    $mimeTypes = implode(',', array_map(function ($t) {
+    $mimeTypes = implode(',', array_map(function($t) {
         return "'".$t."'";
     }, explode(',', $type)));
     ?>

src/PanelTraits/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/PanelTraits/Filters.php

Doc Comments +1 added lines, -1 removed lines

@@ -124,7 +124,7 @@
 
     /**
      * Determine if the current CRUD action is a list operation (using standard or ajax DataTables).
-     * @return bool
+     * @return boolean|null
      */
     public function doingListOperation()
     {

Spacing +8 added lines, -8 removed lines

@@ -11,7 +11,7 @@ 
      */
     public function filtersEnabled()
     {
-        return ! is_array($this->filters);
+        return !is_array($this->filters);
     }
 
     /**
@@ -59,7 +59,7 @@ 
         $this->enableFilters();
 
         // check if another filter with the same name exists
-        if (! isset($options['name'])) {
+        if (!isset($options['name'])) {
             abort(500, 'All your filters need names.');
         }
         if ($this->filters->contains('name', $options['name'])) {
@@ -160,7 +160,7 @@ 
     {
         $filter = $this->filters->firstWhere('name', $name);
 
-        if (! $filter) {
+        if (!$filter) {
             abort(500, 'CRUD Filter "'.$name.'" not found. Please check the filter exists before you modify it.');
         }
 
@@ -175,7 +175,7 @@ 
 
     public function removeFilter($name)
     {
-        $this->filters = $this->filters->reject(function ($filter) use ($name) {
+        $this->filters = $this->filters->reject(function($filter) use ($name) {
             return $filter->name == $name;
         });
     }
@@ -259,16 +259,16 @@ 
 
     public function checkOptionsIntegrity($options)
     {
-        if (! isset($options['name'])) {
+        if (!isset($options['name'])) {
             abort(500, 'Please make sure all your filters have names.');
         }
-        if (! isset($options['type'])) {
+        if (!isset($options['type'])) {
             abort(500, 'Please make sure all your filters have types.');
         }
-        if (! \View::exists('crud::filters.'.$options['type'])) {
+        if (!\View::exists('crud::filters.'.$options['type'])) {
             abort(500, 'No filter view named "'.$options['type'].'.blade.php" was found.');
         }
-        if (! isset($options['label'])) {
+        if (!isset($options['label'])) {
             abort(500, 'Please make sure all your filters have labels.');
         }
     }

src/ModelTraits/SpatieTranslatable/SluggableObserver.php

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

@@ -2,8 +2,8 @@
 
 namespace Backpack\CRUD\ModelTraits\SpatieTranslatable;
 
-use Illuminate\Database\Eloquent\Model;
 use Illuminate\Contracts\Events\Dispatcher;
+use Illuminate\Database\Eloquent\Model;
 
 class SluggableObserver extends \Cviebrock\EloquentSluggable\SluggableObserver
 {

src/ModelTraits/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/ModelTraits/SpatieTranslatable/Sluggable.php

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

@@ -2,9 +2,9 @@
 
 namespace Backpack\CRUD\ModelTraits\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

@@ -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 "

src/resources/views/fields/checklist_dependency.blade.php

Indentation +4 added lines, -4 removed lines

@@ -23,13 +23,13 @@
             }
         }
 
-      //for update form, get initial state of the entity
-      if( isset($id) && $id ){
+        //for update form, get initial state of the entity
+        if( isset($id) && $id ){
 
         //get entity with relations for primary dependency
         $entity_dependencies = $entity_model->with($primary_dependency['entity'])
-          ->with($primary_dependency['entity'].'.'.$primary_dependency['entity_secondary'])
-          ->find($id);
+            ->with($primary_dependency['entity'].'.'.$primary_dependency['entity_secondary'])
+            ->find($id);
 
             $secondaries_from_primary = [];
 

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
         }
 
       //for update form, get initial state of the entity
-      if( isset($id) && $id ){
+      if (isset($id) && $id) {
 
         //get entity with relations for primary dependency
         $entity_dependencies = $entity_model->with($primary_dependency['entity'])

src/PanelTraits/FakeColumns.php

Spacing +3 added lines, -3 removed lines

@@ -24,19 +24,19 @@
             if (isset($field['fake']) && $field['fake'] == true) {
                 // add it to the request in its appropriate variable - the one defined, if defined
                 if (isset($field['store_in'])) {
-                    if (! in_array($field['store_in'], $fakeFieldsColumnsArray, true)) {
+                    if (!in_array($field['store_in'], $fakeFieldsColumnsArray, true)) {
                         array_push($fakeFieldsColumnsArray, $field['store_in']);
                     }
                 } else {
                     //otherwise in the one defined in the $crud variable
-                    if (! in_array('extras', $fakeFieldsColumnsArray, true)) {
+                    if (!in_array('extras', $fakeFieldsColumnsArray, true)) {
                         array_push($fakeFieldsColumnsArray, 'extras');
                     }
                 }
             }
         }
 
-        if (! count($fakeFieldsColumnsArray)) {
+        if (!count($fakeFieldsColumnsArray)) {
             $fakeFieldsColumnsArray = ['extras'];
         }
 

Doc Comments +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
      * been found.
      *
      * @param string   $form The CRUD form. Can be 'create', 'update' or 'both'. Default is 'create'.
-     * @param int|bool $id   Optional entity ID needed in the case of the update form.
+     * @param integer $id   Optional entity ID needed in the case of the update form.
      *
      * @return array The fake columns array.
      */