Laravel-Backpack / CRUD

src/resources/views/reorder.blade.php

Indentation +26 added lines, -26 removed lines

@@ -20,42 +20,42 @@ 
 
 @section('content')
 <?php
-  function tree_element($entry, $key, $all_entries, $crud)
-  {
+    function tree_element($entry, $key, $all_entries, $crud)
+    {
     if (!isset($entry->tree_element_shown)) {
-      // mark the element as shown
-      $all_entries[$key]->tree_element_shown = true;
-      $entry->tree_element_shown = true;
+        // mark the element as shown
+        $all_entries[$key]->tree_element_shown = true;
+        $entry->tree_element_shown = true;
 
-      // show the tree element
-      echo '<li id="list_'.$entry->getKey().'">';
-      echo '<div><span class="disclose"><span></span></span>'.$entry->{$crud->reorder_label}.'</div>';
+        // show the tree element
+        echo '<li id="list_'.$entry->getKey().'">';
+        echo '<div><span class="disclose"><span></span></span>'.$entry->{$crud->reorder_label}.'</div>';
 
-      // see if this element has any children
-      $children = [];
-      foreach ($all_entries as $key => $subentry) {
+        // see if this element has any children
+        $children = [];
+        foreach ($all_entries as $key => $subentry) {
         if ($subentry->parent_id == $entry->getKey()) {
-          $children[] = $subentry;
+            $children[] = $subentry;
+        }
         }
-      }
 
-      $children = collect($children)->sortBy('lft');
+        $children = collect($children)->sortBy('lft');
 
-      // if it does have children, show them
-      if (count($children)) {
+        // if it does have children, show them
+        if (count($children)) {
         echo '<ol>';
         foreach ($children as $key => $child) {
-          $children[$key] = tree_element($child, $child->getKey(), $all_entries, $crud);
+            $children[$key] = tree_element($child, $child->getKey(), $all_entries, $crud);
         }
         echo '</ol>';
-      }
-      echo '</li>';
+        }
+        echo '</li>';
     }
 
     return $entry;
-  }
+    }
 
- ?>
+    ?>
 <div class="row">
   <div class="col-md-8 col-md-offset-2">
     @if ($crud->hasAccess('list'))
@@ -73,15 +73,15 @@ 
 
           <ol class="sortable">
             <?php
-              $all_entries = collect($entries->all())->sortBy('lft')->keyBy($crud->getModel()->getKeyName());
-              $root_entries = $all_entries->filter(function($item) {
+                $all_entries = collect($entries->all())->sortBy('lft')->keyBy($crud->getModel()->getKeyName());
+                $root_entries = $all_entries->filter(function($item) {
                 return $item->parent_id == 0;
-              });
-             ?>
+                });
+                ?>
             @foreach ($root_entries as $key => $entry)
               <?php
                 $root_entries[$key] = tree_element($entry, $key, $all_entries, $crud);
-              ?>
+                ?>
             @endforeach
           </ol>
 

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