Laravel-Backpack / CRUD

src/PanelTraits/Search.php

Indentation +4 added lines, -4 removed lines

@@ -119,10 +119,10 @@
         // add the details_row button to the first column
         if ($this->details_row) {
             $details_row_button = \View::make('crud::columns.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 +4 added lines, -4 removed lines

@@ -19,9 +19,9 @@ 
      */
     public function applySearchTerm($searchTerm)
     {
-        return $this->query->where(function ($query) use ($searchTerm) {
+        return $this->query->where(function($query) use ($searchTerm) {
             foreach ($this->getColumns() as $column) {
-                if (! isset($column['type'])) {
+                if (!isset($column['type'])) {
                     abort(400, 'Missing column type when trying to apply search term.');
                 }
 
@@ -60,7 +60,7 @@ 
 
                 case 'select':
                 case 'select_multiple':
-                    $query->orWhereHas($column['entity'], function ($q) use ($column, $searchTerm) {
+                    $query->orWhereHas($column['entity'], function($q) use ($column, $searchTerm) {
                         $q->where($column['attribute'], 'like', '%'.$searchTerm.'%');
                     });
                     break;
@@ -177,7 +177,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
         }
 

Doc Comments +2 added lines, -1 removed lines

@@ -35,6 +35,7 @@ 
 
     /**
      * Apply the search logic for each CRUD column.
+     * @param string $searchTerm
      */
     public function applySearchLogicForColumn($query, $column, $searchTerm)
     {
@@ -230,7 +231,7 @@ 
      *
      * @param string   $view
      * @param array    $column
-     * @param object   $entry
+     * @param \Illuminate\Database\Eloquent\Model   $entry
      * @param bool|int $rowNumber The number shown to the user as row number (index)
      *
      * @return string

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,8 +2,8 @@
 
 namespace Backpack\CRUD\PanelTraits;
 
-use Validator;
 use Carbon\Carbon;
+use Validator;
 
 trait Search
 {

src/PanelTraits/RequiredFields.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
      */
     public function isRequired($inputName, $operation)
     {
-        if (! isset($this->requiredFields[$operation])) {
+        if (!isset($this->requiredFields[$operation])) {
             return false;
         }
 

src/app/Http/Controllers/Operations/SaveActions.php

Spacing +3 added lines, -4 removed lines

@@ -22,11 +22,11 @@ 
 
         $saveOptions = collect($permissions)
             // Restrict list to allowed actions.
-            ->filter(function ($action, $permission) {
+            ->filter(function($action, $permission) {
                 return $this->crud->hasAccess($permission);
             })
             // Generate list of possible actions.
-            ->mapWithKeys(function ($action, $permission) {
+            ->mapWithKeys(function($action, $permission) {
                 return [$action => $this->getSaveActionButtonName($action)];
             })
             ->all();
@@ -61,8 +61,7 @@ 
      */
     public function setSaveAction($forceSaveAction = null)
     {
-        $saveAction = $forceSaveAction ?:
-            \Request::input('save_action', config('backpack.crud.default_save_action', 'save_and_back'));
+        $saveAction = $forceSaveAction ?: \Request::input('save_action', config('backpack.crud.default_save_action', 'save_and_back'));
 
         if (config('backpack.crud.show_save_action_change', true) &&
             session('save_action', 'save_and_back') !== $saveAction) {

src/PanelTraits/HeadingsAndTitle.php

Spacing +6 added lines, -6 removed lines

@@ -21,7 +21,7 @@ 
      */
     public function getTitle($action = false)
     {
-        if (! $action) {
+        if (!$action) {
             $action = $this->getActionMethod();
         }
 
@@ -38,7 +38,7 @@ 
      */
     public function setTitle($string, $action = false)
     {
-        if (! $action) {
+        if (!$action) {
             $action = $this->getActionMethod();
         }
 
@@ -58,7 +58,7 @@ 
      */
     public function getHeading($action = false)
     {
-        if (! $action) {
+        if (!$action) {
             $action = $this->getActionMethod();
         }
 
@@ -75,7 +75,7 @@ 
      */
     public function setHeading($string, $action = false)
     {
-        if (! $action) {
+        if (!$action) {
             $action = $this->getActionMethod();
         }
 
@@ -95,7 +95,7 @@ 
      */
     public function getSubheading($action = false)
     {
-        if (! $action) {
+        if (!$action) {
             $action = $this->getActionMethod();
         }
 
@@ -112,7 +112,7 @@ 
      */
     public function setSubheading($string, $action = false)
     {
-        if (! $action) {
+        if (!$action) {
             $action = $this->getActionMethod();
         }
 

src/CrudUsageStats.php

Doc Comments +2 added lines, -2 removed lines

@@ -10,7 +10,7 @@ 
      * Check if the application is running in normal conditions
      * (production env, not in console, not in unit tests).
      *
-     * @return void
+     * @return boolean
      */
     private function runningInProduction()
     {
@@ -86,7 +86,7 @@ 
      * @param  string $method  HTTP Method to use for the request.
      * @param  string $url     URL to point the request at.
      * @param  array $payload  The data you want sent to the URL.
-     * @return void
+     * @return boolean
      */
     private function makeCurlRequest($method, $url, $payload)
     {

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
     private function sendUsageStats()
     {
         // only send usage stats in production
-        if (! $this->runningInProduction()) {
+        if (!$this->runningInProduction()) {
             return;
         }
 

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

Spacing +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 <span>
     <?php
     $array = $entry->{$column['name']};
-    $suffix = isset($column['suffix'])?$column['suffix']:'items';
+    $suffix = isset($column['suffix']) ? $column['suffix'] : 'items';
 
     // the value should be an array wether or not attribute casting is used
     if (!is_array($array)) {

src/PanelTraits/Columns.php

Doc Comments +3 added lines, -3 removed lines

@@ -143,7 +143,7 @@ 
     /**
      * Move the most recently added column after the given target column.
      *
-     * @param string|array $targetColumn The target column name or array.
+     * @param string $targetColumn The target column name or array.
      */
     public function afterColumn($targetColumn)
     {
@@ -162,7 +162,7 @@ 
 
     /**
      * Move this column to be first in the columns list.
-     * @return bool|null
+     * @return false|null
      */
     public function makeFirstColumn()
     {
@@ -281,7 +281,7 @@ 
     /**
      * Change attributes for multiple columns.
      *
-     * @param array $columns
+     * @param string[] $columns
      * @param array $attributes
      */
     public function setColumnsDetails($columns, $attributes)

Spacing +16 added lines, -17 removed lines

@@ -68,7 +68,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];
         }
 
@@ -76,7 +76,7 @@ 
         $column_with_details = $this->addDefaultLabel($column);
 
         // make sure the column has a name
-        if (! array_key_exists('name', $column_with_details)) {
+        if (!array_key_exists('name', $column_with_details)) {
             $column_with_details['name'] = 'anonymous_column_'.str_random(5);
         }
 
@@ -84,27 +84,27 @@ 
         $columnExistsInDb = $this->hasColumn($this->model->getTable(), $column_with_details['name']);
 
         // make sure the column has a type
-        if (! array_key_exists('type', $column_with_details)) {
+        if (!array_key_exists('type', $column_with_details)) {
             $column_with_details['type'] = 'text';
         }
 
         // make sure the column has a key
-        if (! array_key_exists('key', $column_with_details)) {
+        if (!array_key_exists('key', $column_with_details)) {
             $column_with_details['key'] = $column_with_details['name'];
         }
 
         // make sure the column has a tableColumn boolean
-        if (! array_key_exists('tableColumn', $column_with_details)) {
+        if (!array_key_exists('tableColumn', $column_with_details)) {
             $column_with_details['tableColumn'] = $columnExistsInDb ? true : false;
         }
 
         // make sure the column has a orderable boolean
-        if (! array_key_exists('orderable', $column_with_details)) {
+        if (!array_key_exists('orderable', $column_with_details)) {
             $column_with_details['orderable'] = $columnExistsInDb ? true : false;
         }
 
         // make sure the column has a searchLogic
-        if (! array_key_exists('searchLogic', $column_with_details)) {
+        if (!array_key_exists('searchLogic', $column_with_details)) {
             $column_with_details['searchLogic'] = $columnExistsInDb ? true : false;
         }
 
@@ -112,14 +112,14 @@ 
 
         // make sure the column has a priority in terms of visibility
         // if no priority has been defined, use the order in the array plus one
-        if (! array_key_exists('priority', $column_with_details)) {
+        if (!array_key_exists('priority', $column_with_details)) {
             $position_in_columns_array = (int) array_search($column_with_details['key'], array_keys($this->columns));
             $this->columns[$column_with_details['key']]['priority'] = $position_in_columns_array + 1;
         }
 
         // if this is a relation type field and no corresponding model was specified, get it from the relation method
         // defined in the main model
-        if (isset($column_with_details['entity']) && ! isset($column_with_details['model'])) {
+        if (isset($column_with_details['entity']) && !isset($column_with_details['model'])) {
             $column_with_details['model'] = $this->getRelationModel($column_with_details['entity']);
         }
 
@@ -166,7 +166,7 @@ 
      */
     public function makeFirstColumn()
     {
-        if (! $this->columns) {
+        if (!$this->columns) {
             return false;
         }
 
@@ -187,8 +187,7 @@ 
         $targetColumnName = is_array($targetColumn) ? $targetColumn['name'] : $targetColumn;
 
         if (array_key_exists($targetColumnName, $this->columns)) {
-            $targetColumnPosition = $before ? array_search($targetColumnName, array_keys($this->columns)) :
-                array_search($targetColumnName, array_keys($this->columns)) + 1;
+            $targetColumnPosition = $before ? array_search($targetColumnName, array_keys($this->columns)) : array_search($targetColumnName, array_keys($this->columns)) + 1;
 
             $element = array_pop($this->columns);
             $beginningPart = array_slice($this->columns, 0, $targetColumnPosition, true);
@@ -225,7 +224,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' => mb_ucfirst($this->makeLabel($array['name']))], $array);
 
             return $array;
@@ -251,7 +250,7 @@ 
      */
     public function removeColumns($columns)
     {
-        if (! empty($columns)) {
+        if (!empty($columns)) {
             foreach ($columns as $columnName) {
                 $this->removeColumn($columnName);
             }
@@ -273,8 +272,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);
         }));
     }
 
@@ -331,7 +330,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/Buttons.php

Spacing +4 added lines, -4 removed lines

@@ -107,7 +107,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.');
         }
 
@@ -128,7 +128,7 @@ 
      */
     public function removeButton($name, $stack = null)
     {
-        $this->buttons = $this->buttons->reject(function ($button) use ($name, $stack) {
+        $this->buttons = $this->buttons->reject(function($button) use ($name, $stack) {
             return $stack == null ? $button->name == $name : ($button->stack == $stack) && ($button->name == $name);
         });
     }
@@ -140,14 +140,14 @@ 
 
     public function removeAllButtonsFromStack($stack)
     {
-        $this->buttons = $this->buttons->reject(function ($button) use ($stack) {
+        $this->buttons = $this->buttons->reject(function($button) use ($stack) {
             return $button->stack == $stack;
         });
     }
 
     public function removeButtonFromStack($name, $stack)
     {
-        $this->buttons = $this->buttons->reject(function ($button) use ($name, $stack) {
+        $this->buttons = $this->buttons->reject(function($button) use ($name, $stack) {
             return $button->name == $name && $button->stack == $stack;
         });
     }

Doc Comments +13 added lines, -1 removed lines

@@ -62,6 +62,9 @@ 
         $this->addButton($stack, $name, 'model_function', $model_function_name, $position);
     }
 
+    /**
+     * @param string $view
+     */
     public function addButtonFromView($stack, $name, $view, $position = false)
     {
         $view = 'vendor.backpack.crud.buttons.'.$view;
@@ -98,7 +101,7 @@ 
      * @param  string $name          The button name.
      * @param  array  $modifications The attributes and their new values.
      *
-     * @return CrudButton                The button that has suffered the changes, for daisychaining methods.
+     * @return Collection                The button that has suffered the changes, for daisychaining methods.
      */
     public function modifyButton($name, $modifications = null)
     {
@@ -138,6 +141,9 @@ 
         $this->buttons = collect([]);
     }
 
+    /**
+     * @param string $stack
+     */
     public function removeAllButtonsFromStack($stack)
     {
         $this->buttons = $this->buttons->reject(function ($button) use ($stack) {
@@ -160,6 +166,12 @@ 
     public $type = 'view';
     public $content;
 
+    /**
+     * @param string $stack
+     * @param string $name
+     * @param string $type
+     * @param string $content
+     */
     public function __construct($stack, $name, $type, $content)
     {
         $this->stack = $stack;

src/PanelTraits/Tabs.php

Spacing +7 added lines, -7 removed lines

@@ -35,7 +35,7 @@ 
      */
     public function tabsDisabled()
     {
-        return ! $this->tabsEnabled;
+        return !$this->tabsEnabled;
     }
 
     public function setTabsType($type)
@@ -116,8 +116,8 @@ 
     {
         $all_fields = $this->getCurrentFields();
 
-        $fields_without_a_tab = collect($all_fields)->filter(function ($value, $key) {
-            return ! isset($value['tab']);
+        $fields_without_a_tab = collect($all_fields)->filter(function($value, $key) {
+            return !isset($value['tab']);
         });
 
         return $fields_without_a_tab;
@@ -133,7 +133,7 @@ 
         if ($this->tabExists($label)) {
             $all_fields = $this->getCurrentFields();
 
-            $fields_for_current_tab = collect($all_fields)->filter(function ($value, $key) use ($label) {
+            $fields_for_current_tab = collect($all_fields)->filter(function($value, $key) use ($label) {
                 return isset($value['tab']) && $value['tab'] == $label;
             });
 
@@ -152,11 +152,11 @@ 
         $fields = $this->getCurrentFields();
 
         $fields_with_tabs = collect($fields)
-            ->filter(function ($value, $key) {
+            ->filter(function($value, $key) {
                 return isset($value['tab']);
             })
-            ->each(function ($value, $key) use (&$tabs) {
-                if (! in_array($value['tab'], $tabs)) {
+            ->each(function($value, $key) use (&$tabs) {
+                if (!in_array($value['tab'], $tabs)) {
                     $tabs[] = $value['tab'];
                 }
             });