Laravel-Backpack / CRUD

src/ModelTraits/SpatieTranslatable/HasTranslations.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
      */
     public function getAttributeValue($key)
     {
-        if (! $this->isTranslatableAttribute($key)) {
+        if (!$this->isTranslatableAttribute($key)) {
             return parent::getAttributeValue($key);
         }
 
@@ -72,7 +72,7 @@ 
      */
     public function update(array $attributes = [], array $options = [])
     {
-        if (! $this->exists) {
+        if (!$this->exists) {
             return false;
         }
 

src/ModelTraits/SpatieTranslatable/Sluggable.php

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

@@ -2,9 +2,9 @@
 
 namespace Backpack\CRUD\ModelTraits\SpatieTranslatable;
 
-use Illuminate\Database\Eloquent\Model;
-use Illuminate\Database\Eloquent\Builder;
 use Cviebrock\EloquentSluggable\Sluggable as OriginalSluggable;
+use Illuminate\Database\Eloquent\Builder;
+use Illuminate\Database\Eloquent\Model;
 
 trait Sluggable
 {

Doc Comments -1 removed lines

@@ -37,7 +37,6 @@
      * Query scope for finding "similar" slugs, used to determine uniqueness.
      *
      * @param \Illuminate\Database\Eloquent\Builder $query
-     * @param \Illuminate\Database\Eloquent\Model $model
      * @param string $attribute
      * @param array $config
      * @param string $slug

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
         $separator = $config['separator'];
         $attribute = $attribute.'->'.$this->getLocale();
 
-        return $query->where(function (Builder $q) use ($attribute, $slug, $separator) {
+        return $query->where(function(Builder $q) use ($attribute, $slug, $separator) {
             $q->where($attribute, '=', $slug)
                 ->orWhere($attribute, 'LIKE', $slug.$separator.'%')
                 // Fixes issues with Json data types in MySQL where data is sourrounded by "

src/CrudServiceProvider.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace Backpack\CRUD;
 
-use Route;
 use Illuminate\Support\ServiceProvider;
 
 class CrudServiceProvider extends ServiceProvider

Spacing +3 added lines, -3 removed lines

@@ -76,7 +76,7 @@ 
      */
     public function register()
     {
-        $this->app->bind('CRUD', function ($app) {
+        $this->app->bind('CRUD', function($app) {
             return new CRUD($app);
         });
 
@@ -91,7 +91,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');
         }
     }
@@ -111,7 +111,7 @@ 
     {
         $crudPubPath = public_path('vendor/backpack/crud');
 
-        if (! is_dir($crudPubPath)) {
+        if (!is_dir($crudPubPath)) {
             return true;
         }
 

src/resources/views/fields/checklist_dependency.blade.php

Indentation +4 added lines, -4 removed lines

@@ -23,13 +23,13 @@
             }
         }
 
-      //for update form, get initial state of the entity
-      if( isset($id) && $id ){
+        //for update form, get initial state of the entity
+        if( isset($id) && $id ){
 
         //get entity with relations for primary dependency
         $entity_dependencies = $entity_model->with($primary_dependency['entity'])
-          ->with($primary_dependency['entity'].'.'.$primary_dependency['entity_secondary'])
-          ->find($id);
+            ->with($primary_dependency['entity'].'.'.$primary_dependency['entity_secondary'])
+            ->find($id);
 
             $secondaries_from_primary = [];
 

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
         }
 
       //for update form, get initial state of the entity
-      if( isset($id) && $id ){
+      if (isset($id) && $id) {
 
         //get entity with relations for primary dependency
         $entity_dependencies = $entity_model->with($primary_dependency['entity'])

src/resources/lang/pt/crud.php

Indentation +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
     'delete_confirmation_not_deleted_title'       => 'Não apagado',
     'delete_confirmation_not_deleted_message'     => 'Está tudo bem!. O item não foi apagado.',
 
-     // DataTables translation
+        // DataTables translation
     'emptyTable'     => 'Sem dados disponíveis na tabela',
     'info'           => 'A mostrar _START_ a _END_ de _TOTAL_ registos',
     'infoEmpty'      => 'A mostrar 0 a 0 de 0 registos',

src/PanelTraits/FakeColumns.php

Spacing +3 added lines, -3 removed lines

@@ -24,19 +24,19 @@
             if (isset($field['fake']) && $field['fake'] == true) {
                 // add it to the request in its appropriate variable - the one defined, if defined
                 if (isset($field['store_in'])) {
-                    if (! in_array($field['store_in'], $fakeFieldsColumnsArray, true)) {
+                    if (!in_array($field['store_in'], $fakeFieldsColumnsArray, true)) {
                         array_push($fakeFieldsColumnsArray, $field['store_in']);
                     }
                 } else {
                     //otherwise in the one defined in the $crud variable
-                    if (! in_array('extras', $fakeFieldsColumnsArray, true)) {
+                    if (!in_array('extras', $fakeFieldsColumnsArray, true)) {
                         array_push($fakeFieldsColumnsArray, 'extras');
                     }
                 }
             }
         }
 
-        if (! count($fakeFieldsColumnsArray)) {
+        if (!count($fakeFieldsColumnsArray)) {
             $fakeFieldsColumnsArray = ['extras'];
         }
 

Doc Comments +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
      * been found.
      *
      * @param string   $form The CRUD form. Can be 'create', 'update' or 'both'. Default is 'create'.
-     * @param int|bool $id   Optional entity ID needed in the case of the update form.
+     * @param integer $id   Optional entity ID needed in the case of the update form.
      *
      * @return array The fake columns array.
      */

src/PanelTraits/Delete.php

Doc Comments +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
      *
      * @param  int $id The id of the item to be deleted.
      *
-     * @return bool True if the item was deleted.
+     * @return string True if the item was deleted.
      *
      * @throws \Illuminate\Database\Eloquent\ModelNotFoundException if the model was not found.
      *

src/PanelTraits/Access.php

Doc Comments +6 added lines

@@ -12,12 +12,18 @@
     |--------------------------------------------------------------------------
     */
 
+    /**
+     * @param string $access
+     */
     public function allowAccess($access)
     {
         // $this->addButtons((array)$access);
         return $this->access = array_merge(array_diff((array) $access, $this->access), $this->access);
     }
 
+    /**
+     * @param string $access
+     */
     public function denyAccess($access)
     {
         // $this->removeButtons((array)$access);

Spacing +3 added lines, -3 removed lines

@@ -33,7 +33,7 @@ 
      */
     public function hasAccess($permission)
     {
-        if (! in_array($permission, $this->access)) {
+        if (!in_array($permission, $this->access)) {
             return false;
         }
 
@@ -68,7 +68,7 @@ 
     public function hasAccessToAll($permission_array)
     {
         foreach ($permission_array as $key => $permission) {
-            if (! in_array($permission, $this->access)) {
+            if (!in_array($permission, $this->access)) {
                 return false;
             }
         }
@@ -86,7 +86,7 @@ 
      */
     public function hasAccessOrFail($permission)
     {
-        if (! in_array($permission, $this->access)) {
+        if (!in_array($permission, $this->access)) {
             throw new AccessDeniedException(trans('backpack::crud.unauthorized_access', ['access' => $permission]));
         }
 

tests/Unit/CrudPanel/CrudPanelCreateTest.php

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

@@ -2,9 +2,9 @@
 
 namespace Backpack\CRUD\Tests\Unit\CrudPanel;
 
-use Faker\Factory;
-use Backpack\CRUD\Tests\Unit\Models\User;
 use Backpack\CRUD\Tests\Unit\Models\Article;
+use Backpack\CRUD\Tests\Unit\Models\User;
+use Faker\Factory;
 
 class CrudPanelCreateTest extends BaseDBCrudPanelTest
 {