Laravel-Backpack / CRUD

tests/Unit/CrudPanel/CrudPanelReadTest.php

Indentation +4 added lines, -4 removed lines

@@ -438,10 +438,10 @@
         $this->assertTrue(in_array(-1, $this->crudPanel->getOperationSetting('pageLengthMenu')[0]));
     }
 
-     /**
-     * Tests if paginator aborts when 0 is provided as key.
-     *
-     */
+        /**
+         * Tests if paginator aborts when 0 is provided as key.
+         *
+         */
     public function testCrudPanelPaginatorWithZeroAsOption()
     {
         $this->crudPanel->setModel(User::class);

Spacing +1 added lines, -1 removed lines

@@ -429,7 +429,7 @@
         $this->assertCount(2, $this->crudPanel->getOperationSetting('pageLengthMenu'));
         $this->assertTrue(in_array(-1, $this->crudPanel->getOperationSetting('pageLengthMenu')[0]));
 
-        $this->crudPanel->setPageLengthMenu([-1 , 1]);
+        $this->crudPanel->setPageLengthMenu([-1, 1]);
         $this->assertCount(2, $this->crudPanel->getOperationSetting('pageLengthMenu'));
         $this->assertTrue(in_array(-1, $this->crudPanel->getOperationSetting('pageLengthMenu')[0]));
 

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

Spacing +18 added lines, -18 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();
         }
@@ -184,9 +184,9 @@ 
      */
     public function setDefaultPageLength($value)
     {
-        if($value !== 0) {
+        if ($value !== 0) {
             $this->setOperationSetting('defaultPageLength', $value);
-        }else{
+        } else {
             abort(500, 'You should not use 0 as a key in paginator. If you are looking for "ALL" option, use -1 instead.');
         }
     }
@@ -217,7 +217,7 @@ 
             if ($position !== false) {
                 array_unshift($values, $this->getDefaultPageLength());
                 array_unshift($labels, $labels[$position]);
-            }else{
+            } else {
                 // if it's not in array we add it as the first element
                 array_unshift($values, $this->getDefaultPageLength());
                 array_unshift($labels, $this->getDefaultPageLength());
@@ -239,27 +239,27 @@ 
      */
     public function setPageLengthMenu($menu)
     {
-        if(is_array($menu)) {
+        if (is_array($menu)) {
             // start checking $menu integrity
-            if(count($menu) !== count($menu, COUNT_RECURSIVE)) {
+            if (count($menu) !== count($menu, COUNT_RECURSIVE)) {
                 // developer defined as setPageLengthMenu([[50, 100, 300]]) or setPageLengthMenu([[50, 100, 300],['f','h','t']])
                 // we will apply the same labels as the values to the menu if developer didn't
-                if(!in_array(0, $menu[0])) {
-                    if (! isset($menu[1]) || ! is_array($menu[1])) {
+                if (!in_array(0, $menu[0])) {
+                    if (!isset($menu[1]) || !is_array($menu[1])) {
                         $menu[1] = $menu[0];
                     }
-                }else{
+                } else {
                     abort(500, 'You should not use 0 as a key in paginator. If you are looking for "ALL" option, use -1 instead.');
                 }
-            }else{
+            } else {
                 // developer defined setPageLengthMenu([10 => 'f', 100 => 'h', 300 => 't']) OR setPageLengthMenu([50, 100, 300])
                 $menu = $this->buildPageLengthMenuFromArray($menu);
             }
-        }else{
+        } else {
             // developer added only a single value setPageLengthMenu(10)
-            if($menu !== 0) {
+            if ($menu !== 0) {
                 $menu = [[$menu], [$menu]];
-            }else{
+            } else {
                 abort(500, 'You should not use 0 as a key in paginator. If you are looking for "ALL" option, use -1 instead.');
             }
         }
@@ -279,20 +279,20 @@ 
     private function buildPageLengthMenuFromArray($menu) {
         // check if the values of the array are strings, in case developer defined:
         // setPageLengthMenu([0 => 'f', 100 => 'h', 300 => 't'])
-        if(count(array_filter(array_values($menu), 'is_string')) > 0) {
+        if (count(array_filter(array_values($menu), 'is_string')) > 0) {
             $values = array_keys($menu);
             $labels = array_values($menu);
 
-            if(!in_array(0, $values)) {
+            if (!in_array(0, $values)) {
                 return [$values, $labels];
             }
 
             abort(500, 'You should not use 0 as a key in paginator. If you are looking for "ALL" option, use -1 instead.');
 
-        }else{
+        } else {
             // developer defined length as setPageLengthMenu([50, 100, 300])
             // we will use the same values as labels
-            if(!in_array(0, $menu)) {
+            if (!in_array(0, $menu)) {
                 return [$menu, $menu];
             }
 

Braces +7 added lines, -7 removed lines

@@ -186,7 +186,7 @@ 
     {
         if($value !== 0) {
             $this->setOperationSetting('defaultPageLength', $value);
-        }else{
+        } else{
             abort(500, 'You should not use 0 as a key in paginator. If you are looking for "ALL" option, use -1 instead.');
         }
     }
@@ -217,7 +217,7 @@ 
             if ($position !== false) {
                 array_unshift($values, $this->getDefaultPageLength());
                 array_unshift($labels, $labels[$position]);
-            }else{
+            } else{
                 // if it's not in array we add it as the first element
                 array_unshift($values, $this->getDefaultPageLength());
                 array_unshift($labels, $this->getDefaultPageLength());
@@ -248,18 +248,18 @@ 
                     if (! isset($menu[1]) || ! is_array($menu[1])) {
                         $menu[1] = $menu[0];
                     }
-                }else{
+                } else{
                     abort(500, 'You should not use 0 as a key in paginator. If you are looking for "ALL" option, use -1 instead.');
                 }
-            }else{
+            } else{
                 // developer defined setPageLengthMenu([10 => 'f', 100 => 'h', 300 => 't']) OR setPageLengthMenu([50, 100, 300])
                 $menu = $this->buildPageLengthMenuFromArray($menu);
             }
-        }else{
+        } else{
             // developer added only a single value setPageLengthMenu(10)
             if($menu !== 0) {
                 $menu = [[$menu], [$menu]];
-            }else{
+            } else{
                 abort(500, 'You should not use 0 as a key in paginator. If you are looking for "ALL" option, use -1 instead.');
             }
         }
@@ -289,7 +289,7 @@ 
 
             abort(500, 'You should not use 0 as a key in paginator. If you are looking for "ALL" option, use -1 instead.');
 
-        }else{
+        } else{
             // developer defined length as setPageLengthMenu([50, 100, 300])
             // we will use the same values as labels
             if(!in_array(0, $menu)) {