Laravel-Backpack / CRUD

src/config/backpack/base.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
     ],
 
     // CSS files that are loaded in all pages, using Laravel's mix() helper
-    'mix_styles' => [ // file_path => manifest_directory_path
+    'mix_styles' => [// file_path => manifest_directory_path
         // 'css/app.css' => '',
     ],
 

src/Stats.php

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

Doc Comments +2 added lines, -2 removed lines

@@ -8,7 +8,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()
     {
@@ -85,7 +85,7 @@ 
      * @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)
     {

src/app/Library/CrudPanel/Traits/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 +3 added lines, -3 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();
         }
@@ -203,7 +203,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->getOperationSetting('pageLengthMenu')[0])) {
+        if (!in_array($this->getDefaultPageLength(), $this->getOperationSetting('pageLengthMenu')[0])) {
             // Loop through 2 arrays of prop. pageLengthMenu
             foreach ($this->getOperationSetting('pageLengthMenu') as $key => &$page_length_choices) {
                 // This is a condition that should be always true.

src/app/Library/CrudPanel/Traits/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/BackpackServiceProvider.php

Spacing +5 added lines, -5 removed lines

@@ -54,13 +54,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 \Backpack\CRUD\app\Library\CrudPanel\CrudPanel($app);
         });
 
         // load a macro for Route,
         // helps developers load all routes for a CRUD resource in one line
-        if (! Route::hasMacro('crud')) {
+        if (!Route::hasMacro('crud')) {
             $this->addRouteMacro();
         }
 
@@ -81,7 +81,7 @@ 
         // }
 
         // 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');
         }
     }
@@ -91,7 +91,7 @@ 
         $middleware_key = config('backpack.base.middleware_key');
         $middleware_class = config('backpack.base.middleware_class');
 
-        if (! is_array($middleware_class)) {
+        if (!is_array($middleware_class)) {
             $router->pushMiddlewareToGroup($middleware_key, $middleware_class);
 
             return;
@@ -195,7 +195,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 = '';

src/LicenseCheck.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
             return;
         }
 
-        if (! $this->validCode(config('backpack.base.license_code'))) {
+        if (!$this->validCode(config('backpack.base.license_code'))) {
             \Alert::add('warning', "<strong>You're using unlicensed software.</strong> Please ask your web developer to <a target='_blank' class='alert-link text-white' href='http://backpackforlaravel.com'>purchase a license code</a> to hide this message.");
         }
     }

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

Doc Comments -1 removed lines

@@ -9,7 +9,6 @@
     /**
      * Define which routes are needed for this operation.
      *
-     * @param string $name       Name of the current entity (singular). Used as first URL segment.
      * @param string $routeName  Prefix of the route name.
      * @param string $controller Name of the current CrudController.
      */

Spacing +3 added lines, -3 removed lines

@@ -36,11 +36,11 @@ 
         $this->crud->set('reorder.enabled', true);
         $this->crud->allowAccess('reorder');
 
-        $this->crud->operation('reorder', function () {
+        $this->crud->operation('reorder', function() {
             $this->crud->loadDefaultOperationSettingsFromConfig();
         });
 
-        $this->crud->operation('list', function () {
+        $this->crud->operation('list', function() {
             $this->crud->addButton('top', 'reorder', 'view', 'crud::buttons.reorder');
         });
     }
@@ -56,7 +56,7 @@ 
     {
         $this->crud->hasAccessOrFail('reorder');
 
-        if (! $this->crud->isReorderEnabled()) {
+        if (!$this->crud->isReorderEnabled()) {
             abort(403, 'Reorder is disabled.');
         }
 

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

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 = 'crud::buttons.'.$view;
@@ -119,7 +122,7 @@ 
     }
 
     /**
-     * @param array       $names Button names
+     * @param string[]       $names Button names
      * @param string|null $stack Optional stack name.
      */
     public function removeButtons($names, $stack = null)
@@ -136,6 +139,9 @@ 
         $this->setOperationSetting('buttons', collect());
     }
 
+    /**
+     * @param string $stack
+     */
     public function removeAllButtonsFromStack($stack)
     {
         $this->setOperationSetting('buttons', $this->buttons()->reject(function ($button) use ($stack) {
@@ -158,6 +164,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;

Spacing +9 added lines, -9 removed lines

@@ -20,18 +20,18 @@ 
     {
         $newBtns = collect([]);
 
-        $this->buttons()->each(function ($btn) use ($stack, $newBtns) {
+        $this->buttons()->each(function($btn) use ($stack, $newBtns) {
             if ($btn->stack != $stack) {
                 $newBtns->push($btn);
             }
         });
 
-        collect($buttons)->each(function ($btnKey) use ($newBtns) {
-            $btnInstance = $this->buttons()->filter(function ($btn) use ($btnKey) {
+        collect($buttons)->each(function($btnKey) use ($newBtns) {
+            $btnInstance = $this->buttons()->filter(function($btn) use ($btnKey) {
                 return $btn->name == $btnKey;
             })->first();
             
-            if (! $btnInstance) {
+            if (!$btnInstance) {
                 abort(500, 'Sorry, button cannot be found');
             }
 
@@ -121,7 +121,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.');
         }
 
@@ -142,7 +142,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);
         }));
     }
@@ -153,7 +153,7 @@ 
      */
     public function removeButtons($names, $stack = null)
     {
-        if (! empty($names)) {
+        if (!empty($names)) {
             foreach ($names as $name) {
                 $this->removeButton($name, $stack);
             }
@@ -167,14 +167,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;
         }));
     }

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

Doc Comments +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
      *
      * TODO: should this delete items with relations to it too?
      *
-     * @return bool True if the item was deleted.
+     * @return string True if the item was deleted.
      */
     public function delete($id)
     {