Laravel-Backpack / CRUD

src/PanelTraits/AutoFocus.php

Doc Comments +3 added lines

@@ -11,6 +11,9 @@
         return $this->autoFocusOnFirstField;
     }
 
+    /**
+     * @param boolean $value
+     */
     public function setAutoFocusOnFirstField($value)
     {
         return $this->autoFocusOnFirstField = (bool) $value;

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

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
     if (is_array($array) && count($array)) {
         $list = [];
         foreach ($array as $item) {
-            if( isset( $item->{$column['visible_key']} ) ) {
+            if (isset($item->{$column['visible_key']} )) {
                 $list[] = $item->{$column['visible_key']};
             } elseif (is_array($item) && isset($item[$column['visible_key']])) {
                 $list[] = $item[$column['visible_key']];

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

Indentation +5 added lines, -5 removed lines

@@ -1,11 +1,11 @@
 {{-- enumerate the values in an array  --}}
 <td>
     <?php
-    	$array = $entry->{$column['name']};
-    	// the value should be an array wether or not attribute casting is used
-    	if (!is_array($array)) {
-    		$array = json_decode($array, true);
-    	}
+        $array = $entry->{$column['name']};
+        // the value should be an array wether or not attribute casting is used
+        if (!is_array($array)) {
+            $array = json_decode($array, true);
+        }
         if ($array && count($array)) {
             echo count($array).' items';
         } else {

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

Braces +1 added lines, -2 removed lines

@@ -12,8 +12,7 @@
     if(is_array($items)) {
         if (count($items)) {
             $items = json_encode($items);
-        }
-        else
+        } else
         {
             $items = '[]';
         }

Spacing +3 added lines, -3 removed lines

@@ -3,13 +3,13 @@
 <?php
     $max = isset($field['max']) && (int) $field['max'] > 0 ? $field['max'] : -1;
     $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']);
+    $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
-    if(is_array($items)) {
+    if (is_array($items)) {
         if (count($items)) {
             $items = json_encode($items);
         }

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

Spacing +2 added lines, -2 removed lines

@@ -1,11 +1,11 @@
 <!-- text input -->
 <?php
 
-$value = old($field['name']) ? (old($field['name'])) : (isset($field['value']) ? ($field['value']) : (isset($field['default']) ? ($field['default']) : '' ));
+$value = old($field['name']) ? (old($field['name'])) : (isset($field['value']) ? ($field['value']) : (isset($field['default']) ? ($field['default']) : ''));
 
 // if attribute casting is used, convert to JSON
 if (is_array($value)) {
-    $value = json_encode((object)$value);
+    $value = json_encode((object) $value);
 } elseif (is_object($value)) {
     $value = json_encode($value);
 } else {

src/resources/views/fields/date_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' );
+    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/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 +5 added lines, -7 removed lines

@@ -3,18 +3,16 @@
 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\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
 {