Fix styling

Author: lrljoe

src/Views/Columns/LivewireComponentColumn.php

@@ -18,24 +18,19 @@ class LivewireComponentColumn extends Column
 
     /**
      * The Livewire Component assigned to this Column
-     *
-     * @var string|null
      */
     protected ?string $livewireComponent;
 
     /**
      * Gets the contents for current row
-     *
-     * @param Model $row
-     * @return null|string|HtmlString|DataTableConfigurationException|\Illuminate\Contracts\Foundation\Application|\Illuminate\Contracts\View\Factory|\Illuminate\Contracts\View\View
      */
     public function getContents(Model $row): null|string|HtmlString|DataTableConfigurationException|\Illuminate\Contracts\Foundation\Application|\Illuminate\Contracts\View\Factory|\Illuminate\Contracts\View\View
     {
         $this->runPreChecks();
-        
+
         $attributes = $this->retrieveAttributes($row);
 
-        return $this->getHtmlString($attributes, $this->getTable()."-".$row->{$row->getKeyName()});
+        return $this->getHtmlString($attributes, $this->getTable().'-'.$row->{$row->getKeyName()});
 
     }
 }

src/Views/Columns/Traits/Configuration/LivewireComponentColumnConfiguration.php

@@ -6,12 +6,8 @@
 
 trait LivewireComponentColumnConfiguration
 {
-
     /**
      * Defines which component to use
-     *
-     * @param string $livewireComponent
-     * @return self
      */
     public function component(string $livewireComponent): self
     {

src/Views/Columns/Traits/Helpers/LivewireComponentColumnHelpers.php

@@ -2,40 +2,31 @@
 
 namespace Rappasoft\LaravelLivewireTables\Views\Columns\Traits\Helpers;
 
-use Rappasoft\LaravelLivewireTables\Exceptions\DataTableConfigurationException;
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Support\Facades\Blade;
 use Illuminate\Support\HtmlString;
+use Rappasoft\LaravelLivewireTables\Exceptions\DataTableConfigurationException;
 
 trait LivewireComponentColumnHelpers
 {
-
     /**
      * Retrieves the defined Component View
-    *
-    * @return string|null
-    */
+     */
     public function getLivewireComponent(): ?string
     {
         return $this->livewireComponent ?? null;
     }
 
-
     /**
      * Determines whether a Livewire Component has been set
-    *
-    * @return boolean
-    */
+     */
     public function hasLivewireComponent(): bool
     {
         return isset($this->livewireComponent);
     }
 
     /**
      * Retrieves attributes based on callback
-     *
-     * @param Model $row
-     * @return array
      */
     protected function retrieveAttributes(Model $row): array
     {
@@ -50,48 +41,43 @@ protected function retrieveAttributes(Model $row): array
                 throw new DataTableConfigurationException('The return type of callback must be an array');
             }
         }
+
         return $attributes;
     }
 
     /**
      * Runs pre-checks
-     * 
-     * @return boolean
      */
     protected function runPreChecks(): bool
     {
         if (! $this->hasLivewireComponent()) {
             throw new DataTableConfigurationException('You must define a Livewire Component for this column');
+
             return false;
         }
 
         if ($this->isLabel()) {
             throw new DataTableConfigurationException('You can not use a label column with a Livewire Component column');
+
             return false;
         }
 
         return true;
     }
 
     /**
-      * Implodes defined attributes to be used
-      *
-      * @param array $attributes
-      * @return string
-      */
+     * Implodes defined attributes to be used
+     */
     protected function implodeAttributes(array $attributes): string
     {
         return collect($attributes)->map(function ($value, $key) {
             return ':'.$key.'="$'.$key.'"';
         })->implode(' ');
     }
 
-     /**
-      * getBlade Render
-      *
-      * @param array $attributes
-      * @param string $key
-      */
+    /**
+     * getBlade Render
+     */
     protected function getBlade(array $attributes, string $key)
     {
         return Blade::render(
@@ -106,17 +92,10 @@ protected function getBlade(array $attributes, string $key)
 
     /**
      * Gets HTML STring
-     *
-     * @param array $attributes
-     * @param string $key
-     * @return HtmlString
      */
-     protected function getHtmlString(array $attributes, string $key): HtmlString
+    protected function getHtmlString(array $attributes, string $key): HtmlString
     {
         return new HtmlString($this->getBlade($attributes, $key));
 
     }
-
-
-
 }

tests/Http/Livewire/TestComponent.php

@@ -1,27 +1,27 @@
-<?php
-
-namespace Rappasoft\LaravelLivewireTables\Tests\Http;
-
-use Closure;
-use Illuminate\Contracts\View\View;
-use Illuminate\View\Component;
-
-class TestComponent extends Component
-{
-    public int $testItem = 0;
-
-    public function __construct(int $age)
-    {
-        $this->testItem = $age * 110;
-    }
-
-    /**
-     * Get the view / contents that represent the component.
-     */
-    public function render(): View|Closure|string
-    {
-        return \Illuminate\Support\Facades\Blade::render(
-            '<div>'.($this->testItem ?? 'Unknown').'</div>');
-
-    }
-}
+<?php
+
+namespace Rappasoft\LaravelLivewireTables\Tests\Http;
+
+use Closure;
+use Illuminate\Contracts\View\View;
+use Illuminate\View\Component;
+
+class TestComponent extends Component
+{
+    public int $testItem = 0;
+
+    public function __construct(int $age)
+    {
+        $this->testItem = $age * 110;
+    }
+
+    /**
+     * Get the view / contents that represent the component.
+     */
+    public function render(): View|Closure|string
+    {
+        return \Illuminate\Support\Facades\Blade::render(
+            '<div>'.($this->testItem ?? 'Unknown').'</div>');
+
+    }
+}

tests/Unit/Views/Columns/LivewireComponentColumnTest.php

@@ -2,12 +2,12 @@
 
 namespace Rappasoft\LaravelLivewireTables\Tests\Unit\Views\Columns;
 
+use Illuminate\Database\Eloquent\Model;
 use PHPUnit\Framework\Attributes\Group;
 use Rappasoft\LaravelLivewireTables\Exceptions\DataTableConfigurationException;
 use Rappasoft\LaravelLivewireTables\Tests\Models\Pet;
 use Rappasoft\LaravelLivewireTables\Views\Column;
 use Rappasoft\LaravelLivewireTables\Views\Columns\LivewireComponentColumn;
-use Illuminate\Database\Eloquent\Model;
 
 #[Group('Columns')]
 final class LivewireComponentColumnTest extends ColumnTestCase
@@ -97,9 +97,8 @@ public static function setup_with_public_methods()
 
         $row = Pet::find(1);
 
-        $temp = (new class("name", "name") extends LivewireComponentColumn
+        $temp = (new class('name', 'name') extends LivewireComponentColumn
         {
-
             public function pubRetrieveAttributes(Model $row)
             {
                 return $this->retrieveAttributes($row);
@@ -112,17 +111,16 @@ public function pubImplodeAttributes(array $attributes)
 
             public function pubGetBlade(array $attributes, string $key)
             {
-                return $this->getBlade($attributes,$key);
+                return $this->getBlade($attributes, $key);
             }
 
             public function pubGetHtmlString(array $attributes, string $key)
             {
-                return $this->getHtmlString($attributes,$key);
+                return $this->getHtmlString($attributes, $key);
             }
-
         })->component('test-component')->attributes(function ($columnValue, $row) {
             return [
-                'type' => "test",
+                'type' => 'test',
                 'name' => $row->name,
             ];
         });
@@ -132,12 +130,11 @@ public function pubGetHtmlString(array $attributes, string $key)
         return $temp;
     }
 
-
     public function test_can_get_attributes_correctly(): void
     {
         $row = Pet::find(1);
         $temp = self::setup_with_public_methods();
-        $key = "test-table-".$row->{$row->getKeyName()};
+        $key = 'test-table-'.$row->{$row->getKeyName()};
 
         $this->assertSame(['type' => 'test', 'name' => 'Cartman'], $temp->pubRetrieveAttributes($row));
 
@@ -148,21 +145,19 @@ public function test_can_get_blade_correctly(): void
     {
         $row = Pet::find(1);
         $temp = self::setup_with_public_methods();
-        $key = "test-table-".$row->{$row->getKeyName()};
+        $key = 'test-table-'.$row->{$row->getKeyName()};
+
+        $this->assertStringContainsString('wire:snapshot="{"data":{"id":null,"name":"Cartman","value":null,"type":"test"}', $temp->pubGetBlade($temp->pubRetrieveAttributes($row), $key));
 
-        $this->assertStringContainsString('wire:snapshot="{"data":{"id":null,"name":"Cartman","value":null,"type":"test"}', $temp->pubGetBlade($temp->pubRetrieveAttributes($row),$key));
-        
-        $this->assertStringContainsString('<div>Name:Cartman</div><div>Type:test</div>', $temp->pubGetBlade($temp->pubRetrieveAttributes($row),$key));
+        $this->assertStringContainsString('<div>Name:Cartman</div><div>Type:test</div>', $temp->pubGetBlade($temp->pubRetrieveAttributes($row), $key));
     }
 
     public function test_can_get_html_string_correctly(): void
     {
         $row = Pet::find(1);
         $temp = self::setup_with_public_methods();
-        $key = "test-table-".$row->{$row->getKeyName()};
+        $key = 'test-table-'.$row->{$row->getKeyName()};
 
-        $this->assertStringContainsString('<div>Name:Cartman</div><div>Type:test</div>', $temp->pubGetHtmlString($temp->pubRetrieveAttributes($row),$key));
+        $this->assertStringContainsString('<div>Name:Cartman</div><div>Type:test</div>', $temp->pubGetHtmlString($temp->pubRetrieveAttributes($row), $key));
     }
-
 }
-