Laravel-Backpack / CRUD

src/CrudPanel.php

Spacing +7 added lines, -7 removed lines

@@ -70,7 +70,7 @@ 
      */
     public function setModel($model_namespace)
     {
-        if (! class_exists($model_namespace)) {
+        if (!class_exists($model_namespace)) {
             throw new \Exception('This model does not exist.', 404);
         }
 
@@ -112,7 +112,7 @@ 
     {
         $complete_route = $route.'.index';
 
-        if (! \Route::has($complete_route)) {
+        if (!\Route::has($complete_route)) {
             throw new \Exception('There are no routes for this route name.', 404);
         }
 
@@ -162,7 +162,7 @@ 
      */
     public function getFirstOfItsTypeInArray($type, $array)
     {
-        return array_first($array, function ($item) use ($type) {
+        return array_first($array, function($item) use ($type) {
             return $item['type'] == $type;
         });
     }
@@ -179,8 +179,8 @@ 
 
     public function sync($type, $fields, $attributes)
     {
-        if (! empty($this->{$type})) {
-            $this->{$type} = array_map(function ($field) use ($fields, $attributes) {
+        if (!empty($this->{$type})) {
+            $this->{$type} = array_map(function($field) use ($fields, $attributes) {
                 if (in_array($field['name'], (array) $fields)) {
                     $field = array_merge($field, $attributes);
                 }
@@ -206,8 +206,8 @@ 
                 }
             }
 
-            return $this->{$items} = array_merge($elements, array_filter($this->{$items}, function ($item) use ($items) {
-                return ! in_array($item['name'], $this->sort[$items]);
+            return $this->{$items} = array_merge($elements, array_filter($this->{$items}, function($item) use ($items) {
+                return !in_array($item['name'], $this->sort[$items]);
             }));
         }
 

Unused Use Statements +1 added lines, -1 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;
@@ -16,7 +17,6 @@ 
 use Backpack\CRUD\PanelTraits\Reorder;
 use Backpack\CRUD\PanelTraits\Update;
 use Backpack\CRUD\PanelTraits\ViewsAndRestoresRevisions;
-use Backpack\CRUD\PanelTraits\AutoFocus;
 
 class CrudPanel
 {

src/PanelTraits/Read.php

Spacing +2 added lines, -2 removed lines

@@ -75,7 +75,7 @@ 
     public function hasUploadFields($form)
     {
         $fields = $this->getFields($form);
-        $upload_fields = array_where($fields, function ($value, $key) {
+        $upload_fields = array_where($fields, function($value, $key) {
             return isset($value['upload']) && $value['upload'] == true;
         });
 
@@ -173,7 +173,7 @@ 
      */
     public function getCellView($column, $entry)
     {
-        if (! isset($column['type'])) {
+        if (!isset($column['type'])) {
             return \View::make('crud::columns.text')->with('crud', $this)->with('column', $column)->with('entry', $entry)->render();
         } else {
             if (view()->exists('vendor.backpack.crud.columns.'.$column['type'])) {

Doc Comments +1 added lines

@@ -33,6 +33,7 @@
      * Get all fields needed for the EDIT ENTRY form.
      *
      * @param  [integer] The id of the entry that is being edited.
+     * @param integer $id
      *
      * @return [array] The fields with attributes, fake attributes and values.
      */

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/resources/views/columns/array.blade.php

Indentation +5 added lines, -5 removed lines

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

src/app/Http/Controllers/CrudController.php

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
 
         // 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->setup();
             return $next($request);
         });
@@ -56,7 +56,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 +2 added lines, -4 removed lines

@@ -2,13 +2,11 @@
 
 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 Backpack\CRUD\app\Http\Requests\CrudRequest as StoreRequest;
-use Backpack\CRUD\app\Http\Requests\CrudRequest as UpdateRequest;
-use Backpack\CRUD\CrudPanel;
 // CRUD Traits for non-core features
 use Backpack\CRUD\app\Http\Controllers\CrudFeatures\AjaxTable;
 use Backpack\CRUD\app\Http\Controllers\CrudFeatures\Reorder;

src/app/Http/Controllers/CrudFeatures/Revisions.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
         // is somewhat superfluous.. however if we are POSTing, it makes sense to actually have data to post.
         // Perhaps the route shoud be better named to reflect this (e.g. just /model/{id}/revisions) (??)
         $revisionId = \Request::input('revision_id', false);
-        if (! $revisionId) {
+        if (!$revisionId) {
             abort(500, 'Can\'t restore revision without revision_id');
         } else {
             $this->crud->restoreRevision($id, $revisionId); // do the update

src/app/Http/Controllers/CrudFeatures/Reorder.php

Spacing +1 added lines, -1 removed lines

@@ -174,7 +174,7 @@
     {
         $this->crud->hasAccessOrFail('reorder');
 
-        if (! $this->crud->isReorderEnabled()) {
+        if (!$this->crud->isReorderEnabled()) {
             abort(403, 'Reorder is disabled.');
         }
 

src/app/Http/Controllers/CrudFeatures/AjaxTable.php

Spacing +2 added lines, -2 removed lines

@@ -14,7 +14,7 @@ 
 
         // crate an array with the names of the searchable columns
         $columns = collect($this->crud->columns)
-                    ->reject(function ($column, $key) {
+                    ->reject(function($column, $key) {
                         // the select_multiple columns are not searchable
                         return isset($column['type']) && $column['type'] == 'select_multiple';
                     })
@@ -27,7 +27,7 @@ 
         $dataTable = new \LiveControl\EloquentDataTable\DataTable($this->crud->query, $columns);
 
         // make the datatable use the column types instead of just echoing the text
-        $dataTable->setFormatRowFunction(function ($entry) {
+        $dataTable->setFormatRowFunction(function($entry) {
             // get the actual HTML for each row's cell
             $row_items = $this->crud->getRowViews($entry, $this->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;