Laravel-Backpack / CRUD

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

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 <?php
 // if the column has been cast to Carbon or Date (using attribute casting)
 // get the value as a date string
-if (isset($field['value']) && ( $field['value'] instanceof \Carbon\Carbon || $field['value'] instanceof \Jenssegers\Date\Date )) {
+if (isset($field['value']) && ($field['value'] instanceof \Carbon\Carbon || $field['value'] instanceof \Jenssegers\Date\Date)) {
     $field['value'] = $field['value']->toDateTimeString();
 }
 ?>

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

Spacing +2 added lines, -2 removed lines

@@ -3,11 +3,11 @@
 <?php
     // if the column has been cast to Carbon or Date (using attribute casting)
     // get the value as a date string
-    if (isset($field['value']) && ( $field['value'] instanceof \Carbon\Carbon || $field['value'] instanceof \Jenssegers\Date\Date )) {
+    if (isset($field['value']) && ($field['value'] instanceof \Carbon\Carbon || $field['value'] instanceof \Jenssegers\Date\Date)) {
         $field['value'] = $field['value']->format('Y-m-d');
     }
 
-    $field_language = isset($field['date_picker_options']['language'])?$field['date_picker_options']['language']:\App::getLocale();
+    $field_language = isset($field['date_picker_options']['language']) ? $field['date_picker_options']['language'] : \App::getLocale();
 ?>
 
 <div @include('crud::inc.field_wrapper_attributes') >

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

Spacing +1 added lines, -1 removed lines

@@ -5,7 +5,7 @@
     $min = isset($field['min']) && (int) $field['min'] > 0 ? $field['min'] : -1;
     $item_name = strtolower(isset($field['entity_singular']) && !empty($field['entity_singular']) ? $field['entity_singular'] : $field['label']);
 
-    $items = old($field['name']) ? (old($field['name'])) : (isset($field['value']) ? ($field['value']) : (isset($field['default']) ? ($field['default']) : '' ));
+    $items = old($field['name']) ? (old($field['name'])) : (isset($field['value']) ? ($field['value']) : (isset($field['default']) ? ($field['default']) : ''));
 
     // make sure not matter the attribute casting
     // the $items variable contains a properly defined JSON

src/resources/views/reorder.blade.php

Spacing +2 added lines, -2 removed lines

@@ -76,10 +76,10 @@
                 <ol class="sortable">
                 <?php
                     $all_entries = collect($entries->all())->sortBy('lft')->keyBy($crud->getModel()->getKeyName());
-                    $root_entries = $all_entries->filter(function ($item) {
+                    $root_entries = $all_entries->filter(function($item) {
                         return $item->parent_id == 0;
                     });
-                    foreach ($root_entries as $key => $entry){
+                    foreach ($root_entries as $key => $entry) {
                         $root_entries[$key] = tree_element($entry, $key, $all_entries, $crud);
                     }
                 ?>

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

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,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.'%');
         });

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

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/app/Http/Controllers/CrudController.php

Unused Use Statements +5 added lines, -7 removed lines

@@ -3,19 +3,17 @@
 namespace Backpack\CRUD\app\Http\Controllers;
 
 use Backpack\CRUD\CrudPanel;
-use Illuminate\Http\Request;
-use Illuminate\Support\Facades\Form as Form;
+use Backpack\CRUD\app\Http\Controllers\CrudFeatures\AjaxTable;
+use Backpack\CRUD\app\Http\Controllers\CrudFeatures\Reorder;
 use Illuminate\Foundation\Bus\DispatchesJobs;
-use Illuminate\Routing\Controller as BaseController;
 use Illuminate\Foundation\Validation\ValidatesRequests;
-use Backpack\CRUD\app\Http\Controllers\CrudFeatures\Reorder;
-use Backpack\CRUD\app\Http\Controllers\CrudFeatures\AjaxTable;
+use Illuminate\Http\Request;
+use Illuminate\Routing\Controller as BaseController;
 // CRUD Traits for non-core features
 use Backpack\CRUD\app\Http\Controllers\CrudFeatures\Revisions;
 use Backpack\CRUD\app\Http\Controllers\CrudFeatures\SaveActions;
-use Backpack\CRUD\app\Http\Requests\CrudRequest as StoreRequest;
-use Backpack\CRUD\app\Http\Requests\CrudRequest as UpdateRequest;
 use Backpack\CRUD\app\Http\Controllers\CrudFeatures\ShowDetailsRow;
+use Backpack\CRUD\app\Http\Requests\CrudRequest as UpdateRequest;
 
 class CrudController extends BaseController
 {

Indentation +28 added lines, -28 removed lines

@@ -125,7 +125,7 @@ 
 
         // show a success message
         if($this->crud->model::find($item->id))
-         \Alert::success(trans('backpack::crud.insert_success'))->flash();
+            \Alert::success(trans('backpack::crud.insert_success'))->flash();
 
         // save the redirect choice for next time
         $this->setSaveAction();
@@ -159,18 +159,18 @@ 
         if($this->crud->eagerLoad && !empty($this->crud->eagerLoad))
         foreach($this->crud->eagerLoad as $relation_name)
         {
-         if($this->data['entry'] && $rel = $this->data['entry']->{$relation_name})
-         {
-          foreach($this->data['fields'] as $field_name => $field)
-          {
-           if(starts_with($field_name, $relation_name))
-           {
+            if($this->data['entry'] && $rel = $this->data['entry']->{$relation_name})
+            {
+            foreach($this->data['fields'] as $field_name => $field)
+            {
+            if(starts_with($field_name, $relation_name))
+            {
             $column_name = str_replace("{$relation_name}.", '', $field_name);
 
             $this->data['fields'][$field_name]['value'] = $rel->{$column_name};
-           }
-          }
-         }
+            }
+            }
+            }
         }
 
         // load the view from /resources/views/vendor/backpack/crud/ if it exists, otherwise load the one in the package
@@ -271,23 +271,23 @@ 
 
     public function handleEagerLoad($item)
     {
-     collect($this->crud->eagerLoad)->each(function($relation_name, $key) use ($item)
-     {
-      $data = collect(request()->all())
-      ->filter(function($value, $key) use ($relation_name)
-      {
-       return starts_with($key, $relation_name);
-      })
-      ->mapWithKeys(function($value, $key) use ($relation_name)
-      {
-       return [str_replace("{$relation_name}_", '', $key) => $value ?? ''];
-      })
-      ->toArray();
-
-      if(!$rel = $item->{$relation_name})
-       $item->{$relation_name}()->create($data);
-      else
-       $item->{$relation_name}->update($data);
-     });
+        collect($this->crud->eagerLoad)->each(function($relation_name, $key) use ($item)
+        {
+        $data = collect(request()->all())
+        ->filter(function($value, $key) use ($relation_name)
+        {
+        return starts_with($key, $relation_name);
+        })
+        ->mapWithKeys(function($value, $key) use ($relation_name)
+        {
+        return [str_replace("{$relation_name}_", '', $key) => $value ?? ''];
+        })
+        ->toArray();
+
+        if(!$rel = $item->{$relation_name})
+        $item->{$relation_name}()->create($data);
+        else
+        $item->{$relation_name}->update($data);
+        });
     }
 }

Spacing +11 added lines, -11 removed lines

@@ -33,13 +33,13 @@ 
 
     public function __construct()
     {
-        if (! $this->crud) {
+        if (!$this->crud) {
             $this->crud = app()->make(CrudPanel::class);
-            $this->crud->ajax_table=false;
+            $this->crud->ajax_table = false;
 
             // call the setup function inside this closure to also have the request there
             // this way, developers can use things stored in session (auth variables, etc)
-            $this->middleware(function ($request, $next) {
+            $this->middleware(function($request, $next) {
                 $this->request = $request;
                 $this->crud->request = $request;
                 $this->setup();
@@ -69,7 +69,7 @@ 
         $this->data['title'] = ucfirst($this->crud->entity_name_plural);
 
         // get all entries if AJAX is not enabled
-        if (! $this->data['crud']->ajaxTable()) {
+        if (!$this->data['crud']->ajaxTable()) {
             $this->data['entries'] = $this->data['crud']->getEntries();
         }
 
@@ -124,7 +124,7 @@ 
         $this->data['entry'] = $this->crud->entry = $item;
 
         // show a success message
-        if($this->crud->model::find($item->id))
+        if ($this->crud->model::find($item->id))
          \Alert::success(trans('backpack::crud.insert_success'))->flash();
 
         // save the redirect choice for next time
@@ -156,14 +156,14 @@ 
 
         $this->data['id'] = $id;
 
-        if($this->crud->eagerLoad && !empty($this->crud->eagerLoad))
-        foreach($this->crud->eagerLoad as $relation_name)
+        if ($this->crud->eagerLoad && !empty($this->crud->eagerLoad))
+        foreach ($this->crud->eagerLoad as $relation_name)
         {
-         if($this->data['entry'] && $rel = $this->data['entry']->{$relation_name})
+         if ($this->data['entry'] && $rel = $this->data['entry']->{$relation_name})
          {
-          foreach($this->data['fields'] as $field_name => $field)
+          foreach ($this->data['fields'] as $field_name => $field)
           {
-           if(starts_with($field_name, $relation_name))
+           if (starts_with($field_name, $relation_name))
            {
             $column_name = str_replace("{$relation_name}.", '', $field_name);
 
@@ -284,7 +284,7 @@ 
       })
       ->toArray();
 
-      if(!$rel = $item->{$relation_name})
+      if (!$rel = $item->{$relation_name})
        $item->{$relation_name}()->create($data);
       else
        $item->{$relation_name}->update($data);

Braces +11 added lines, -8 removed lines

@@ -124,8 +124,9 @@ 
         $this->data['entry'] = $this->crud->entry = $item;
 
         // show a success message
-        if($this->crud->model::find($item->id))
-         \Alert::success(trans('backpack::crud.insert_success'))->flash();
+        if($this->crud->model::find($item->id)) {
+                 \Alert::success(trans('backpack::crud.insert_success'))->flash();
+        }
 
         // save the redirect choice for next time
         $this->setSaveAction();
@@ -156,8 +157,8 @@ 
 
         $this->data['id'] = $id;
 
-        if($this->crud->eagerLoad && !empty($this->crud->eagerLoad))
-        foreach($this->crud->eagerLoad as $relation_name)
+        if($this->crud->eagerLoad && !empty($this->crud->eagerLoad)) {
+                foreach($this->crud->eagerLoad as $relation_name)
         {
          if($this->data['entry'] && $rel = $this->data['entry']->{$relation_name})
          {
@@ -166,6 +167,7 @@ 
            if(starts_with($field_name, $relation_name))
            {
             $column_name = str_replace("{$relation_name}.", '', $field_name);
+        }
 
             $this->data['fields'][$field_name]['value'] = $rel->{$column_name};
            }
@@ -284,10 +286,11 @@ 
       })
       ->toArray();
 
-      if(!$rel = $item->{$relation_name})
-       $item->{$relation_name}()->create($data);
-      else
-       $item->{$relation_name}->update($data);
+      if(!$rel = $item->{$relation_name}) {
+             $item->{$relation_name}()->create($data);
+      } else {
+             $item->{$relation_name}->update($data);
+      }
      });
     }
 }

src/CrudServiceProvider.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace Backpack\CRUD;
 
-use Route;
 use Illuminate\Support\ServiceProvider;
 
 class CrudServiceProvider extends ServiceProvider

Spacing +3 added lines, -3 removed lines

@@ -76,7 +76,7 @@ 
      */
     public function register()
     {
-        $this->app->bind('CRUD', function ($app) {
+        $this->app->bind('CRUD', function($app) {
             return new CRUD($app);
         });
 
@@ -91,7 +91,7 @@ 
         $this->commands($this->commands);
 
         // map the elfinder prefix
-        if (! \Config::get('elfinder.route.prefix')) {
+        if (!\Config::get('elfinder.route.prefix')) {
             \Config::set('elfinder.route.prefix', \Config::get('backpack.base.route_prefix').'/elfinder');
         }
     }
@@ -111,7 +111,7 @@ 
     {
         $crudPubPath = public_path('vendor/backpack/crud');
 
-        if (! is_dir($crudPubPath)) {
+        if (!is_dir($crudPubPath)) {
             return true;
         }