Laravel-Backpack / CRUD

src/app/Library/CrudPanel/Traits/Settings.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
      */
     public function settings()
     {
-        return Arr::sort($this->settings, function ($value, $key) {
+        return Arr::sort($this->settings, function($value, $key) {
             return $key;
         });
     }

src/app/Library/CrudPanel/Traits/Search.php

Indentation +4 added lines, -4 removed lines

@@ -224,10 +224,10 @@
         // add the details_row button to the first column
         if ($this->getOperationSetting('detailsRow')) {
             $details_row_button = \View::make('crud::columns.inc.details_row_button')
-                                           ->with('crud', $this)
-                                           ->with('entry', $entry)
-                                           ->with('row_number', $rowNumber)
-                                           ->render();
+                                            ->with('crud', $this)
+                                            ->with('entry', $entry)
+                                            ->with('row_number', $rowNumber)
+                                            ->render();
             $row_items[0] = $details_row_button.$row_items[0];
         }
 

Spacing +9 added lines, -9 removed lines

@@ -23,9 +23,9 @@ 
      */
     public function applySearchTerm($searchTerm)
     {
-        return $this->query->where(function ($query) use ($searchTerm) {
+        return $this->query->where(function($query) use ($searchTerm) {
             foreach ($this->columns() as $column) {
-                if (! isset($column['type'])) {
+                if (!isset($column['type'])) {
                     abort(400, 'Missing column type when trying to apply search term.');
                 }
 
@@ -85,7 +85,7 @@ 
 
                 case 'select':
                 case 'select_multiple':
-                    $query->orWhereHas($column['entity'], function ($q) use ($column, $searchTerm, $searchOperator) {
+                    $query->orWhereHas($column['entity'], function($q) use ($column, $searchTerm, $searchOperator) {
                         $q->where($this->getColumnWithTableNamePrefixed($q, $column['attribute']), $searchOperator, '%'.$searchTerm.'%');
                     });
                     break;
@@ -109,7 +109,7 @@ 
                 $column_number = (int) $order['column'];
                 $column_direction = (strtolower((string) $order['dir']) == 'asc' ? 'ASC' : 'DESC');
                 $column = $this->findColumnById($column_number);
-                if ($column['tableColumn'] && ! isset($column['orderLogic'])) {
+                if ($column['tableColumn'] && !isset($column['orderLogic'])) {
                     if (method_exists($this->model, 'translationEnabled') &&
                         $this->model->translationEnabled() &&
                         $this->model->isTranslatableAttribute($column['name']) &&
@@ -136,12 +136,12 @@ 
         $table = $this->model->getTable();
         $key = $this->model->getKeyName();
 
-        $hasOrderByPrimaryKey = collect($orderBy)->some(function ($item) use ($key, $table) {
+        $hasOrderByPrimaryKey = collect($orderBy)->some(function($item) use ($key, $table) {
             return (isset($item['column']) && $item['column'] === $key)
                 || (isset($item['sql']) && str_contains($item['sql'], "$table.$key"));
         });
 
-        if (! $hasOrderByPrimaryKey) {
+        if (!$hasOrderByPrimaryKey) {
             $this->orderByWithPrefix($key, 'DESC');
         }
     }
@@ -320,13 +320,13 @@ 
         if (isset($column['type'])) {
             // create a list of paths to column blade views
             // including the configured view_namespaces
-            $columnPaths = array_map(function ($item) use ($column) {
+            $columnPaths = array_map(function($item) use ($column) {
                 return $item.'.'.$column['type'];
             }, ViewNamespaces::getFor('columns'));
 
             // but always fall back to the stock 'text' column
             // if a view doesn't exist
-            if (! in_array('crud::columns.text', $columnPaths)) {
+            if (!in_array('crud::columns.text', $columnPaths)) {
                 $columnPaths[] = 'crud::columns.text';
             }
 
@@ -365,7 +365,7 @@ 
      */
     private function renderCellView($view, $column, $entry, $rowNumber = false)
     {
-        if (! view()->exists($view)) {
+        if (!view()->exists($view)) {
             $view = 'crud::columns.text'; // fallback to text column
         }
 

tests/Unit/CrudPanel/BaseCrudPanelTest.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
     {
         parent::setUp();
 
-        $this->app->singleton('crud', function ($app) {
+        $this->app->singleton('crud', function($app) {
             return new CrudPanel($app);
         });
         $this->crudPanel = app('crud');

src/app/Library/CrudPanel/CrudFilter.php

Braces +3 added lines, -1 removed lines

@@ -389,10 +389,12 @@
         return $this->fallbackLogic($value);
     }
 
-    public function else($value)
+    public function else {
+        ($value)
     {
         return $this->fallbackLogic($value);
     }
+    }
 
     // ---------------
     // PRIVATE METHODS

Spacing +5 added lines, -5 removed lines

@@ -26,7 +26,7 @@ 
 
     public function __construct($options, $values, $logic, $fallbackLogic)
     {
-        if (! backpack_pro()) {
+        if (!backpack_pro()) {
             throw new BackpackProRequiredException('Filter');
         }
         // if filter exists
@@ -89,7 +89,7 @@ 
      */
     public function wasNotApplied()
     {
-        return ! $this->applied;
+        return !$this->applied;
     }
 
     /**
@@ -110,7 +110,7 @@ 
 
         $input = $input ?? new ParameterBag($this->crud()->getRequest()->all());
 
-        if (! $input->has($this->name)) {
+        if (!$input->has($this->name)) {
             // if fallback logic was supplied and is a closure
             if (is_callable($this->fallbackLogic)) {
                 return ($this->fallbackLogic)();
@@ -152,7 +152,7 @@ 
             $namespaces = array_merge([$this->viewNamespace], $namespaces);
         }
 
-        return array_map(function ($item) use ($type) {
+        return array_map(function($item) use ($type) {
             return $item.'.'.$type;
         }, $namespaces);
     }
@@ -314,7 +314,7 @@ 
      */
     public function values($value)
     {
-        $this->values = (! is_string($value) && is_callable($value)) ? $value() : $value;
+        $this->values = (!is_string($value) && is_callable($value)) ? $value() : $value;
 
         return $this->save();
     }

src/app/Library/CrudPanel/Traits/Buttons.php

Spacing +13 added lines, -13 removed lines

@@ -25,7 +25,7 @@ 
         $otherButtons = collect([]);
 
         // we get the buttons that belong to the specified stack
-        $stackButtons = $this->buttons()->reject(function ($item) use ($stack, $otherButtons) {
+        $stackButtons = $this->buttons()->reject(function($item) use ($stack, $otherButtons) {
             if ($item->stack != $stack) {
                 // if the button does not belong to this stack we just add it for merging later
                 $otherButtons->push($item);
@@ -37,8 +37,8 @@ 
         });
 
         // we parse the ordered buttons
-        collect($order)->each(function ($btnKey) use ($newButtons, $stackButtons) {
-            if (! $button = $stackButtons->where('name', $btnKey)->first()) {
+        collect($order)->each(function($btnKey) use ($newButtons, $stackButtons) {
+            if (!$button = $stackButtons->where('name', $btnKey)->first()) {
                 abort(500, 'Button name [«'.$btnKey.'»] not found.');
             }
             $newButtons->push($button);
@@ -48,7 +48,7 @@ 
         // we add the remaining buttons to the end of the ordered ones
         if (count($newButtons) < count($stackButtons)) {
             foreach ($stackButtons as $button) {
-                if (! $newButtons->where('name', $button->name)->first()) {
+                if (!$newButtons->where('name', $button->name)->first()) {
                     $newButtons->push($button);
                 }
             }
@@ -114,7 +114,7 @@ 
          */
         $button = $this->buttons()->firstWhere('name', $name);
 
-        if (! $button) {
+        if (!$button) {
             abort(500, 'CRUD Button "'.$name.'" not found. Please check the button exists before you modify it.');
         }
 
@@ -135,7 +135,7 @@ 
      */
     public function removeButton($name, $stack = null)
     {
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($name, $stack) {
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($name, $stack) {
             return $stack == null ? $button->name == $name : ($button->stack == $stack) && ($button->name == $name);
         }));
     }
@@ -146,7 +146,7 @@ 
      */
     public function removeButtons($names, $stack = null)
     {
-        if (! empty($names)) {
+        if (!empty($names)) {
             foreach ($names as $name) {
                 $this->removeButton($name, $stack);
             }
@@ -160,14 +160,14 @@ 
 
     public function removeAllButtonsFromStack($stack)
     {
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($stack) {
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($stack) {
             return $button->stack == $stack;
         }));
     }
 
     public function removeButtonFromStack($name, $stack)
     {
-        $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($name, $stack) {
+        $this->setOperationSetting('buttons', $this->buttons()->reject(function($button) use ($name, $stack) {
             return $button->name == $name && $button->stack == $stack;
         }));
     }
@@ -185,15 +185,15 @@ 
         $destinationButton = $this->firstButtonWhere('name', $destination);
         $destinationKey = $this->getButtonKey($destination);
         $newDestinationKey = ($where == 'before' ? $destinationKey : $destinationKey + 1);
-        $newButtons = $this->buttons()->filter(function ($value, $key) use ($target) {
+        $newButtons = $this->buttons()->filter(function($value, $key) use ($target) {
             return $value->name != $target;
         });
 
-        if (! $targetButton) {
+        if (!$targetButton) {
             return;
         }
 
-        if (! $destinationButton) {
+        if (!$destinationButton) {
             return;
         }
 
@@ -201,7 +201,7 @@ 
         $lastSlice = $newButtons->slice($newDestinationKey, null);
 
         $newButtons = $firstSlice->push($targetButton);
-        $lastSlice->each(function ($item, $key) use ($newButtons) {
+        $lastSlice->each(function($item, $key) use ($newButtons) {
             $newButtons->push($item);
         });
 

src/app/Library/CrudPanel/Traits/Filters.php

Spacing +10 added lines, -10 removed lines

@@ -72,7 +72,7 @@ 
         $this->enableFilters();
 
         // check if another filter with the same name exists
-        if (! isset($options['name'])) {
+        if (!isset($options['name'])) {
             abort(500, 'All your filters need names.');
         }
 
@@ -120,7 +120,7 @@ 
     {
         $unappliedFilters = $this->filters()->where('applied', false);
         if ($unappliedFilters->count()) {
-            $unappliedFilters->each(function ($filter) {
+            $unappliedFilters->each(function($filter) {
                 $filter->apply();
             });
         }
@@ -170,7 +170,7 @@ 
     {
         $filter = $this->filters()->firstWhere('name', $name);
 
-        if (! $filter) {
+        if (!$filter) {
             abort(500, 'CRUD Filter "'.$name.'" not found. Please check the filter exists before you modify it.');
         }
 
@@ -185,7 +185,7 @@ 
 
     public function replaceFilter($name, $newFilter)
     {
-        $newFilters = $this->filters()->map(function ($filter, $key) use ($name, $newFilter) {
+        $newFilters = $this->filters()->map(function($filter, $key) use ($name, $newFilter) {
             if ($filter->name != $name) {
                 return $filter;
             }
@@ -198,7 +198,7 @@ 
 
     public function removeFilter($name)
     {
-        $strippedCollection = $this->filters()->reject(function ($filter) use ($name) {
+        $strippedCollection = $this->filters()->reject(function($filter) use ($name) {
             return $filter->name == $name;
         });
 
@@ -241,7 +241,7 @@ 
      */
     public function makeFirstFilter()
     {
-        if (! $this->filters()) {
+        if (!$this->filters()) {
             return false;
         }
 
@@ -271,15 +271,15 @@ 
         $destinationFilter = $this->firstFilterWhere('name', $destination);
         $destinationKey = $this->getFilterKey($destination);
         $newDestinationKey = ($where == 'before' ? $destinationKey : $destinationKey + 1);
-        $newFilters = $this->filters()->filter(function ($value, $key) use ($target) {
+        $newFilters = $this->filters()->filter(function($value, $key) use ($target) {
             return $value->name != $target;
         });
 
-        if (! $targetFilter) {
+        if (!$targetFilter) {
             return;
         }
 
-        if (! $destinationFilter) {
+        if (!$destinationFilter) {
             return;
         }
 
@@ -287,7 +287,7 @@ 
         $lastSlice = $newFilters->slice($newDestinationKey, null);
 
         $newFilters = $firstSlice->push($targetFilter);
-        $lastSlice->each(function ($item, $key) use ($newFilters) {
+        $lastSlice->each(function($item, $key) use ($newFilters) {
             $newFilters->push($item);
         });
 

src/routes/backpack/custom.php

Spacing +1 added lines, -1 removed lines

@@ -13,5 +13,5 @@
         (array) config('backpack.base.middleware_key', 'admin')
     ),
     'namespace'  => 'App\Http\Controllers\Admin',
-], function () { // custom admin routes
+], function() { // custom admin routes
 }); // this should be the absolute last line of this file

tests/config/database/migrations/2017_09_08_000001_create_addresses_table.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
      */
     public function up()
     {
-        Schema::create('addresses', function (Blueprint $table) {
+        Schema::create('addresses', function(Blueprint $table) {
             $table->increments('id');
             $table->integer('account_details_id')->unsigned();
             $table->string('city')->nullable();

tests/Unit/Http/CrudControllerTest.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 
         $app['router']->get('admin/users', "$controller@index")->name('admin.users.index');
 
-        $app->singleton('crud', function ($app) {
+        $app->singleton('crud', function($app) {
             return new CrudPanel($app);
         });