Laravel-Backpack / CRUD

src/app/Http/Controllers/CrudController.php

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

@@ -2,14 +2,12 @@
 
 namespace Backpack\CRUD\app\Http\Controllers;
 
+use Backpack\CRUD\CrudPanel;
+use Backpack\CRUD\app\Http\Requests\CrudRequest as UpdateRequest;
 use Illuminate\Foundation\Bus\DispatchesJobs;
 use Illuminate\Foundation\Validation\ValidatesRequests;
-use Illuminate\Routing\Controller as BaseController;
-use Illuminate\Support\Facades\Form as Form;
 use Illuminate\Http\Request;
-use Backpack\CRUD\app\Http\Requests\CrudRequest as StoreRequest;
-use Backpack\CRUD\app\Http\Requests\CrudRequest as UpdateRequest;
-use Backpack\CRUD\CrudPanel;
+use Illuminate\Routing\Controller as BaseController;
 // CRUD Traits for non-core features
 use Backpack\CRUD\app\Http\Controllers\CrudFeatures\AjaxTable;
 use Backpack\CRUD\app\Http\Controllers\CrudFeatures\Reorder;

Spacing +3 added lines, -3 removed lines

@@ -27,12 +27,12 @@ 
 
     public function __construct()
     {
-        if (! $this->crud) {
+        if (!$this->crud) {
             $this->crud = new CrudPanel();
 
             // 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();
@@ -62,7 +62,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();
         }
 

src/CrudPanel.php

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

@@ -3,6 +3,7 @@ 
 namespace Backpack\CRUD;
 
 use Backpack\CRUD\PanelTraits\Access;
+use Backpack\CRUD\PanelTraits\AutoFocus;
 use Backpack\CRUD\PanelTraits\AutoSet;
 use Backpack\CRUD\PanelTraits\Buttons;
 use Backpack\CRUD\PanelTraits\Columns;
@@ -11,13 +12,12 @@ 
 use Backpack\CRUD\PanelTraits\FakeColumns;
 use Backpack\CRUD\PanelTraits\FakeFields;
 use Backpack\CRUD\PanelTraits\Fields;
+use Backpack\CRUD\PanelTraits\Filters;
 use Backpack\CRUD\PanelTraits\Query;
 use Backpack\CRUD\PanelTraits\Read;
 use Backpack\CRUD\PanelTraits\Reorder;
 use Backpack\CRUD\PanelTraits\Update;
 use Backpack\CRUD\PanelTraits\ViewsAndRestoresRevisions;
-use Backpack\CRUD\PanelTraits\AutoFocus;
-use Backpack\CRUD\PanelTraits\Filters;
 
 class CrudPanel
 {

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

@@ -33,7 +33,7 @@ 
         }
 
         // 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'])) {
@@ -112,7 +112,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;
         });
     }
@@ -177,7 +177,7 @@ 
         $this->type = $options['type'];
         $this->label = $options['label'];
 
-        if (! isset($options['placeholder'])) {
+        if (!isset($options['placeholder'])) {
             $this->placeholder = '';
         } else {
             $this->placeholder = $options['placeholder'];
@@ -194,16 +194,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.');
         }
     }