Fix options to work on re-render

Author: rappasoft

CHANGELOG.md

@@ -12,9 +12,9 @@ All notable changes to `laravel-livewire-tables` will be documented in this file
 
 - Config file to choose frontend framework - currently limited to bootstrap
 - Render method to columns which returns whatever you put into it, you can return a view, html, an attribute, etc.
-- Pulled in and modified the HTML component library from laravelcollective so you an return html components from the render method. i.e.: $this->image(...);
+- Pulled in and modified the HTML component library from laravelcollective so you can return html components from the render method. i.e.: $this->image(...);
 - Added new loading config on whether to keep displaying the current data while loading or collapse it
-- Added ability to set frontend framework specific options via the mount method on a per component basis.
+- Added ability to set frontend framework specific options via a property on a per component basis.
 
 ### Changed
 

README.md

@@ -288,27 +288,24 @@ public function html($html): HtmlString
 
 There are some frontend framework specific options that can be set.
 
-These have to be set from the `mount()` method of your component.
+These have to be set from the `$options` property of your component.
 
 They are done this way instead of the config file that way you can have per-component control over these settings.
 
 ```php
-public function mount()
-{
-    $this->setOptions([
-        // The class set on the table when using bootstrap
-        'bootstrap.classes.table' => 'table table-striped table-bordered',
-        
-        // Whether or not the table is wrapped in a `.container-fluid` or not
-        'bootstrap.container' => true,
+protected $options = [
+    // The class set on the table when using bootstrap
+    'bootstrap.classes.table' => 'table table-striped table-bordered',
 
-        // Whether or not the table is wrapped in a `.table-responsive` or not
-        'bootstrap.responsive' => true,
-    ]);
-}
+    // Whether or not the table is wrapped in a `.container-fluid` or not
+    'bootstrap.container' => true,
+    
+    // Whether or not the table is wrapped in a `.table-responsive` or not
+    'bootstrap.responsive' => true,
+];
 ```
 
-For this to work you have to pass an associative array of overrides to the `setOptions()` method. The above are the defaults, if you're not changing them then you can leave them out.
+For this to work you have to pass an associative array of overrides to the `$options` property. The above are the defaults, if you're not changing them then you can leave them out or disregard the property all together.
 
 ### Passing Properties
 

src/TableComponent.php

@@ -64,7 +64,7 @@ public function __construct($id = null)
             $this->paginationTheme = 'bootstrap';
         }
 
-        $this->setOptions();
+        $this->setOptions($this->options);
 
         parent::__construct($id);
     }

src/Traits/Options.php

@@ -9,10 +9,16 @@
  */
 trait Options
 {
+
+    /**
+     * @var array
+     */
+    protected $options = [];
+
     /**
      * @var array
      */
-    protected $options = [
+    protected $optionDefaults = [
         'bootstrap' => [
             'classes' => [
                 'table' => 'table table-bordered table-striped',
@@ -29,7 +35,7 @@ trait Options
      */
     public function getOption($option)
     {
-        return Arr::dot($this->options)[$option] ?? null;
+        return Arr::dot($this->optionDefaults)[$option] ?? null;
     }
 
     /**
@@ -38,7 +44,7 @@ public function getOption($option)
     protected function setOptions(array $overrides = []): void
     {
         foreach ($overrides as $key => $value) {
-            data_set($this->options, $key, $value);
+            data_set($this->optionDefaults, $key, $value);
         }
     }
 }