Laravel-Backpack / CRUD

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

Spacing +3 added lines, -3 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 )) {
-        $field['value'] = $field['value']->format( 'Y-m-d H:i:s' );
+    if (isset($field['value']) && ($field['value'] instanceof \Carbon\Carbon || $field['value'] instanceof \Jenssegers\Date\Date)) {
+        $field['value'] = $field['value']->format('Y-m-d H:i:s');
     }
 
-    $field_language = isset($field['datetime_picker_options']['language'])?$field['datetime_picker_options']['language']:\App::getLocale();
+    $field_language = isset($field['datetime_picker_options']['language']) ? $field['datetime_picker_options']['language'] : \App::getLocale();
 ?>
 
 <div @include('crud::inc.field_wrapper_attributes') >

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.');
         }
     }

src/app/Http/Controllers/CrudController.php

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();
         }
 

Unused Use Statements +6 added lines, -8 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 Backpack\CRUD\app\Http\Controllers\CrudFeatures\Views;
 use Illuminate\Foundation\Bus\DispatchesJobs;
-use Illuminate\Routing\Controller as BaseController;
 use Illuminate\Foundation\Validation\ValidatesRequests;
-use Backpack\CRUD\app\Http\Controllers\CrudFeatures\Views;
-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\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
 {

src/resources/views/columns/select.blade.php

Indentation +4 added lines, -4 removed lines

@@ -1,8 +1,8 @@
 {{-- single relationships (1-1, 1-n) --}}
 <td>
 	<?php
-		if ($entry->{$column['entity']}) {
-	    	echo $entry->{$column['entity']}->{$column['attribute']};
-	    }
-	?>
+        if ($entry->{$column['entity']}) {
+            echo $entry->{$column['entity']}->{$column['attribute']};
+        }
+    ?>
 </td>
\ No newline at end of file

src/PanelTraits/Columns.php

Spacing +5 added lines, -5 removed lines

@@ -54,7 +54,7 @@ 
     public function addColumn($column)
     {
         // if a string was passed, not an array, change it to an array
-        if (! is_array($column)) {
+        if (!is_array($column)) {
             $column = ['name' => $column];
         }
 
@@ -105,7 +105,7 @@ 
      */
     public function addDefaultLabel($array)
     {
-        if (! array_key_exists('label', (array) $array) && array_key_exists('name', (array) $array)) {
+        if (!array_key_exists('label', (array) $array) && array_key_exists('name', (array) $array)) {
             $array = array_merge(['label' => ucfirst($this->makeLabel($array['name']))], $array);
 
             return $array;
@@ -139,8 +139,8 @@ 
      */
     public function remove($entity, $fields)
     {
-        return array_values(array_filter($this->{$entity}, function ($field) use ($fields) {
-            return ! in_array($field['name'], (array) $fields);
+        return array_values(array_filter($this->{$entity}, function($field) use ($fields) {
+            return !in_array($field['name'], (array) $fields);
         }));
     }
 
@@ -191,7 +191,7 @@ 
     {
         $columns = $this->getColumns();
 
-        return collect($columns)->pluck('entity')->reject(function ($value, $key) {
+        return collect($columns)->pluck('entity')->reject(function($value, $key) {
             return $value == null;
         })->toArray();
     }

src/PanelTraits/Fields.php

Spacing +4 added lines, -4 removed lines

@@ -24,12 +24,12 @@ 
         }
 
         // if the label is missing, we should set it
-        if (! isset($complete_field_array['label'])) {
+        if (!isset($complete_field_array['label'])) {
             $complete_field_array['label'] = ucfirst($complete_field_array['name']);
         }
 
         // if the field type is missing, we should set it
-        if (! isset($complete_field_array['type'])) {
+        if (!isset($complete_field_array['type'])) {
             $complete_field_array['type'] = $this->getFieldTypeFromDbColumnType($complete_field_array['name']);
         }
 
@@ -139,7 +139,7 @@ 
      */
     public function removeFields($array_of_names, $form = 'both')
     {
-        if (! empty($array_of_names)) {
+        if (!empty($array_of_names)) {
             foreach ($array_of_names as $name) {
                 $this->removeField($name, $form);
             }
@@ -200,7 +200,7 @@ 
                 $jsonCastables = ['array', 'object', 'json'];
                 $fieldCasting = $this->model->getCasts()[$field['name']];
 
-                if (in_array($fieldCasting, $jsonCastables) && isset($data[$field['name']]) && ! empty($data[$field['name']]) && ! is_array($data[$field['name']])) {
+                if (in_array($fieldCasting, $jsonCastables) && isset($data[$field['name']]) && !empty($data[$field['name']]) && !is_array($data[$field['name']])) {
                     try {
                         $data[$field['name']] = json_decode($data[$field['name']]);
                     } catch (Exception $e) {