Laravel-Backpack / CRUD

src/CrudServiceProvider.php

Spacing +5 added lines, -5 removed lines

@@ -86,13 +86,13 @@ 
     public function register()
     {
         // Bind the CrudPanel object to Laravel's service container
-        $this->app->singleton('crud', function ($app) {
+        $this->app->singleton('crud', function($app) {
             return new CrudPanel($app);
         });
 
         // load a macro for Route,
         // for developers to be able to load all routes for a CRUD resource in one line
-        if (! Route::hasMacro('crud')) {
+        if (!Route::hasMacro('crud')) {
             $this->addRouteMacro();
         }
 
@@ -103,7 +103,7 @@ 
         $this->commands($this->commands);
 
         // map the elfinder prefix
-        if (! \Config::get('elfinder.route.prefix')) {
+        if (!\Config::get('elfinder.route.prefix')) {
             \Config::set('elfinder.route.prefix', \Config::get('backpack.base.route_prefix').'/elfinder');
         }
     }
@@ -116,7 +116,7 @@ 
      */
     private function addRouteMacro()
     {
-        Route::macro('crud', function ($name, $controller) {
+        Route::macro('crud', function($name, $controller) {
             // put together the route name prefix,
             // as passed to the Route::group() statements
             $routeName = '';
@@ -162,7 +162,7 @@ 
     {
         $crudPubPath = public_path('vendor/backpack/crud');
 
-        if (! is_dir($crudPubPath)) {
+        if (!is_dir($crudPubPath)) {
             return true;
         }
 

src/PanelTraits/SaveActions.php

Spacing +3 added lines, -4 removed lines

@@ -24,11 +24,11 @@ 
 
         $saveOptions = collect($permissions)
             // Restrict list to allowed actions.
-            ->filter(function ($action, $permission) {
+            ->filter(function($action, $permission) {
                 return $this->hasAccess($permission);
             })
             // Generate list of possible actions.
-            ->mapWithKeys(function ($action, $permission) {
+            ->mapWithKeys(function($action, $permission) {
                 return [$action => $this->getSaveActionButtonName($action)];
             })
             ->all();
@@ -63,8 +63,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/Views.php

Doc Comments +6 added lines

@@ -301,6 +301,9 @@ 
         return $this->getShowView();
     }
 
+    /**
+     * @param string $view
+     */
     public function setPreviewView($view)
     {
         return $this->setShowView($view);
@@ -311,6 +314,9 @@ 
         return $this->getEditView();
     }
 
+    /**
+     * @param string $view
+     */
     public function setUpdateView($view)
     {
         return $this->setEditView($view);

src/CrudPanel.php

Spacing +11 added lines, -11 removed lines

@@ -76,11 +76,11 @@ 
      */
     public function setModel($model_namespace)
     {
-        if (! class_exists($model_namespace)) {
+        if (!class_exists($model_namespace)) {
             throw new \Exception('The model does not exist.', 500);
         }
 
-        if (! method_exists($model_namespace, 'hasCrudTrait')) {
+        if (!method_exists($model_namespace, 'hasCrudTrait')) {
             throw new \Exception('Please use CrudTrait on the model.', 500);
         }
 
@@ -143,7 +143,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);
         }
 
@@ -244,7 +244,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;
         });
     }
@@ -261,8 +261,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);
                 }
@@ -294,15 +294,15 @@ 
     {
         $relationArray = explode('.', $relationString);
 
-        if (! isset($length)) {
+        if (!isset($length)) {
             $length = count($relationArray);
         }
 
-        if (! isset($model)) {
+        if (!isset($model)) {
             $model = $this->model;
         }
 
-        $result = array_reduce(array_splice($relationArray, 0, $length), function ($obj, $method) {
+        $result = array_reduce(array_splice($relationArray, 0, $length), function($obj, $method) {
             return $obj->$method()->getRelated();
         }, $model);
 
@@ -352,7 +352,7 @@ 
         $relation = $model->{$firstRelationName};
 
         $results = [];
-        if (! empty($relation)) {
+        if (!empty($relation)) {
             if ($relation instanceof Collection) {
                 $currentResults = $relation->toArray();
             } else {
@@ -361,7 +361,7 @@ 
 
             array_shift($relationArray);
 
-            if (! empty($relationArray)) {
+            if (!empty($relationArray)) {
                 foreach ($currentResults as $currentResult) {
                     $results = array_merge($results, $this->getRelationModelInstances($currentResult, implode('.', $relationArray)));
                 }

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

@@ -2,31 +2,31 @@
 
 namespace Backpack\CRUD;
 
-use Backpack\CRUD\PanelTraits\Read;
-use Backpack\CRUD\PanelTraits\Tabs;
-use Backpack\CRUD\PanelTraits\Query;
-use Backpack\CRUD\PanelTraits\Views;
 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;
 use Backpack\CRUD\PanelTraits\Create;
 use Backpack\CRUD\PanelTraits\Delete;
 use Backpack\CRUD\PanelTraits\Errors;
+use Backpack\CRUD\PanelTraits\FakeColumns;
+use Backpack\CRUD\PanelTraits\FakeFields;
 use Backpack\CRUD\PanelTraits\Fields;
-use Backpack\CRUD\PanelTraits\Search;
-use Backpack\CRUD\PanelTraits\Update;
-use Backpack\CRUD\PanelTraits\AutoSet;
-use Backpack\CRUD\PanelTraits\Buttons;
-use Backpack\CRUD\PanelTraits\Columns;
 use Backpack\CRUD\PanelTraits\Filters;
-use Backpack\CRUD\PanelTraits\Reorder;
-use Backpack\CRUD\PanelTraits\AutoFocus;
+use Backpack\CRUD\PanelTraits\HeadingsAndTitles;
 use Backpack\CRUD\PanelTraits\Macroable;
-use Backpack\CRUD\PanelTraits\FakeFields;
 use Backpack\CRUD\PanelTraits\Operations;
-use Backpack\CRUD\PanelTraits\FakeColumns;
+use Backpack\CRUD\PanelTraits\Query;
+use Backpack\CRUD\PanelTraits\Read;
+use Backpack\CRUD\PanelTraits\Reorder;
+use Backpack\CRUD\PanelTraits\RequiredFields;
 use Backpack\CRUD\PanelTraits\SaveActions;
+use Backpack\CRUD\PanelTraits\Search;
 use Backpack\CRUD\PanelTraits\Settings;
-use Backpack\CRUD\PanelTraits\RequiredFields;
-use Backpack\CRUD\PanelTraits\HeadingsAndTitles;
+use Backpack\CRUD\PanelTraits\Tabs;
+use Backpack\CRUD\PanelTraits\Update;
+use Backpack\CRUD\PanelTraits\Views;
 use Backpack\CRUD\PanelTraits\ViewsAndRestoresRevisions;
 use Illuminate\Database\Eloquent\Collection;
 

src/PanelTraits/Read.php

Doc Comments +1 added lines

@@ -175,6 +175,7 @@
 
     /**
      * Set the number of rows that should be show on the list view.
+     * @param integer $value
      */
     public function setDefaultPageLength($value)
     {

Spacing +4 added lines, -4 removed lines

@@ -38,7 +38,7 @@ 
     {
         $id = $this->getCurrentEntryId();
 
-        if (! $id) {
+        if (!$id) {
             return false;
         }
 
@@ -54,7 +54,7 @@ 
      */
     public function getEntry($id)
     {
-        if (! $this->entry) {
+        if (!$this->entry) {
             $this->entry = $this->model->findOrFail($id);
             $this->entry = $this->entry->withFakes();
         }
@@ -199,7 +199,7 @@ 
     {
         // If the default Page Length isn't in the menu's values, Add it the beginnin and resort all to show a croissant list.
         // assume both arrays are the same length.
-        if (! in_array($this->getDefaultPageLength(), $this->get('list.pageLengthMenu')[0])) {
+        if (!in_array($this->getDefaultPageLength(), $this->get('list.pageLengthMenu')[0])) {
             // Loop through 2 arrays of prop. page_length_menu
             foreach ($this->get('list.pageLengthMenu') as $key => &$page_length_choices) {
                 // This is a condition that should be always true.
@@ -230,7 +230,7 @@ 
     public function getPageLengthMenu()
     {
         // if already set, use that
-        if (! $this->get('list.pageLengthMenu')) {
+        if (!$this->get('list.pageLengthMenu')) {
             // try to get the menu settings from the config file
             $this->set('list.pageLengthMenu', config('backpack.crud.page_length_menu') ?? [[10, 25, 50, 100, -1], [10, 25, 50, 100, 'backpack::crud.all']]);
 

src/PanelTraits/RequiredFields.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
      */
     public function isRequired($inputName, $operation)
     {
-        if (! $this->has($operation.'.requiredFields')) {
+        if (!$this->has($operation.'.requiredFields')) {
             return false;
         }
 

src/PanelTraits/HeadingsAndTitles.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();
         }